[Mesa-dev] [PATCH 2/6] New unary opcodes for ARB_shader_bit_encoding support.

Kenneth Graunke kenneth at whitecape.org
Mon Apr 30 13:40:44 PDT 2012


On 04/30/2012 04:19 AM, Olivier Galibert wrote:
> The opcodes are bf2u, bf2i, bi2f and bu2f, with "b" meaning bitwise
> conversions.
>
> Signed-off-by: Olivier Galibert<galibert at pobox.com>

I can't say I'm crazy about the names bf2u, bf2i, bi2f, and bu2f.  It 
seems like something I could easily mistake for 'b2f', or might not 
understand at a glance.

What about calling them:
- ir_unop_bitcast_f2u
- ir_unop_bitcast_f2i
- ir_unop_bitcast_i2f
- ir_unop_bitcast_u2f

It's more wordy, but "bitcast" immediately jumps out at me and screams 
"this is a bit reinterpretation."  Apparently LLVM also uses that 
terminology.

But it's just a preference.  Either way, patches 1-5 are:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

Once you drop all the Mesa IR stuff from patch 6, it boils down to a few 
lines in st_glsl_to_tgsi, which looks fine too.

Would you like to try and implement it in the i965 driver (FS - 
brw_fs_visitor.cpp and VS - brw_vec4_visitor.cpp)?  If not, I'd be glad to.


More information about the mesa-dev mailing list