XFAIL Headers/cxx11.cpp for Hexagon

Chandler Carruth chandlerc at google.com
Tue Jul 2 19:31:10 PDT 2013


On Tue, Jul 2, 2013 at 2:55 PM, Richard Smith <richard at metafoo.co.uk> wrote:

> On Tue, Jul 2, 2013 at 1:59 PM, Jyotsna Verma <jverma at codeaurora.org>
> wrote:
> >>I would still like an explanation of the nature of the failure. Which
> > header or
> >>#include is problematic? Also, this isn't really a C++11 test, it's a
> test
> > for our
> >>built-in headers, and even if it were, I don't see why it'd be reasonable
> > to
> >>XFAIL C++11 tests for Hexagon.
> >
> > We do define __STDC_HOSTED__ for Hexagon which probably shouldn't be.
> > The test includes stdint.h file from lib/Headers which in turn includes
> > system stdint.h if __STDC_HOSTED__ macro is defined. In addition, it also
> > requires some other system headers such as features.h, bits/predefs.h and
> > gnu/stubs-32.h. There is no way for us to make these header files
> available
> > in the build area.
>
> Does this help:
>
> Index: test/Headers/cxx11.cpp
> ===================================================================
> --- test/Headers/cxx11.cpp      (revision 185111)
> +++ test/Headers/cxx11.cpp      (working copy)
> @@ -1,5 +1,5 @@
> -// RUN: %clang -fsyntax-only -std=c++11 %s
> -// RUN: %clang -fsyntax-only -std=c++11 -fmodules %s
> +// RUN: %clang -ffreestanding -fsyntax-only -std=c++11 %s
> +// RUN: %clang -ffreestanding -fsyntax-only -std=c++11 -fmodules %s


FWIW, any test that includes stdint.h should use -ffreestanding. Not sure
how this worked on platforms *other* than Hexagon...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130702/13ac464f/attachment.html>


More information about the cfe-commits mailing list