Skip to content

Commit 8363222

Browse files
jurahuljurahul
committed
[NFC][TableGen] Use StringRef::str() instead of casting
1 parent 802d8d9 commit 8363222

33 files changed

+131
-140
lines changed

‎llvm/include/llvm/TableGen/Record.h‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -745,9 +745,7 @@ class StringInit final : public TypedInit {
745745
return "[{" + Value.str() + "}]";
746746
}
747747

748-
std::string getAsUnquotedString() const override {
749-
return std::string(Value);
750-
}
748+
std::string getAsUnquotedString() const override { return Value.str(); }
751749

752750
const Init *getBit(unsigned Bit) const override {
753751
llvm_unreachable("Illegal bit reference off string");

‎llvm/lib/TableGen/Record.cpp‎

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1787,22 +1787,21 @@ const Init *TernOpInit::Fold(const Record *CurRec) const {
17871787
return Val->getDefInit();
17881788
}
17891789
if (LHSv && MHSv && RHSv) {
1790-
std::string Val = std::string(RHSv->getName());
1790+
std::string Val = RHSv->getName().str();
17911791
if (LHSv->getAsString() == RHSv->getAsString())
1792-
Val = std::string(MHSv->getName());
1792+
Val = MHSv->getName().str();
17931793
return VarInit::get(Val, getType());
17941794
}
17951795
if (LHSs && MHSs && RHSs) {
1796-
std::string Val = std::string(RHSs->getValue());
1796+
std::string Val = RHSs->getValue().str();
17971797

17981798
std::string::size_type found;
17991799
std::string::size_type idx = 0;
18001800
while (true) {
1801-
found = Val.find(std::string(LHSs->getValue()), idx);
1801+
found = Val.find(LHSs->getValue().str(), idx);
18021802
if (found == std::string::npos)
18031803
break;
1804-
Val.replace(found, LHSs->getValue().size(),
1805-
std::string(MHSs->getValue()));
1804+
Val.replace(found, LHSs->getValue().size(), MHSs->getValue().str());
18061805
idx = found + MHSs->getValue().size();
18071806
}
18081807

@@ -2417,7 +2416,7 @@ const RecTy *DefInit::getFieldType(const StringInit *FieldName) const {
24172416
return nullptr;
24182417
}
24192418

2420-
std::string DefInit::getAsString() const { return std::string(Def->getName()); }
2419+
std::string DefInit::getAsString() const { return Def->getName().str(); }
24212420

24222421
static void ProfileVarDefInit(FoldingSetNodeID &ID, const Record *Class,
24232422
ArrayRef<const ArgumentInit *> Args) {

‎llvm/lib/TableGen/SetTheory.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ struct SequenceOp : public SetTheory::Operator {
191191

192192
std::string Format;
193193
if (const auto *SI = dyn_cast<StringInit>(Expr->arg_begin()[0]))
194-
Format = std::string(SI->getValue());
194+
Format = SI->getValue().str();
195195
else
196196
PrintFatalError(Loc, "Format must be a string: " + Expr->getAsString());
197197

‎llvm/lib/TableGen/TGParser.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4326,7 +4326,7 @@ bool TGParser::ParseDefm(MultiClass *CurMultiClass) {
43264326
// through its template argument names. Substs contains a substitution
43274327
// value for each argument, either the value specified or the default.
43284328
// Then we can resolve the template arguments.
4329-
MultiClass *MC = MultiClasses[std::string(Ref.Rec->getName())].get();
4329+
MultiClass *MC = MultiClasses[Ref.Rec->getName().str()].get();
43304330
assert(MC && "Didn't lookup multiclass correctly?");
43314331

43324332
SubstStack Substs;

‎llvm/lib/TableGen/TGParser.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ class TGVarScope {
131131
}
132132

133133
void addVar(StringRef Name, const Init *I) {
134-
bool Ins = Vars.try_emplace(std::string(Name), I).second;
134+
bool Ins = Vars.try_emplace(Name.str(), I).second;
135135
(void)Ins;
136136
assert(Ins && "Local variable already exists");
137137
}

‎llvm/utils/TableGen/AsmMatcherEmitter.cpp‎

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1114,7 +1114,7 @@ bool MatchableInfo::validate(StringRef CommentDelimiter, bool IsAlias) const {
11141114
// Verify that any operand is only mentioned once.
11151115
// We reject aliases and ignore instructions for now.
11161116
if (!IsAlias && TheDef->getValueAsString("AsmMatchConverter").empty() &&
1117-
Tok[0] == '$' && !OperandNames.insert(std::string(Tok)).second) {
1117+
Tok[0] == '$' && !OperandNames.insert(Tok.str()).second) {
11181118
LLVM_DEBUG({
11191119
errs() << "warning: '" << TheDef->getName() << "': "
11201120
<< "ignoring instruction with tied operand '" << Tok << "'\n";
@@ -1170,15 +1170,15 @@ static std::string getEnumNameForToken(StringRef Str) {
11701170
}
11711171

11721172
ClassInfo *AsmMatcherInfo::getTokenClass(StringRef Token) {
1173-
ClassInfo *&Entry = TokenClasses[std::string(Token)];
1173+
ClassInfo *&Entry = TokenClasses[Token.str()];
11741174

11751175
if (!Entry) {
11761176
Classes.emplace_front();
11771177
Entry = &Classes.front();
11781178
Entry->Kind = ClassInfo::Token;
11791179
Entry->ClassName = "Token";
11801180
Entry->Name = "MCK_" + getEnumNameForToken(Token);
1181-
Entry->ValueName = std::string(Token);
1181+
Entry->ValueName = Token.str();
11821182
Entry->PredicateMethod = "<invalid>";
11831183
Entry->RenderMethod = "<invalid>";
11841184
Entry->ParserMethod = "";
@@ -1352,11 +1352,11 @@ void AsmMatcherInfo::buildRegisterClasses(
13521352

13531353
const Init *DiagnosticType = Def->getValueInit("DiagnosticType");
13541354
if (const StringInit *SI = dyn_cast<StringInit>(DiagnosticType))
1355-
CI->DiagnosticType = std::string(SI->getValue());
1355+
CI->DiagnosticType = SI->getValue().str();
13561356

13571357
const Init *DiagnosticString = Def->getValueInit("DiagnosticString");
13581358
if (const StringInit *SI = dyn_cast<StringInit>(DiagnosticString))
1359-
CI->DiagnosticString = std::string(SI->getValue());
1359+
CI->DiagnosticString = SI->getValue().str();
13601360

13611361
// If we have a diagnostic string but the diagnostic type is not specified
13621362
// explicitly, create an anonymous diagnostic type.
@@ -1376,11 +1376,12 @@ void AsmMatcherInfo::buildRegisterClasses(
13761376
assert(CI && "Missing singleton register class info!");
13771377

13781378
if (CI->ValueName.empty()) {
1379-
CI->ClassName = std::string(Rec->getName());
1379+
CI->ClassName = Rec->getName().str();
13801380
CI->Name = "MCK_" + Rec->getName().str();
1381-
CI->ValueName = std::string(Rec->getName());
1382-
} else
1381+
CI->ValueName = Rec->getName().str();
1382+
} else {
13831383
CI->ValueName = CI->ValueName + "," + Rec->getName().str();
1384+
}
13841385
}
13851386
}
13861387

@@ -1413,14 +1414,14 @@ void AsmMatcherInfo::buildOperandClasses() {
14131414
else
14141415
CI->SuperClasses.push_back(SC);
14151416
}
1416-
CI->ClassName = std::string(Rec->getValueAsString("Name"));
1417+
CI->ClassName = Rec->getValueAsString("Name").str();
14171418
CI->Name = "MCK_" + CI->ClassName;
1418-
CI->ValueName = std::string(Rec->getName());
1419+
CI->ValueName = Rec->getName().str();
14191420

14201421
// Get or construct the predicate method name.
14211422
const Init *PMName = Rec->getValueInit("PredicateMethod");
14221423
if (const StringInit *SI = dyn_cast<StringInit>(PMName)) {
1423-
CI->PredicateMethod = std::string(SI->getValue());
1424+
CI->PredicateMethod = SI->getValue().str();
14241425
} else {
14251426
assert(isa<UnsetInit>(PMName) && "Unexpected PredicateMethod field!");
14261427
CI->PredicateMethod = "is" + CI->ClassName;
@@ -1429,7 +1430,7 @@ void AsmMatcherInfo::buildOperandClasses() {
14291430
// Get or construct the render method name.
14301431
const Init *RMName = Rec->getValueInit("RenderMethod");
14311432
if (const StringInit *SI = dyn_cast<StringInit>(RMName)) {
1432-
CI->RenderMethod = std::string(SI->getValue());
1433+
CI->RenderMethod = SI->getValue().str();
14331434
} else {
14341435
assert(isa<UnsetInit>(RMName) && "Unexpected RenderMethod field!");
14351436
CI->RenderMethod = "add" + CI->ClassName + "Operands";
@@ -1438,15 +1439,15 @@ void AsmMatcherInfo::buildOperandClasses() {
14381439
// Get the parse method name or leave it as empty.
14391440
const Init *PRMName = Rec->getValueInit("ParserMethod");
14401441
if (const StringInit *SI = dyn_cast<StringInit>(PRMName))
1441-
CI->ParserMethod = std::string(SI->getValue());
1442+
CI->ParserMethod = SI->getValue().str();
14421443

14431444
// Get the diagnostic type and string or leave them as empty.
14441445
const Init *DiagnosticType = Rec->getValueInit("DiagnosticType");
14451446
if (const StringInit *SI = dyn_cast<StringInit>(DiagnosticType))
1446-
CI->DiagnosticType = std::string(SI->getValue());
1447+
CI->DiagnosticType = SI->getValue().str();
14471448
const Init *DiagnosticString = Rec->getValueInit("DiagnosticString");
14481449
if (const StringInit *SI = dyn_cast<StringInit>(DiagnosticString))
1449-
CI->DiagnosticString = std::string(SI->getValue());
1450+
CI->DiagnosticString = SI->getValue().str();
14501451
// If we have a DiagnosticString, we need a DiagnosticType for use within
14511452
// the matcher.
14521453
if (!CI->DiagnosticString.empty() && CI->DiagnosticType.empty())
@@ -1459,7 +1460,7 @@ void AsmMatcherInfo::buildOperandClasses() {
14591460
// Get or construct the default method name.
14601461
const Init *DMName = Rec->getValueInit("DefaultMethod");
14611462
if (const StringInit *SI = dyn_cast<StringInit>(DMName)) {
1462-
CI->DefaultMethod = std::string(SI->getValue());
1463+
CI->DefaultMethod = SI->getValue().str();
14631464
} else {
14641465
assert(isa<UnsetInit>(DMName) && "Unexpected DefaultMethod field!");
14651466
CI->DefaultMethod = "default" + CI->ClassName + "Operands";
@@ -3052,7 +3053,7 @@ static void emitAsmTiedOperandConstraints(CodeGenTarget &Target,
30523053
AsmMatcherInfo &Info, raw_ostream &OS,
30533054
bool HasOptionalOperands) {
30543055
std::string AsmParserName =
3055-
std::string(Info.AsmParser->getValueAsString("AsmParserClassName"));
3056+
Info.AsmParser->getValueAsString("AsmParserClassName").str();
30563057
OS << "static bool ";
30573058
OS << "checkAsmTiedOperandConstraints(const " << Target.getName()
30583059
<< AsmParserName << "&AsmParser,\n";

‎llvm/utils/TableGen/AsmWriterEmitter.cpp‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ void AsmWriterEmitter::FindUniqueOperandCommands(
192192
InstIdxs[idx].push_back(i);
193193
} else {
194194
UniqueOperandCommands.push_back(std::move(Command));
195-
InstrsForCase.push_back(std::string(Inst.CGI->TheDef->getName()));
195+
InstrsForCase.push_back(Inst.CGI->TheDef->getName().str());
196196
InstIdxs.emplace_back();
197197
InstIdxs.back().push_back(i);
198198

@@ -592,9 +592,9 @@ emitRegisterNameString(raw_ostream &O, StringRef AltName,
592592
// "NoRegAltName" is special. We don't need to do a lookup for that,
593593
// as it's just a reference to the default register name.
594594
if (AltName == "" || AltName == "NoRegAltName") {
595-
AsmName = std::string(Reg.TheDef->getValueAsString("AsmName"));
595+
AsmName = Reg.TheDef->getValueAsString("AsmName").str();
596596
if (AsmName.empty())
597-
AsmName = std::string(Reg.getName());
597+
AsmName = Reg.getName().str();
598598
} else {
599599
// Make sure the register has an alternate name for this index.
600600
std::vector<const Record *> AltNameList =
@@ -612,7 +612,7 @@ emitRegisterNameString(raw_ostream &O, StringRef AltName,
612612
PrintFatalError(Reg.TheDef->getLoc(),
613613
"Register definition missing alt name for '" +
614614
AltName + "'.");
615-
AsmName = std::string(AltNames[Idx]);
615+
AsmName = AltNames[Idx].str();
616616
}
617617
}
618618
StringTable.add(AsmName);
@@ -939,7 +939,7 @@ void AsmWriterEmitter::EmitPrintAliasInstruction(raw_ostream &O) {
939939
}) -
940940
PrintMethods.begin();
941941
if (static_cast<unsigned>(PrintMethodIdx) == PrintMethods.size())
942-
PrintMethods.emplace_back(std::string(PrintMethod), IsPCRel);
942+
PrintMethods.emplace_back(PrintMethod.str(), IsPCRel);
943943
}
944944
}
945945

‎llvm/utils/TableGen/CodeEmitterGen.cpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -309,8 +309,7 @@ CodeEmitterGen::getInstructionCases(const Record *R,
309309
" case " + itostr(DefaultMode) + ": InstBitsByHw = InstBits";
310310
} else {
311311
Case += " case " + itostr(ModeId) +
312-
": InstBitsByHw = InstBits_" +
313-
std::string(HWM.getMode(ModeId).Name);
312+
": InstBitsByHw = InstBits_" + HWM.getMode(ModeId).Name.str();
314313
}
315314
Case += "; break;\n";
316315
}
@@ -362,7 +361,7 @@ void CodeEmitterGen::addInstructionCasesForEncoding(
362361
if (RV.isNonconcreteOK() || RV.getValue()->isComplete())
363362
continue;
364363

365-
Success &= addCodeToMergeInOperand(R, BI, std::string(RV.getName()), Case,
364+
Success &= addCodeToMergeInOperand(R, BI, RV.getName().str(), Case,
366365
BitOffsetCase, Target);
367366
}
368367
// Avoid empty switches.

‎llvm/utils/TableGen/CodeGenMapTable.cpp‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ class InstrMap {
103103

104104
public:
105105
InstrMap(const Record *MapRec) {
106-
Name = std::string(MapRec->getName());
106+
Name = MapRec->getName().str();
107107

108108
// FilterClass - It's used to reduce the search space only to the
109109
// instructions that define the kind of relationship modeled by
@@ -133,8 +133,8 @@ class InstrMap {
133133

134134
// Each instruction map must specify at least one column for it to be valid.
135135
if (ColValList->empty())
136-
PrintFatalError(MapRec->getLoc(), "InstrMapping record `" +
137-
MapRec->getName() + "' has empty " +
136+
PrintFatalError(MapRec->getLoc(), "InstrMapping record `" + Name +
137+
"' has empty " +
138138
"`ValueCols' field!");
139139

140140
for (const Init *I : ColValList->getValues()) {
@@ -144,7 +144,7 @@ class InstrMap {
144144
// elements as the fields in 'ColFields'.
145145
if (ColI->size() != ColFields->size())
146146
PrintFatalError(MapRec->getLoc(),
147-
"Record `" + MapRec->getName() +
147+
"Record `" + Name +
148148
"', field `ValueCols' entries don't match with " +
149149
" the entries in 'ColFields'!");
150150
ValueCols.push_back(ColI);

‎llvm/utils/TableGen/Common/AsmWriterInst.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ struct AsmWriterOperand {
3838
unsigned MIOpNo = 0;
3939

4040
/// Str - For isLiteralTextOperand, this IS the literal text. For
41-
/// isMachineInstrOperand, this is the PrinterMethodName for the operand..
41+
/// isMachineInstrOperand, this is the PrinterMethodName for the operand.
4242
/// For isLiteralStatementOperand, this is the code to insert verbatim
4343
/// into the asm writer.
4444
std::string Str;

0 commit comments

Comments
 (0)