[llvm] [mlir] [TableGen] Tach getInputFilename to return StringRef (NFC) (PR #140690)
via llvm-commits
llvm-commits at lists.llvm.org
Tue May 20 00:28:22 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
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".
---
Full diff: https://github.com/llvm/llvm-project/pull/140690.diff
2 Files Affected:
- (modified) llvm/include/llvm/TableGen/Record.h (+1-1)
- (modified) mlir/lib/TableGen/CodeGenHelpers.cpp (+1-1)
``````````diff
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);
``````````
</details>
https://github.com/llvm/llvm-project/pull/140690
More information about the llvm-commits
mailing list