sfertile accepted this revision. sfertile added a comment. This revision is now accepted and ready to land.
LGTM, other then 1 small test update. ================ Comment at: clang/test/Layout/aix-alignof-align-and-pack-attr.cpp:21 +// CHECK: @{{.*}}test2{{.*}}c{{.*}} = global %"struct.test2::C" zeroinitializer, align 2 +} // namespace test2 ---------------- Minor nit: add ``` namespace test3 { struct __attribute__((__aligned__(16))) C { double x; } c; ``` as a third test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107598/new/ https://reviews.llvm.org/D107598 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits