[PATCH] D76289: [NFC][Attr TableGen] Emit Spelling Enum for Attr types if there >1

[Reid Kleckner via Phabricator via cfe-commits]

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