Re: [Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

2015-08-10 Thread Matt Turner
On Mon, Aug 10, 2015 at 11:37 AM, Jason Ekstrand wrote: > On Mon, Aug 10, 2015 at 11:30 AM, Matt Turner wrote: >> On Mon, Aug 10, 2015 at 11:19 AM, Jason Ekstrand >> wrote: >>> On Mon, Aug 10, 2015 at 11:16 AM, Matt Turner wrote: On Mon, Aug 3, 2015 at 5:22 PM, Jason Ekstrand wrote

Re: [Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

2015-08-10 Thread Jason Ekstrand
On Mon, Aug 10, 2015 at 11:30 AM, Matt Turner wrote: > On Mon, Aug 10, 2015 at 11:19 AM, Jason Ekstrand wrote: >> On Mon, Aug 10, 2015 at 11:16 AM, Matt Turner wrote: >>> On Mon, Aug 3, 2015 at 5:22 PM, Jason Ekstrand wrote: On BDW+, the negation source modifier on NOT, AND, OR, and XOR, i

Re: [Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

2015-08-10 Thread Matt Turner
On Mon, Aug 10, 2015 at 11:19 AM, Jason Ekstrand wrote: > On Mon, Aug 10, 2015 at 11:16 AM, Matt Turner wrote: >> On Mon, Aug 3, 2015 at 5:22 PM, Jason Ekstrand wrote: >>> On BDW+, the negation source modifier on NOT, AND, OR, and XOR, is actually >>> a boolean negate and not an integer negate.

Re: [Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

2015-08-10 Thread Jason Ekstrand
On Mon, Aug 10, 2015 at 11:16 AM, Matt Turner wrote: > On Mon, Aug 3, 2015 at 5:22 PM, Jason Ekstrand wrote: >> On BDW+, the negation source modifier on NOT, AND, OR, and XOR, is actually >> a boolean negate and not an integer negate. However, NIR's soruce >> modifiers are the integer version.

Re: [Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

2015-08-10 Thread Matt Turner
On Mon, Aug 3, 2015 at 5:22 PM, Jason Ekstrand wrote: > On BDW+, the negation source modifier on NOT, AND, OR, and XOR, is actually > a boolean negate and not an integer negate. However, NIR's soruce > modifiers are the integer version. We have to resolve it with a MOV prior > to emitting the ac

[Mesa-dev] [PATCH 4/5] i965/vec4_nir: Do boolean source modifier resolves on BDW+

2015-08-03 Thread Jason Ekstrand
On BDW+, the negation source modifier on NOT, AND, OR, and XOR, is actually a boolean negate and not an integer negate. However, NIR's soruce modifiers are the integer version. We have to resolve it with a MOV prior to emitting the actual instruction. This is basically the same thing we do in th