Fix build: the dump() method is only available in Asserts build (NFC)

joker-eph · joker-eph · commit 3e4ad66fc167 · 2023-10-17T16:09:22.000-07:00
diff --git a/mlir/lib/TableGen/Constraint.cpp b/mlir/lib/TableGen/Constraint.cpp
@@ -31,8 +31,7 @@ Constraint::Constraint(const llvm::Record *record)
   } else if (def->isSubClassOf("SuccessorConstraint")) {
     kind = CK_Successor;
   } else if(!def->isSubClassOf("Constraint")) {
-    llvm::errs() << "Expected a constraint but got: \n";
-    def->dump();
+    llvm::errs() << "Expected a constraint but got: \n" << *def << "\n";
     llvm::report_fatal_error("Abort");
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -31,8 +31,7 @@ Constraint::Constraint(const llvm::Record *record)`
`31`	`31`	`} else if (def->isSubClassOf("SuccessorConstraint")) {`
`32`	`32`	`kind = CK_Successor;`
`33`	`33`	`} else if(!def->isSubClassOf("Constraint")) {`
`34`		`- llvm::errs() << "Expected a constraint but got: \n";`
`35`		`- def->dump();`
	`34`	`+ llvm::errs() << "Expected a constraint but got: \n" << *def << "\n";`
`36`	`35`	`llvm::report_fatal_error("Abort");`
`37`	`36`	`}`
`38`	`37`	`}`