[PATCH] [AArch64] Change int64_t from 'long long int' to 'long int' for AArch64 target.
Kevin Qin
kevinqindev at gmail.com
Thu Feb 20 23:40:51 PST 2014
Hi Tim,
Thanks for review. I added a new modifier 'W' to represent int64, and change to use it instead of 'LL' in NeonEmitter.cpp. Then I think the effect on 32-bit ARM target would be fixed. Also fixed other bugs you mentioned and added a test case to show how int64_t and uint64_t be mangled for both ARM and AArch64. Please review again.
http://llvm-reviews.chandlerc.com/D2845
CHANGE SINCE LAST DIFF
http://llvm-reviews.chandlerc.com/D2845?vs=7246&id=7272#toc
Files:
include/clang/Basic/Builtins.def
lib/AST/ASTContext.cpp
lib/AST/ItaniumMangle.cpp
lib/Basic/Targets.cpp
lib/Sema/SemaChecking.cpp
test/CodeGenCXX/aarch64-mangle-neon-vectors.cpp
test/CodeGenCXX/int64_uint64.cpp
test/Preprocessor/init.c
utils/TableGen/NeonEmitter.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2845.2.patch
Type: text/x-patch
Size: 11988 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140220/34b8e6ca/attachment.bin>
More information about the llvm-commits
mailing list