Rebase & address review comments

Author: jpienaar

mlir/include/mlir/TableGen/CodeGenHelpers.h

@@ -251,12 +251,12 @@ std::string stringify(T &&t) {
       apply(std::forward<T>(t));
 }
 
-/// Helper to generate a C++ streaming error messages from a given message.
+/// Helper to generate a C++ streaming error message from a given message.
 /// Message can contain '{{...}}' placeholders that are substituted with
 /// C-expressions via tgfmt. It would effectively convert:
-///   "Failed to verify {{foo}}"
+///   "failed to verify {{foo}}"
 /// into:
-///   "Failed to verify " << tgfmt(foo, &ctx)
+///   "failed to verify " << tgfmt(foo, &ctx)
 std::string buildErrorStreamingString(
     StringRef message, const FmtContext &ctx,
     ErrorStreamType errorStreamType = ErrorStreamType::InString);

mlir/lib/TableGen/CodeGenHelpers.cpp

@@ -165,7 +165,7 @@ std::string mlir::tblgen::buildErrorStreamingString(
     if (split.second.empty() &&
         errorStreamType == ErrorStreamType::InsideOpError) {
       // To enable having part of string post, this adds a parenthesis before
-      // the last string segment to match the exiting one.
+      // the last string segment to match the existing one.
       os << " << (\"" << split.first;
     } else {
       os << " << \"" << split.first;