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

Keith Walker kwalker at arm.com
Mon Aug 4 03:25:31 PDT 2014


Committed as r214706

 

From: cfe-commits-bounces at cs.uiuc.edu
[mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of Keith Walker
Sent: 01 August 2014 16:40
To: cfe-commits at cs.uiuc.edu
Subject: [PATCH] Fix for 2 Sema test failures when AArch64 is the default
target.

 

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/20140804/32e9d4e4/attachment.html>


More information about the cfe-commits mailing list