KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

i965: Remove default from brw_instruction_name switch to catch missing names. 

The case-range extension is available in clang and gcc at least back to
3.4.0.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
This commit is contained in:
Matt Turner 2014-12-06 14:18:21 -08:00
parent b6a71cbb64
commit f65200ccc9
1 changed files with 5 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/mesa/drivers/dri/i965/brw_shader.cpp
View File

@ -380,12 +380,10 @@ brw_texture_offset(struct gl_context *ctx, int *offsets,
const char *
brw_instruction_name(enum opcode op)
{
char *fallback;
if (op < ARRAY_SIZE(opcode_descs) && opcode_descs[op].name)
return opcode_descs[op].name;
switch (op) {
case BRW_OPCODE_MOV ... BRW_OPCODE_NOP:
assert(opcode_descs[op].name);
return opcode_descs[op].name;
case FS_OPCODE_FB_WRITE:
return "fb_write";
case FS_OPCODE_BLORP_FB_WRITE:
@ -557,14 +555,9 @@ brw_instruction_name(enum opcode op)
return "gs_svb_set_dst_index";
case GS_OPCODE_FF_SYNC_SET_PRIMITIVES:
return "gs_ff_sync_set_primitives";
default:
/* Yes, this leaks. It's in debug code, it should never occur, and if
* it does, you should just add the case to the list above.
*/
asprintf(&fallback, "op%d", op);
return fallback;
}
unreachable("not reached");
}
backend_visitor::backend_visitor(struct brw_context *brw,