[PATCH] D93911: [TableGen] Add the assert statement, step 1

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/TableGen/TGParser.cpp:3203
+
+  if (CurMultiClass)
+    assert(false && "assert in multiclass not yet implemented");
----------------
Put curly braces on this to be consistent with the elses


================
Comment at: llvm/lib/TableGen/TGParser.cpp:3204
+  if (CurMultiClass)
+    assert(false && "assert in multiclass not yet implemented");
+
----------------
Can we use a real error here? I assert(false) should usually be llvm_unreachable but since this is something the user can create it looks like its not truly unreachable.


================
Comment at: llvm/lib/TableGen/TGParser.cpp:3208
+    CurRec->addAssertion(ConditionLoc, Condition, Message);
+
+  } else { // at top level
----------------
Drop this blank line


================
Comment at: llvm/lib/TableGen/TGParser.cpp:3311
 ///  MultiClassObject ::= LETCommand Object
-///
 bool TGParser::ParseMultiClass() {
----------------
Why is this line removed?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93911/new/

https://reviews.llvm.org/D93911