On Fri, Oct 14, 2011 at 2:57 PM, Richard Guenther
<richard.guent...@gmail.com> wrote:
> 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.
Committed with 179991.
>>
>> Thanks,
>> Artem.
>>
>
