diff options
| author | Kenneth Graunke <[email protected]> | 2012-11-26 21:56:06 -0800 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2012-11-28 18:15:57 -0800 |
| commit | db6231fece32a0cec6050ca570a04362036f4f48 (patch) | |
| tree | e208b341841b69869500da6a392902ba43a96ed5 /src/mesa/drivers/dri | |
| parent | 8af8a26480e9e71fb1501b675f21a469c1699b9b (diff) | |
i965/vs: Abort on unsupported opcodes rather than failing.
Final code generation should never fail. This is a bug, and there
should be no user-triggerable cases where this could occur.
Also, we're not going to have a fail() method after the split.
Reviewed-by: Eric Anholt <[email protected]>
Reviewed-by: Anuj Phogat <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4_emit.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp b/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp index b911983c0ac..a2a5975c6b2 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp @@ -641,11 +641,12 @@ vec4_visitor::generate_vs_instruction(vec4_instruction *instruction, default: if (inst->opcode < (int) ARRAY_SIZE(opcode_descs)) { - fail("unsupported opcode in `%s' in VS\n", - opcode_descs[inst->opcode].name); + _mesa_problem(ctx, "Unsupported opcode in `%s' in VS\n", + opcode_descs[inst->opcode].name); } else { - fail("Unsupported opcode %d in VS", inst->opcode); + _mesa_problem(ctx, "Unsupported opcode %d in VS", inst->opcode); } + abort(); } } |