[PATCH] Fix for 2 Sema test failures when AArch64 is the default target.

Keith Walker kwalker at arm.com
Fri Aug 1 08:40:22 PDT 2014


The tests Clang::Sema/statements.c and
Clang::SemaTemplate/instantiate-expr-1.cpp use assembler statements as part
of the test.    A recent change in commit r213935 means that the
constraints/sizes of the registers specified in the assembler statements are
now validated for the AArch64 target.

 

As these tests do not specify a target they use the compiler default target.
However if you build the compiler with AArch64 as the default target then
these 2 tests now fail as they generate the warning : "value size does not
match register size specified by the constraint and modifier" due to the
recently added validation for AArch64.

 

The attached patch addresses this issue by ensuring that the test is run
using the x86_64-pc-linux-gnu triple which matches the assembler in the
tests.

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140801/90793718/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Sema-Test.patch
Type: application/octet-stream
Size: 792 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140801/90793718/attachment.obj>


More information about the cfe-commits mailing list