efriedma added a comment.

Adding "Z" makes sense.

If you're going to mess with the 64-bit builtins, it's probably worth adding a 
testcase that can be built with gcc to show that int64_t is actually correct.  
You should be able to write a C++ testcase using decltype (declare a variable 
of type "int64_t", then redeclare it as type "decltype(__builtin_bswap64(0))").


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61845/new/

https://reviews.llvm.org/D61845



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to