On Thu, Oct 13, 2011 at 10:40 AM, Artem Shinkarov
<artyom.shinkar...@gmail.com> wrote:
> On Thu, Oct 13, 2011 at 10:23 AM, Richard Guenther
> <richard.guent...@gmail.com> wrote:
>> On Thu, Oct 13, 2011 at 10:59 AM, Mike Stump <mikest...@comcast.net> wrote:
>>> On Oct 12, 2011, at 2:37 PM, Artem Shinkarov wrote:
>>>> This patch fixed PR50704.
>>>>
>>>> gcc/testsuite:
>>>> * gcc.target/i386/warn-vect-op-3.c: Exclude ia32 target.
>>>> * gcc.target/i386/warn-vect-op-1.c: Ditto.
>>>> * gcc.target/i386/warn-vect-op-2.c: Ditto.
>>>>
>>>> Ok for trunk?
>>>
>>> Ok. Is this x32 clean? :-) If not, HJ will offer an even better spelling.
>>
>> I suppose you instead want sth like
>>
>> { dg-require-effective-target lp64 }
>>
>> ?
>>
>
> See our discussion with HJ here:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50704
> /* { dg-do compile { target { ! { ia32 } } } } */ was his idea. As
> far as x32 sets UNITS_PER_WORD to 8, these tests should work fine.
>
> Artem.
>
Ping.
So can I commit the changes?
Thanks,
Artem.
