r209878 - clang/test/Frontend/Weverything-and-remarks.cpp: Add explicit -target.

NAKAMURA Takumi geek4civic at gmail.com
Fri May 30 06:38:30 PDT 2014


Diego, thanks.

Sorry that I introduced x86-specific flag.

2014-05-30 22:23 GMT+09:00 Diego Novillo <dnovillo at google.com>:
> I am removing this test. It was temporary while the -Rpass front end patch
> was being reviewed. I simply forgot to remove it when the patch landed.
>
>
> On Fri, May 30, 2014 at 9:16 AM, David Blaikie <dblaikie at gmail.com> wrote:
>>
>> I assume/guess/hope this test could be simplified, too - it looks like
>> it contains a fair few artefacts of whatever the original bug report
>> was that are hopefully not strictly necessary to reproduce the issue
>> in a minimal way.
>>
>> On Fri, May 30, 2014 at 3:01 AM, Renato Golin <renato.golin at linaro.org>
>> wrote:
>> > On 30 May 2014 08:03, NAKAMURA Takumi <geek4civic at gmail.com> wrote:
>> >> +// RUN: %clang -target x86_64-unknown-unknown -c -S -Weverything -O0
>> >> -o /dev/null %s 2> %t.err
>> >
>> > Hi Nakamura,
>> >
>> > This is generating errors on ARM:
>> >
>> >
>> > http://lab.llvm.org:8011/builders/clang-native-arm-cortex-a9/builds/18879/steps/check-all/logs/Clang%3A%3AWeverything-and-remarks.cpp
>> >
>> > I don't have access to the boards to tell you what the errors are, but
>> > it does seem a fragile way to fix this test. Maybe splitting into
>> > multiple tests, and XFAIL on Windows, or something?
>> >
>> > cheers,
>> > --renato
>> > _______________________________________________
>> > cfe-commits mailing list
>> > cfe-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>



More information about the cfe-commits mailing list