[PATCH] D124006: [DebugInfo] Use the 'getTypeAlignIfRequired' function to get DW_AT_alignment correct when attribute((__aligned__)) is present.

Ying Yi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 22 02:41:05 PDT 2022


MaggieYi added inline comments.


================
Comment at: clang/test/CodeGenCXX/debug-info-struct-align.cpp:11
+} __attribute__((aligned(1)));
+struct MyType mt;
+
----------------
dblaikie wrote:
> You can drop the "struct" here and from other references to these types (in mt1/mt2 and the alignof calls)
Thanks David. All unnecessary `struct`s have been removed. 


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

https://reviews.llvm.org/D124006



More information about the cfe-commits mailing list