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

[James Molloy]

Hi Keith,

 

LGTM.

 

Cheers,

 

James

 

From: Keith Walker 
Sent: 01 August 2014 16:40
To: cfe-commits at cs.uiuc.edu
Cc: James Molloy; gaborb at inf.u-szeged.hu
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/20140801/6e4b3574/attachment.html>