[PATCH] [AArch64] Change int64_t from 'long long int' to 'long int' for AArch64 target.
Kevin Qin
kevinqindev at gmail.com
Thu Feb 20 03:53:35 PST 2014
Hi Tim, Joerg and other reviewers,
Most 64-bit targets define int64_t as long int, and AArch64 should make same definition to follow LP64 model. In GNU tool chain, int64_t is defined as long int for 64-bit target. So to get consistent with GNU, it's better Changing int64_t from 'long long int' to 'long int', otherwise clang will get different name mangling suffix compared with g++.
Also, this patch will fix the bug in http://llvm.org/bugs/show_bug.cgi?id=18892.
http://llvm-reviews.chandlerc.com/D2845
Files:
lib/AST/ItaniumMangle.cpp
lib/Basic/Targets.cpp
lib/Sema/SemaChecking.cpp
test/CodeGenCXX/aarch64-mangle-neon-vectors.cpp
test/Preprocessor/init.c
utils/TableGen/NeonEmitter.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2845.1.patch
Type: text/x-patch
Size: 11415 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140220/95fb2b58/attachment.bin>
More information about the cfe-commits
mailing list