[llvm] [LLVM][Support] Add getTrailingObjects() for single trailing type (PR #138970)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu May 8 09:54:20 PDT 2025


================
@@ -49,9 +50,9 @@ class Class1 final : protected TrailingObjects<Class1, short> {
   using TrailingObjects::getTrailingObjects;
 };
 
-// Here, there are two singular optional object types appended.  Note
-// that the alignment of Class2 is automatically increased to account
-// for the alignment requirements of the trailing objects.
+// Here, there are two singular optional object types appended. Note that the
----------------
dwblaikie wrote:

Looks like this and some other comment wrapping is unrelated change? Could you remove these? (if there's a desire to fix existing problematic formatting, you can send it separately - though we tend to avoid formatting-for-the-sake-of-formatting (except in cases where a file is being substantially changed, where having it be fully formatted to start would avoid new functional changes introducing major formatting changes too, etc)

https://github.com/llvm/llvm-project/pull/138970


More information about the llvm-commits mailing list