[llvm] 86f2fdd - [TableGen] Tach getInputFilename to return StringRef (NFC) (#140690)

via llvm-commits llvm-commits at lists.llvm.org
Tue May 20 06:47:22 PDT 2025


Author: Kazu Hirata
Date: 2025-05-20T06:47:18-07:00
New Revision: 86f2fdd5e448abbe8d215e2f79a0c388bf7ca2ed

URL: https://github.com/llvm/llvm-project/commit/86f2fdd5e448abbe8d215e2f79a0c388bf7ca2ed
DIFF: https://github.com/llvm/llvm-project/commit/86f2fdd5e448abbe8d215e2f79a0c388bf7ca2ed.diff

LOG: [TableGen] Tach getInputFilename to return StringRef (NFC) (#140690)

AFAICT, all callers of getInputFilename consume the string right away.
Nobody seems to rely on the "copy" behavior that comes with returning
"const std::string".

Added: 
    

Modified: 
    llvm/include/llvm/TableGen/Record.h
    mlir/lib/TableGen/CodeGenHelpers.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/TableGen/Record.h b/llvm/include/llvm/TableGen/Record.h
index 593688e7cfc0e..5849344bcb0b5 100644
--- a/llvm/include/llvm/TableGen/Record.h
+++ b/llvm/include/llvm/TableGen/Record.h
@@ -1981,7 +1981,7 @@ class RecordKeeper {
   detail::RecordKeeperImpl &getImpl() { return *Impl; }
 
   /// Get the main TableGen input file's name.
-  const std::string getInputFilename() const { return InputFilename; }
+  StringRef getInputFilename() const { return InputFilename; }
 
   /// Get the map of classes.
   const RecordMap &getClasses() const { return Classes; }

diff  --git a/mlir/lib/TableGen/CodeGenHelpers.cpp b/mlir/lib/TableGen/CodeGenHelpers.cpp
index 0a07071fea217..f4031be24dfdb 100644
--- a/mlir/lib/TableGen/CodeGenHelpers.cpp
+++ b/mlir/lib/TableGen/CodeGenHelpers.cpp
@@ -27,7 +27,7 @@ using namespace mlir::tblgen;
 static std::string getUniqueOutputLabel(const RecordKeeper &records,
                                         StringRef tag) {
   // Use the input file name when generating a unique name.
-  std::string inputFilename = records.getInputFilename();
+  StringRef inputFilename = records.getInputFilename();
 
   // Drop all but the base filename.
   StringRef nameRef = sys::path::filename(inputFilename);


        


More information about the llvm-commits mailing list