Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-11-03 Thread Richard Biener
On Wed, Oct 28, 2015 at 4:23 PM, Ilya Enkovich wrote: > On 23 Oct 13:36, Ilya Enkovich wrote: >> 2015-10-23 13:32 GMT+03:00 Richard Biener : >> > >> > No, we'd get >> > >> > mask_1 = bool != 1; >> > >> > and the 'mask' variable should have

Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-10-28 Thread Ilya Enkovich
On 23 Oct 13:36, Ilya Enkovich wrote: > 2015-10-23 13:32 GMT+03:00 Richard Biener : > > > > No, we'd get > > > > mask_1 = bool != 1; > > > > and the 'mask' variable should have been simplified to 'bool' > > (yes, we'd insert a dead stmt). gimple_build simplifies > >

Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-10-23 Thread Richard Biener
On Thu, Oct 8, 2015 at 5:40 PM, Ilya Enkovich wrote: > Hi, > > This patch replaces integer mask argument for MASK_LOAD ans MASK_STORE calls > with a boolean one. To allow various boolean vector modes assigned by a > target maskload and maskstore optabs were transformed

Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-10-23 Thread Richard Biener
On Fri, Oct 23, 2015 at 12:23 PM, Ilya Enkovich wrote: > 2015-10-23 12:59 GMT+03:00 Richard Biener : >> >> ICK. So what does the above do? It basically preserves the boolean >> condition >> as "mask" unless ... we ought to swap it (formerly

Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-10-23 Thread Ilya Enkovich
2015-10-23 12:59 GMT+03:00 Richard Biener : > > ICK. So what does the above do? It basically preserves the boolean condition > as "mask" unless ... we ought to swap it (formerly easy, just swap arguments > of the cond_expr, now a bit harder, we need to invert the

Re: [mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-10-23 Thread Ilya Enkovich
2015-10-23 13:32 GMT+03:00 Richard Biener : > On Fri, Oct 23, 2015 at 12:23 PM, Ilya Enkovich > wrote: >> 2015-10-23 12:59 GMT+03:00 Richard Biener : >>> >>> ICK. So what does the above do? It basically preserves

[mask-load, patch 1/2] Use boolean predicate for masked loads and store

2015-10-08 Thread Ilya Enkovich
Hi, This patch replaces integer mask argument for MASK_LOAD ans MASK_STORE calls with a boolean one. To allow various boolean vector modes assigned by a target maskload and maskstore optabs were transformed into convert_optab to get mask mode as a second operand. Patch applies on top of