+ bfe_to_bitops = stage != MESA_SHADER_FRAGMENT + ? BFE_TO_BITOPS : 0; + bitfield_insert = stage != MESA_SHADER_FRAGMENT + ? BITFIELD_INSERT_TO_BFM_BITOPS + : BITFIELD_INSERT_TO_BFM_BFI; I think these would be clearer with the conditions inverted?