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

[Rahul Joshi]

================
@@ -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) {
----------------
jurahul wrote:

We can. But this version also does not serialize twice, right?

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