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

Reply via email to