[Mlir-commits] [llvm] [mlir] [NFC][TableGen] Use `Twine` for Name argument in CodeGenHelpers (PR #163581)

[Craig Topper]

================
@@ -77,9 +77,11 @@ class NamespaceEmitter {
   // }
   //
   // and cannot use "namespace ::mlir::toy".
-  static StringRef trim(StringRef Name) {
-    Name.consume_front("::");
-    return Name;
+  static std::string trim(const Twine &NameTwine) {
----------------
topperc wrote:

Should we just serialize the Twine in the constructor and make this take std::string?

https://github.com/llvm/llvm-project/pull/163581