rnk added inline comments.
================ Comment at: clang/utils/TableGen/ClangAttrEmitter.cpp:2333 - if (!ElideSpelling) - SpellingEnum = CreateSemanticSpellings(Spellings, SemanticToSyntacticMap); - if (Header) ---------------- I had to call this function during both header and cpp file generation because some code later used the `SemanticToSyntacticMap` it fills in. ================ Comment at: clang/utils/TableGen/ClangAttrEmitter.cpp:2534 if (!ElideSpelling) { assert(!SemanticToSyntacticMap.empty() && "Empty semantic mapping list"); if (Header) ---------------- Won't this assert? I think we come here even if !Header. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76289/new/ https://reviews.llvm.org/D76289 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits