On 01/10/2013 10:45 AM, Ian Romanick wrote:
> On 01/10/2013 12:10 AM, Chad Versace wrote:
>> Signed-off-by: Chad Versace <chad.vers...@linux.intel.com>
>> ---
>> src/mesa/drivers/dri/i965/brw_defines.h | 1 +
>> src/mesa/drivers/dri/i965/brw_fs.h | 7 ++
>> .../dri/i965/brw_fs_channel_expressions.cpp | 29 +++++++-
>> src/mesa/drivers/dri/i965/brw_fs_emit.cpp | 39 ++++++++++-
>> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 78
>> +++++++++++++++++++++-
>> 5 files changed, 149 insertions(+), 5 deletions(-)
>> + if (expr->operands[i]->type->is_vector()) {
>> + switch (expr->operation) {
>> + case ir_binop_pack_half_2x16_split:
>> + case ir_unop_pack_half_2x16:
>> + case ir_unop_unpack_half_2x16:
>> + case ir_unop_unpack_half_2x16_split_x:
>> + case ir_unop_unpack_half_2x16_split_y:
>> + assert(!"WTF");
>
> Classy. :) Maybe (like below)
>
> assert("!not reached: expression operates on scalars only");
>
Um... thanks. I forgot to fix that.
(The WTF was from a particularly frustrating debugging session).
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
