[clang] [CIR] Make ClangIR compatible with latest nested name specifier AST representation (PR #152846)
Matheus Izvekov via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 9 09:58:31 PDT 2025
================
@@ -138,7 +139,7 @@ isSafeToConvert(const RecordDecl *rd, CIRGenTypes &cgt,
if (!alreadyChecked.insert(rd).second)
return true;
- const Type *key = cgt.getASTContext().getTagDeclType(rd).getTypePtr();
+ const Type *key = cgt.getASTContext().getCanonicalTagType(rd).getTypePtr();
----------------
mizvekov wrote:
FYI slightly unrelated, but it might make sense to `assert(rd->isCompleteDefinition()` since some of the uses here only make sense when applied on the definition, such as the `fields` iteration down below.
https://github.com/llvm/llvm-project/pull/152846
More information about the cfe-commits
mailing list