[TableGen] Correct some comments in the TableGen parser [NFC]

[Paul-C-Anagnostopoulos]

Summary

This revision corrects a few comments in the TableGen parser. These are primarily comments about statement syntax.

I think it's okay to accept this revision myself in a day or two?

[dblaikie]

I think it's okay to accept this revision myself in a day or two?

Generally this isn't an acceptable practice - the idea is that if it was worth sending for review, it should be reviewed before being submitted. (otherwise things might get a bit slippery - "I sent it for review but no one's replied, so I'm just going to commit it now")

There is sometimes narrow space for "I'm curious about feedback, but happy to commit this if no one has any" - but a change like this should be easy enough to get someone to say "I don't see anything wrong with this" when you ask.

But, yeah, something like this I think it'd be fine for you to commit without sending it for review in the first place.

[Paul-C-Anagnostopoulos]

Ah, so a trivial change like this doesn't need review. Thanks!

llvm/lib/TableGen/TGParser.cpp

VarInit *TGParser::ParseForeachDeclaration(Init *&ForeachListValue) {
   if (!IterType)
     return nullptr;
 
   return VarInit::get(DeclName, IterType);
 }
 
 /// ParseTemplateArgList - Read a template argument list, which is a non-empty
 /// sequence of template-declarations in <>'s.  If CurRec is non-null, these are
-/// template args for a class, which may or may not be in a multiclass. If null,
-/// these are the template args for a multiclass.
+/// template args for a class. If null, these are the template args for a
+/// multiclass.
 ///
 ///    TemplateArgList ::= '<' Declaration (',' Declaration)* '>'
 ///
 bool TGParser::ParseTemplateArgList(Record *CurRec) {
   assert(Lex.getCode() == tgtok::less && "Not a template arg list!");
   Lex.Lex(); // eat the '<'
 
   Record *TheRecToAddTo = CurRec ? CurRec : &CurMultiClass->Rec;

 }
 
 /// ParseBodyItem - Parse a single item within the body of a def or class.
 ///
 ///   BodyItem ::= Declaration ';'
 ///   BodyItem ::= LET ID OptionalBitList '=' Value ';'
 ///   BodyItem ::= Defvar
 ///   BodyItem ::= Assert
+///
 bool TGParser::ParseBodyItem(Record *CurRec) {
   if (Lex.getCode() == tgtok::Assert)
     return ParseAssert(nullptr, CurRec);
 
   if (Lex.getCode() == tgtok::Defvar)
     return ParseDefvar();
 
   if (Lex.getCode() != tgtok::Let) {
bool TGParser::ParseTopLevelLet(MultiClass *CurMultiClass) {
   LetStack.pop_back();
   return false;
 }
 
 /// ParseMultiClass - Parse a multiclass definition.
 ///
 ///  MultiClassInst ::= MULTICLASS ID TemplateArgList?
 ///                     ':' BaseMultiClassList '{' MultiClassObject+ '}'
+///  MultiClassObject ::= Assert
 ///  MultiClassObject ::= DefInst
-///  MultiClassObject ::= MultiClassInst
 ///  MultiClassObject ::= DefMInst
 ///  MultiClassObject ::= Defvar
 ///  MultiClassObject ::= Foreach
 ///  MultiClassObject ::= If
 ///  MultiClassObject ::= LETCommand '{' ObjectList '}'
 ///  MultiClassObject ::= LETCommand Object
-///  MultiClassObject ::= Assert
 ///
 bool TGParser::ParseMultiClass() {
   assert(Lex.getCode() == tgtok::MultiClass && "Unexpected token");
   Lex.Lex();  // Eat the multiclass token.
 
   if (Lex.getCode() != tgtok::Id)
     return TokError("expected identifier after multiclass for name");
   std::string Name = Lex.getCurStrVal();