erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land.
Please just note in the comment for the C++11 bit that it is modified by CWG2354, otherwise I think this is good. ================ Comment at: clang/lib/Sema/SemaDeclAttr.cpp:4291 // declared with the register storage class specifier. An // alignment-specifier may also be applied to the declaration of a class // or enumeration type. ---------------- Can you add a comment up near here updating this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121723/new/ https://reviews.llvm.org/D121723 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits