[PATCH] D150528: [Clang][Attribute] Improve the AST/diagnoses fidelity of alignas and _Alignas

Wed May 24 05:47:27 PDT 2023

yronglin added a comment.

In D150528#4365688 <https://reviews.llvm.org/D150528#4365688>, @aaron.ballman wrote:

> In general, I think this is looking pretty good, thank you! I'll leave it to @erichkeane to do the final sign-off as attributes code owner since this is making a fair number of changes in that area.

Thanks for your review @aaron.ballman ! BTW, I have two questions:

1. Should we print the cached alignment value when dump AST?
2. The AlignedAttr::getAlignment has a bit redundant with GetAlignOfType(in ExprConstant.cpp ), should we merge these?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150528