[llvm] [TableGen] Replace all lingering uses of getName with getEnumName (PR #113731)
Jessica Clarke via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 25 12:51:52 PDT 2024
https://github.com/jrtc27 updated https://github.com/llvm/llvm-project/pull/113731
>From c7c3fe2a1ede845f03e0c99bee0705b5c8b06793 Mon Sep 17 00:00:00 2001
From: Jessica Clarke <jrtc27 at jrtc27.com>
Date: Fri, 25 Oct 2024 20:41:16 +0100
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF-8?q?itial=20version?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Created using spr 1.3.4
---
llvm/utils/TableGen/Common/CodeGenTarget.cpp | 13 -------------
llvm/utils/TableGen/Common/CodeGenTarget.h | 1 -
llvm/utils/TableGen/FastISelEmitter.cpp | 20 ++++++++++----------
3 files changed, 10 insertions(+), 24 deletions(-)
diff --git a/llvm/utils/TableGen/Common/CodeGenTarget.cpp b/llvm/utils/TableGen/Common/CodeGenTarget.cpp
index b358518c4290b0..4e75db689a0b57 100644
--- a/llvm/utils/TableGen/Common/CodeGenTarget.cpp
+++ b/llvm/utils/TableGen/Common/CodeGenTarget.cpp
@@ -47,19 +47,6 @@ MVT::SimpleValueType llvm::getValueType(const Record *Rec) {
return (MVT::SimpleValueType)Rec->getValueAsInt("Value");
}
-StringRef llvm::getName(MVT::SimpleValueType T) {
- switch (T) {
- case MVT::Other:
- return "UNKNOWN";
- case MVT::iPTR:
- return "TLI.getPointerTy()";
- case MVT::iPTRAny:
- return "TLI.getPointerTy()";
- default:
- return getEnumName(T);
- }
-}
-
StringRef llvm::getEnumName(MVT::SimpleValueType T) {
// clang-format off
switch (T) {
diff --git a/llvm/utils/TableGen/Common/CodeGenTarget.h b/llvm/utils/TableGen/Common/CodeGenTarget.h
index c7b44f7028eb5b..8bcb2f677a00b0 100644
--- a/llvm/utils/TableGen/Common/CodeGenTarget.h
+++ b/llvm/utils/TableGen/Common/CodeGenTarget.h
@@ -46,7 +46,6 @@ class CodeGenSubRegIndex;
/// record corresponds to.
MVT::SimpleValueType getValueType(const Record *Rec);
-StringRef getName(MVT::SimpleValueType T);
StringRef getEnumName(MVT::SimpleValueType T);
/// getQualifiedName - Return the name of the specified record, with a
diff --git a/llvm/utils/TableGen/FastISelEmitter.cpp b/llvm/utils/TableGen/FastISelEmitter.cpp
index 17198c85f06009..c73060b3c59884 100644
--- a/llvm/utils/TableGen/FastISelEmitter.cpp
+++ b/llvm/utils/TableGen/FastISelEmitter.cpp
@@ -718,19 +718,19 @@ void FastISelMap::printFunctionDefinitions(raw_ostream &OS) {
const PredMap &PM = RI.second;
OS << "unsigned fastEmit_" << getLegalCName(Opcode) << "_"
- << getLegalCName(std::string(getName(VT))) << "_"
- << getLegalCName(std::string(getName(RetVT))) << "_";
+ << getLegalCName(std::string(getEnumName(VT))) << "_"
+ << getLegalCName(std::string(getEnumName(RetVT))) << "_";
Operands.PrintManglingSuffix(OS, ImmediatePredicates);
OS << "(";
Operands.PrintParameters(OS);
OS << ") {\n";
- emitInstructionCode(OS, Operands, PM, std::string(getName(RetVT)));
+ emitInstructionCode(OS, Operands, PM, std::string(getEnumName(RetVT)));
}
// Emit one function for the type that demultiplexes on return type.
OS << "unsigned fastEmit_" << getLegalCName(Opcode) << "_"
- << getLegalCName(std::string(getName(VT))) << "_";
+ << getLegalCName(std::string(getEnumName(VT))) << "_";
Operands.PrintManglingSuffix(OS, ImmediatePredicates);
OS << "(MVT RetVT";
if (!Operands.empty())
@@ -739,10 +739,10 @@ void FastISelMap::printFunctionDefinitions(raw_ostream &OS) {
OS << ") {\nswitch (RetVT.SimpleTy) {\n";
for (const auto &RI : RM) {
MVT::SimpleValueType RetVT = RI.first;
- OS << " case " << getName(RetVT) << ": return fastEmit_"
+ OS << " case " << getEnumName(RetVT) << ": return fastEmit_"
<< getLegalCName(Opcode) << "_"
- << getLegalCName(std::string(getName(VT))) << "_"
- << getLegalCName(std::string(getName(RetVT))) << "_";
+ << getLegalCName(std::string(getEnumName(VT))) << "_"
+ << getLegalCName(std::string(getEnumName(RetVT))) << "_";
Operands.PrintManglingSuffix(OS, ImmediatePredicates);
OS << "(";
Operands.PrintArguments(OS);
@@ -753,7 +753,7 @@ void FastISelMap::printFunctionDefinitions(raw_ostream &OS) {
} else {
// Non-variadic return type.
OS << "unsigned fastEmit_" << getLegalCName(Opcode) << "_"
- << getLegalCName(std::string(getName(VT))) << "_";
+ << getLegalCName(std::string(getEnumName(VT))) << "_";
Operands.PrintManglingSuffix(OS, ImmediatePredicates);
OS << "(MVT RetVT";
if (!Operands.empty())
@@ -761,7 +761,7 @@ void FastISelMap::printFunctionDefinitions(raw_ostream &OS) {
Operands.PrintParameters(OS);
OS << ") {\n";
- OS << " if (RetVT.SimpleTy != " << getName(RM.begin()->first)
+ OS << " if (RetVT.SimpleTy != " << getEnumName(RM.begin()->first)
<< ")\n return 0;\n";
const PredMap &PM = RM.begin()->second;
@@ -781,7 +781,7 @@ void FastISelMap::printFunctionDefinitions(raw_ostream &OS) {
OS << " switch (VT.SimpleTy) {\n";
for (const auto &TI : TM) {
MVT::SimpleValueType VT = TI.first;
- std::string TypeName = std::string(getName(VT));
+ std::string TypeName = std::string(getEnumName(VT));
OS << " case " << TypeName << ": return fastEmit_"
<< getLegalCName(Opcode) << "_" << getLegalCName(TypeName) << "_";
Operands.PrintManglingSuffix(OS, ImmediatePredicates);
>From 0bb09c6baf63082ec4d155ef6448ed21be84374f Mon Sep 17 00:00:00 2001
From: Jessica Clarke <jrtc27 at jrtc27.com>
Date: Fri, 25 Oct 2024 20:51:42 +0100
Subject: [PATCH 2/2] Fix clang-format
Created using spr 1.3.4
---
llvm/utils/TableGen/FastISelEmitter.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/llvm/utils/TableGen/FastISelEmitter.cpp b/llvm/utils/TableGen/FastISelEmitter.cpp
index c73060b3c59884..2052222cae5e5f 100644
--- a/llvm/utils/TableGen/FastISelEmitter.cpp
+++ b/llvm/utils/TableGen/FastISelEmitter.cpp
@@ -725,7 +725,8 @@ void FastISelMap::printFunctionDefinitions(raw_ostream &OS) {
Operands.PrintParameters(OS);
OS << ") {\n";
- emitInstructionCode(OS, Operands, PM, std::string(getEnumName(RetVT)));
+ emitInstructionCode(OS, Operands, PM,
+ std::string(getEnumName(RetVT)));
}
// Emit one function for the type that demultiplexes on return type.
More information about the llvm-commits
mailing list