[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