[PATCH] Add default armv5te CPU to getLLVMSuffixForArm table

Chris Cadwallader ccadwallader at arxan.com
Mon Mar 4 23:06:28 PST 2013


The attached patch adds the default armv5[t]e target CPU to the getLLVMSuffixForArm table and corrects a typo (missing hyphen). This is needed for clang to generate the proper target triple when -emit-llvm is passed to the driver because arm1026ej-s did not have an entry and returned the default of arm7tdmi. I have some reservation that the default for armv5[t]e makes more sense to be arm1022e to begin with, but I am by no means an expert so this immediately addresses the problem.

<repro>
simple.c:
int main() { return 0;}

clang -O0 -emit-llvm -target armv5te-none-linux-gnueabi -c -o simple.bc simple.c
llvm-dis simple.bc
Observe incorrect target triple of arm-none-linux-gnueabi
</repro>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ToolChain.patch
Type: application/octet-stream
Size: 372 bytes
Desc: ToolChain.patch
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130305/36ccccd0/attachment.obj>


More information about the cfe-commits mailing list