diff options
| author | Rafael Antognolli <[email protected]> | 2017-07-12 16:36:03 -0700 |
|---|---|---|
| committer | Rafael Antognolli <[email protected]> | 2017-07-24 16:07:25 -0700 |
| commit | 8c47ccb13a198f4d38c772df1de457de34dde23e (patch) | |
| tree | 915374dee7bd1597fab253a02b1ee1225fa9a141 | |
| parent | bbc29393d3beaf6344c7188547b4ff61b63946ae (diff) | |
i965: Use {} to initialize GENX_* structs.
gen4 have commands which start with KernelStartPointer, which is a
struct, so if we initialize it struct = { 0 }, we get warnings on some
compilers:
"GCC (pre 4.9?) can throw a Wmissing-braces on[1] while clang
-Wmissing-field-initializers [2]." - Emil
[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53119
[2] https://bugs.llvm.org/show_bug.cgi?id=21689
This change works around that and will silence such warnings. It is both
a GCC and a clang extension.
v2:
- Use {} instead of memset macro (Matt)
Signed-off-by: Rafael Antognolli <[email protected]>
Cc: Jason Ekstrand <[email protected]>
Cc: Matt Turner <[email protected]>
Cc: Emil Velikov <[email protected]>
Reviewed-by: Francisco Jerez <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/i965/genX_state_upload.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c index ef04603df8e..dacccf78f57 100644 --- a/src/mesa/drivers/dri/i965/genX_state_upload.c +++ b/src/mesa/drivers/dri/i965/genX_state_upload.c @@ -206,7 +206,7 @@ KSP(struct brw_context *brw, uint32_t offset) }) #define brw_state_emit(brw, cmd, align, offset, name) \ - for (struct cmd name = { 0, }, \ + for (struct cmd name = {}, \ *_dst = brw_state_batch(brw, _brw_cmd_length(cmd) * 4, \ align, offset); \ __builtin_expect(_dst != NULL, 1); \ |