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



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

Reply via email to