[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/llvm-commits/attachments/20140220/95fb2b58/attachment.bin>


More information about the llvm-commits mailing list