[PATCH] D18479: [CodeGenCXX] Fix ItaniumCXXABI::getAlignmentOfExnObject to return 8-byte alignment on Darwin
John McCall via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 31 09:31:24 PDT 2016
rjmccall added inline comments.
================
Comment at: cfe/trunk/include/clang/Basic/TargetInfo.h:426
@@ +425,3 @@
+ /// we assume that alignment here. (It's generally 16 bytes, but
+ /// some targets overwrite it.)
+ return getDefaultAlignForAttributeAligned();
----------------
This should just be an ordinary comment, not a /// comment.
Please also add this to the documentation comment:
This is only meaningful for targets that allocate C++ exceptions in a system runtime, such as those using the Itanium C++ ABI.
Repository:
rL LLVM
http://reviews.llvm.org/D18479
More information about the cfe-commits
mailing list