r265035 - Fix the comment for the function added in r264998.

Akira Hatanaka via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 31 11:32:39 PDT 2016


Author: ahatanak
Date: Thu Mar 31 13:32:38 2016
New Revision: 265035

URL: http://llvm.org/viewvc/llvm-project?rev=265035&view=rev
Log:
Fix the comment for the function added in r264998.

Modified:
    cfe/trunk/include/clang/Basic/TargetInfo.h

Modified: cfe/trunk/include/clang/Basic/TargetInfo.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/TargetInfo.h?rev=265035&r1=265034&r2=265035&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/TargetInfo.h (original)
+++ cfe/trunk/include/clang/Basic/TargetInfo.h Thu Mar 31 13:32:38 2016
@@ -414,16 +414,18 @@ public:
   /// types for the given target.
   unsigned getSimdDefaultAlign() const { return SimdDefaultAlign; }
 
-  /// Return the alignment (in bits) of the thrown exception object.
+  /// Return the alignment (in bits) of the thrown exception object. This is
+  /// only meaningful for targets that allocate C++ exceptions in a system
+  /// runtime, such as those using the Itanium C++ ABI.
   virtual unsigned getExnObjectAlignment() const {
-    /// Itanium says that an _Unwind_Exception has to be "double-word"
-    /// aligned (and thus the end of it is also so-aligned), meaning 16
-    /// bytes.  Of course, that was written for the actual Itanium,
-    /// which is a 64-bit platform.  Classically, the ABI doesn't really
-    /// specify the alignment on other platforms, but in practice
-    /// libUnwind declares the struct with __attribute__((aligned)), so
-    /// we assume that alignment here.  (It's generally 16 bytes, but
-    /// some targets overwrite it.)
+    // Itanium says that an _Unwind_Exception has to be "double-word"
+    // aligned (and thus the end of it is also so-aligned), meaning 16
+    // bytes.  Of course, that was written for the actual Itanium,
+    // which is a 64-bit platform.  Classically, the ABI doesn't really
+    // specify the alignment on other platforms, but in practice
+    // libUnwind declares the struct with __attribute__((aligned)), so
+    // we assume that alignment here.  (It's generally 16 bytes, but
+    // some targets overwrite it.)
     return getDefaultAlignForAttributeAligned();
   }
 




More information about the cfe-commits mailing list