[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/cfe-commits/attachments/20140220/34b8e6ca/attachment.bin>


More information about the cfe-commits mailing list