shafik added inline comments.
================
Comment at: clang/lib/Sema/SemaChecking.cpp:16163
+ // Dependent alignment cannot be resolved -> bail out.
+ if (any_of(VD->specific_attrs<AlignedAttr>(),
+ [](auto *A) { return A->isAlignmentDependent(); }))
----------------
Can't we just use `hasDependentAlignment()` which basically does this.
I am curious that we don't have to do this check in every location we use
`getDeclAlign`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151753/new/
https://reviews.llvm.org/D151753
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
