On Fri, May 3, 2013 at 5:02 PM, Eric Anholt <[email protected]> wrote: > Matt Turner <[email protected]> writes: > >> The Haswell Bspec says "A SIMD16 instruction is not allowed." (but >> 16-wide BFI1 works for me so far). Since GLSL's bitfieldInsert() >> function takes int parameters BFI1 produces the same results in all >> channels, so there's never any reason to emit a 16-wide BFI1. > > Suppose the first 8 pixels didn't take a branch, while the second 8 did, > and you're bitfield inserting in this branch. You'd be using undefined > values for src[0].
You're right. Thanks. I'll send an updated patch. _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
