On Fri, Oct 14, 2011 at 3:42 PM, Artem Shinkarov
<artyom.shinkar...@gmail.com> wrote:
> 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?
Yes.
Thanks,
Richard.
>
> Thanks,
> Artem.
>
