[clang-tools-extra] [flang] [clang] [libc] [libcxx] [compiler-rt] [llvm] [Clang] Generate the GEP instead of adding AST nodes (PR #73730)
Bill Wendling via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 18 15:01:14 PST 2023
================
@@ -8522,6 +8522,22 @@ bool Sema::CheckCountedByAttr(Scope *S, const FieldDecl *FD) {
}
}
+ // We don't support 'counted_by' on flexible array members in substructures.
+ const DeclContext *DC = FD->getParent();
+ while (const auto *RD = dyn_cast<RecordDecl>(DC)) {
+ if (!RD->isAnonymousStructOrUnion() ||
+ !isa<RecordDecl>(RD->getLexicalParent()))
+ break;
+ DC = RD->getLexicalParent();
+ }
----------------
bwendling wrote:
Not really. I'm checking to make sure that the `RecordDecl`'s lexical parent is also a `RecordDecl`. I need to do this, otherwise `DC` will end up as `TranslationUnit` in some cases.
https://github.com/llvm/llvm-project/pull/73730
More information about the cfe-commits
mailing list