Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-09-21 Thread Patrick Palka
On Tue, Sep 20, 2016 at 8:20 AM, Marc Glisse wrote: > On Mon, 19 Sep 2016, Patrick Palka wrote: > >> On Wed, Sep 14, 2016 at 1:58 AM, Marc Glisse wrote: >>> >>> On Fri, 19 Aug 2016, Patrick Palka wrote: >>> On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: > > > integer

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-09-20 Thread Marc Glisse
On Mon, 19 Sep 2016, Patrick Palka wrote: On Wed, Sep 14, 2016 at 1:58 AM, Marc Glisse wrote: On Fri, 19 Aug 2016, Patrick Palka wrote: On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: integer_nonzerop() currently unconditionally returns false for a VECTOR_CST argument. This is conf

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-09-19 Thread Patrick Palka
On Wed, Sep 14, 2016 at 1:58 AM, Marc Glisse wrote: > On Fri, 19 Aug 2016, Patrick Palka wrote: > >> On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka >> wrote: >>> >>> integer_nonzerop() currently unconditionally returns false for a >>> VECTOR_CST argument. This is confusing because one would expe

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-09-13 Thread Marc Glisse
On Fri, 19 Aug 2016, Patrick Palka wrote: On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: integer_nonzerop() currently unconditionally returns false for a VECTOR_CST argument. This is confusing because one would expect that integer_onep(x) => integer_nonzerop(x) for all x but that is cu

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-09-13 Thread Jeff Law
On 08/19/2016 05:30 PM, Patrick Palka wrote: integer_nonzerop() currently unconditionally returns false for a VECTOR_CST argument. This is confusing because one would expect that integer_onep(x) => integer_nonzerop(x) for all x but that is currently not the case. For a VECTOR_CST of all ones i.

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-08-22 Thread Richard Biener
On Mon, Aug 22, 2016 at 9:10 AM, Richard Biener wrote: > On Sat, Aug 20, 2016 at 1:38 AM, Patrick Palka wrote: >> On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: >>> integer_nonzerop() currently unconditionally returns false for a >>> VECTOR_CST argument. This is confusing because one wou

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-08-22 Thread Richard Biener
On Sat, Aug 20, 2016 at 1:38 AM, Patrick Palka wrote: > On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: >> integer_nonzerop() currently unconditionally returns false for a >> VECTOR_CST argument. This is confusing because one would expect that >> integer_onep(x) => integer_nonzerop(x) for

Re: [PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-08-19 Thread Patrick Palka
On Fri, Aug 19, 2016 at 7:30 PM, Patrick Palka wrote: > integer_nonzerop() currently unconditionally returns false for a > VECTOR_CST argument. This is confusing because one would expect that > integer_onep(x) => integer_nonzerop(x) for all x but that is currently > not the case. For a VECTOR_CS

[PATCH] Handle VECTOR_CST in integer_nonzerop()

2016-08-19 Thread Patrick Palka
integer_nonzerop() currently unconditionally returns false for a VECTOR_CST argument. This is confusing because one would expect that integer_onep(x) => integer_nonzerop(x) for all x but that is currently not the case. For a VECTOR_CST of all ones i.e. {1,1,1,1}, integer_onep() returns true but i