r212871 - Require an x86 registered target for this test by splitting it out

Hans Wennborg hans at chromium.org
Thu Jul 17 09:19:08 PDT 2014


On Thu, Jul 17, 2014 at 6:12 AM, Renato Golin <renato.golin at linaro.org> wrote:
> On 17 July 2014 11:51, Artyom Skrobov <Artyom.Skrobov at arm.com> wrote:
>> Should we just remove it for the time being?
>
> Is there a way to put it in an x86 specific directory?

I think Reid tried to fix this in r213180 by passing the --target flag
to this test.

But it's still failing further down, e.g.
http://lab.llvm.org:8011/builders/clang-ppc64-elf-linux2/builds/15641/steps/check-all/logs/Clang%3A%3Acl-x86-flags.c

We could sprinkle more --target options in there, but it's starting to
look weird when we're passing both a --target and /arch: flag, for
example. Maybe we can make lit support REQUIRES: x86-default-target?



More information about the cfe-commits mailing list