Skip to content

Commit b1f2cf5

Browse files
jurahuljurahul
committed
[LLVM][TableGen] Rename ListInit::getValues() to getElements()
Rename `ListInit::getValues()` to `getElements()` to better match with other `ListInit` members like `getElement`.
1 parent bf1d4a0 commit b1f2cf5

File tree

17 files changed

+60
-56
lines changed

17 files changed

+60
-56
lines changed

‎clang/utils/TableGen/ClangOptionDocEmitter.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ void emitOption(const DocumentedOption &Option, const Record *DocInfo,
368368
R->getValueAsListOfDefs("HelpTextsForVariants")) {
369369
// This is a list of visibilities.
370370
ArrayRef<const Init *> Visibilities =
371-
VisibilityHelp->getValueAsListInit("Visibilities")->getValues();
371+
VisibilityHelp->getValueAsListInit("Visibilities")->getElements();
372372

373373
// See if any of the program's visibilities are in the list.
374374
for (StringRef DocInfoMask :

‎clang/utils/TableGen/NeonEmitter.cpp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -486,7 +486,7 @@ class Intrinsic {
486486
return Idx;
487487
}
488488

489-
bool hasBody() const { return Body && !Body->getValues().empty(); }
489+
bool hasBody() const { return Body && !Body->empty(); }
490490

491491
void setNeededEarly() { NeededEarly = true; }
492492

@@ -1436,14 +1436,14 @@ void Intrinsic::emitBodyAsBuiltinCall() {
14361436
void Intrinsic::emitBody(StringRef CallPrefix) {
14371437
std::vector<std::string> Lines;
14381438

1439-
if (!Body || Body->getValues().empty()) {
1439+
if (!Body || Body->empty()) {
14401440
// Nothing specific to output - must output a builtin.
14411441
emitBodyAsBuiltinCall();
14421442
return;
14431443
}
14441444

14451445
// We have a list of "things to output". The last should be returned.
1446-
for (auto *I : Body->getValues()) {
1446+
for (auto *I : Body->getElements()) {
14471447
if (const auto *SI = dyn_cast<StringInit>(I)) {
14481448
Lines.push_back(replaceParamsIn(SI->getAsString()));
14491449
} else if (const auto *DI = dyn_cast<DagInit>(I)) {

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

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -747,7 +747,7 @@ class ListInit final : public TypedInit,
747747
public FoldingSetNode,
748748
private TrailingObjects<ListInit, const Init *> {
749749
friend TrailingObjects;
750-
unsigned NumValues;
750+
unsigned NumElements;
751751

752752
public:
753753
using const_iterator = const Init *const *;
@@ -769,11 +769,14 @@ class ListInit final : public TypedInit,
769769

770770
void Profile(FoldingSetNodeID &ID) const;
771771

772-
ArrayRef<const Init *> getValues() const {
773-
return ArrayRef(getTrailingObjects(), NumValues);
772+
ArrayRef<const Init *> getElements() const {
773+
return ArrayRef(getTrailingObjects(), NumElements);
774774
}
775+
// ArrayRef<const Init *> getValues() const {
776+
// return getElements();
777+
// }
775778

776-
const Init *getElement(unsigned Idx) const { return getValues()[Idx]; }
779+
const Init *getElement(unsigned Idx) const { return getElements()[Idx]; }
777780

778781
const RecTy *getElementType() const {
779782
return cast<ListRecTy>(getType())->getElementType();
@@ -794,11 +797,11 @@ class ListInit final : public TypedInit,
794797
bool isConcrete() const override;
795798
std::string getAsString() const override;
796799

797-
const_iterator begin() const { return getValues().begin(); }
798-
const_iterator end() const { return getValues().end(); }
800+
const_iterator begin() const { return getElements().begin(); }
801+
const_iterator end() const { return getElements().end(); }
799802

800-
size_t size () const { return NumValues; }
801-
bool empty() const { return NumValues == 0; }
803+
size_t size() const { return NumElements; }
804+
bool empty() const { return NumElements == 0; }
802805

803806
const Init *getBit(unsigned Bit) const override {
804807
llvm_unreachable("Illegal bit reference off list");

‎llvm/lib/TableGen/Record.cpp‎

Lines changed: 21 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -692,18 +692,19 @@ const Init *StringInit::convertInitializerTo(const RecTy *Ty) const {
692692
return nullptr;
693693
}
694694

695-
static void ProfileListInit(FoldingSetNodeID &ID, ArrayRef<const Init *> Range,
695+
static void ProfileListInit(FoldingSetNodeID &ID,
696+
ArrayRef<const Init *> Elements,
696697
const RecTy *EltTy) {
697-
ID.AddInteger(Range.size());
698+
ID.AddInteger(Elements.size());
698699
ID.AddPointer(EltTy);
699700

700-
for (const Init *I : Range)
701-
ID.AddPointer(I);
701+
for (const Init *E : Elements)
702+
ID.AddPointer(E);
702703
}
703704

704705
ListInit::ListInit(ArrayRef<const Init *> Elements, const RecTy *EltTy)
705706
: TypedInit(IK_ListInit, ListRecTy::get(EltTy)),
706-
NumValues(Elements.size()) {
707+
NumElements(Elements.size()) {
707708
llvm::uninitialized_copy(Elements, getTrailingObjects());
708709
}
709710

@@ -729,7 +730,7 @@ const ListInit *ListInit::get(ArrayRef<const Init *> Elements,
729730

730731
void ListInit::Profile(FoldingSetNodeID &ID) const {
731732
const RecTy *EltTy = cast<ListRecTy>(getType())->getElementType();
732-
ProfileListInit(ID, getValues(), EltTy);
733+
ProfileListInit(ID, getElements(), EltTy);
733734
}
734735

735736
const Init *ListInit::convertInitializerTo(const RecTy *Ty) const {
@@ -738,13 +739,13 @@ const Init *ListInit::convertInitializerTo(const RecTy *Ty) const {
738739

739740
if (const auto *LRT = dyn_cast<ListRecTy>(Ty)) {
740741
SmallVector<const Init *, 8> Elements;
741-
Elements.reserve(getValues().size());
742+
Elements.reserve(size());
742743

743744
// Verify that all of the elements of the list are subclasses of the
744745
// appropriate class!
745746
bool Changed = false;
746747
const RecTy *ElementType = LRT->getElementType();
747-
for (const Init *I : getValues())
748+
for (const Init *I : getElements())
748749
if (const Init *CI = I->convertInitializerTo(ElementType)) {
749750
Elements.push_back(CI);
750751
if (CI != I)
@@ -773,7 +774,7 @@ const Init *ListInit::resolveReferences(Resolver &R) const {
773774
Resolved.reserve(size());
774775
bool Changed = false;
775776

776-
for (const Init *CurElt : getValues()) {
777+
for (const Init *CurElt : getElements()) {
777778
const Init *E = CurElt->resolveReferences(R);
778779
Changed |= E != CurElt;
779780
Resolved.push_back(E);
@@ -944,9 +945,10 @@ const Init *UnOpInit::Fold(const Record *CurRec, bool IsFinal) const {
944945
case TAIL:
945946
if (const auto *LHSl = dyn_cast<ListInit>(LHS)) {
946947
assert(!LHSl->empty() && "Empty list in tail");
947-
// Note the +1. We can't just pass the result of getValues()
948+
// Note the slice(1). We can't just pass the result of getElements()
948949
// directly.
949-
return ListInit::get(LHSl->getValues().slice(1), LHSl->getElementType());
950+
return ListInit::get(LHSl->getElements().slice(1),
951+
LHSl->getElementType());
950952
}
951953
break;
952954

@@ -1012,11 +1014,11 @@ const Init *UnOpInit::Fold(const Record *CurRec, bool IsFinal) const {
10121014
[](const ListInit *List) -> std::optional<std::vector<const Init *>> {
10131015
std::vector<const Init *> Flattened;
10141016
// Concatenate elements of all the inner lists.
1015-
for (const Init *InnerInit : List->getValues()) {
1017+
for (const Init *InnerInit : List->getElements()) {
10161018
const auto *InnerList = dyn_cast<ListInit>(InnerInit);
10171019
if (!InnerList)
10181020
return std::nullopt;
1019-
llvm::append_range(Flattened, InnerList->getValues());
1021+
llvm::append_range(Flattened, InnerList->getElements());
10201022
};
10211023
return Flattened;
10221024
};
@@ -1116,7 +1118,7 @@ static const StringInit *interleaveStringList(const ListInit *List,
11161118
SmallString<80> Result(Element->getValue());
11171119
StringInit::StringFormat Fmt = StringInit::SF_String;
11181120

1119-
for (const Init *Elem : List->getValues().drop_front()) {
1121+
for (const Init *Elem : List->getElements().drop_front()) {
11201122
Result.append(Delim->getValue());
11211123
const auto *Element = dyn_cast<StringInit>(Elem);
11221124
if (!Element)
@@ -1138,7 +1140,7 @@ static const StringInit *interleaveIntList(const ListInit *List,
11381140
return nullptr;
11391141
SmallString<80> Result(Element->getAsString());
11401142

1141-
for (const Init *Elem : List->getValues().drop_front()) {
1143+
for (const Init *Elem : List->getElements().drop_front()) {
11421144
Result.append(Delim->getValue());
11431145
const auto *Element = dyn_cast_or_null<IntInit>(
11441146
Elem->convertInitializerTo(IntRecTy::get(RK)));
@@ -1720,7 +1722,7 @@ static const Init *FilterHelper(const Init *LHS, const Init *MHS,
17201722
if (const auto *MHSl = dyn_cast<ListInit>(MHS)) {
17211723
SmallVector<const Init *, 8> NewList;
17221724

1723-
for (const Init *Item : MHSl->getValues()) {
1725+
for (const Init *Item : MHSl->getElements()) {
17241726
const Init *Include = ItemApply(LHS, Item, RHS, CurRec);
17251727
if (!Include)
17261728
return nullptr;
@@ -3092,7 +3094,7 @@ std::vector<const Record *>
30923094
Record::getValueAsListOfDefs(StringRef FieldName) const {
30933095
const ListInit *List = getValueAsListInit(FieldName);
30943096
std::vector<const Record *> Defs;
3095-
for (const Init *I : List->getValues()) {
3097+
for (const Init *I : List->getElements()) {
30963098
if (const auto *DI = dyn_cast<DefInit>(I))
30973099
Defs.push_back(DI->getDef());
30983100
else
@@ -3121,7 +3123,7 @@ std::vector<int64_t>
31213123
Record::getValueAsListOfInts(StringRef FieldName) const {
31223124
const ListInit *List = getValueAsListInit(FieldName);
31233125
std::vector<int64_t> Ints;
3124-
for (const Init *I : List->getValues()) {
3126+
for (const Init *I : List->getElements()) {
31253127
if (const auto *II = dyn_cast<IntInit>(I))
31263128
Ints.push_back(II->getValue());
31273129
else
@@ -3137,7 +3139,7 @@ std::vector<StringRef>
31373139
Record::getValueAsListOfStrings(StringRef FieldName) const {
31383140
const ListInit *List = getValueAsListInit(FieldName);
31393141
std::vector<StringRef> Strings;
3140-
for (const Init *I : List->getValues()) {
3142+
for (const Init *I : List->getElements()) {
31413143
if (const auto *SI = dyn_cast<StringInit>(I))
31423144
Strings.push_back(SI->getValue());
31433145
else

‎llvm/utils/TableGen/AsmMatcherEmitter.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1402,7 +1402,7 @@ void AsmMatcherInfo::buildOperandClasses() {
14021402
CI->Kind = ClassInfo::UserClass0 + Index;
14031403

14041404
const ListInit *Supers = Rec->getValueAsListInit("SuperClasses");
1405-
for (const Init *I : Supers->getValues()) {
1405+
for (const Init *I : Supers->getElements()) {
14061406
const DefInit *DI = dyn_cast<DefInit>(I);
14071407
if (!DI) {
14081408
PrintError(Rec->getLoc(), "Invalid super class reference!");

‎llvm/utils/TableGen/Basic/IntrinsicEmitter.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,7 @@ static TypeSigTy ComputeTypeSignature(const CodeGenIntrinsic &Int) {
301301
const Record *TypeInfo = Int.TheDef->getValueAsDef("TypeInfo");
302302
const ListInit *TypeList = TypeInfo->getValueAsListInit("TypeSig");
303303

304-
for (const auto *TypeListEntry : TypeList->getValues())
304+
for (const auto *TypeListEntry : TypeList->getElements())
305305
TypeSig.emplace_back(cast<IntInit>(TypeListEntry)->getValue());
306306
return TypeSig;
307307
}

‎llvm/utils/TableGen/CallingConvEmitter.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ void CallingConvEmitter::emitAction(const Record *Action, indent Indent,
154154
O << Indent << "static const MCPhysReg " << RLName << "[] = {\n";
155155
O << Indent << " ";
156156
ListSeparator LS;
157-
for (const Init *V : RL->getValues())
157+
for (const Init *V : RL->getElements())
158158
O << LS << getQualifiedRegisterName(V);
159159
O << "\n" << Indent << "};\n";
160160
};

‎llvm/utils/TableGen/CodeGenMapTable.cpp‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ class InstrMap {
137137
"' has empty " +
138138
"`ValueCols' field!");
139139

140-
for (const Init *I : ColValList->getValues()) {
140+
for (const Init *I : ColValList->getElements()) {
141141
const auto *ColI = cast<ListInit>(I);
142142

143143
// Make sure that all the sub-lists in 'ValueCols' have same number of
@@ -228,7 +228,7 @@ void MapTableEmitter::buildRowInstrMap() {
228228
for (const Record *CurInstr : InstrDefs) {
229229
std::vector<const Init *> KeyValue;
230230
const ListInit *RowFields = InstrMapDesc.getRowFields();
231-
for (const Init *RowField : RowFields->getValues()) {
231+
for (const Init *RowField : RowFields->getElements()) {
232232
const RecordVal *RecVal = CurInstr->getValue(RowField);
233233
if (RecVal == nullptr)
234234
PrintFatalError(CurInstr->getLoc(),
@@ -303,7 +303,7 @@ const Record *MapTableEmitter::getInstrForColumn(const Record *KeyInstr,
303303
std::vector<const Init *> KeyValue;
304304

305305
// Construct KeyValue using KeyInstr's values for RowFields.
306-
for (const Init *RowField : RowFields->getValues()) {
306+
for (const Init *RowField : RowFields->getElements()) {
307307
const Init *KeyInstrVal = KeyInstr->getValue(RowField)->getValue();
308308
KeyValue.push_back(KeyInstrVal);
309309
}
@@ -475,7 +475,7 @@ void MapTableEmitter::emitTablesWithFunc(raw_ostream &OS) {
475475
OS << "// " << InstrMapDesc.getName() << "\nLLVM_READONLY\n";
476476
OS << "int " << InstrMapDesc.getName() << "(uint16_t Opcode";
477477
if (ValueCols.size() > 1) {
478-
for (const Init *CF : ColFields->getValues()) {
478+
for (const Init *CF : ColFields->getElements()) {
479479
std::string ColName = CF->getAsUnquotedString();
480480
OS << ", enum " << ColName << " in" << ColName;
481481
}

‎llvm/utils/TableGen/Common/CodeGenDAGPatterns.cpp‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1012,7 +1012,7 @@ std::string TreePredicateFn::getPredCode() const {
10121012
" if (";
10131013

10141014
ListSeparator LS(" && ");
1015-
for (const Init *Val : AddressSpaces->getValues()) {
1015+
for (const Init *Val : AddressSpaces->getElements()) {
10161016
Code += LS;
10171017

10181018
const IntInit *IntVal = dyn_cast<IntInit>(Val);
@@ -1489,7 +1489,7 @@ int PatternToMatch::getPatternComplexity(const CodeGenDAGPatterns &CGP) const {
14891489

14901490
void PatternToMatch::getPredicateRecords(
14911491
SmallVectorImpl<const Record *> &PredicateRecs) const {
1492-
for (const Init *I : Predicates->getValues()) {
1492+
for (const Init *I : Predicates->getElements()) {
14931493
if (const DefInit *Pred = dyn_cast<DefInit>(I)) {
14941494
const Record *Def = Pred->getDef();
14951495
if (!Def->isSubClassOf("Predicate")) {
@@ -1934,7 +1934,7 @@ static unsigned GetNumNodeResults(const Record *Operator,
19341934
const ListInit *LI = Operator->getValueAsListInit("Fragments");
19351935
assert(LI && "Invalid Fragment");
19361936
unsigned NumResults = 0;
1937-
for (const Init *I : LI->getValues()) {
1937+
for (const Init *I : LI->getElements()) {
19381938
const Record *Op = nullptr;
19391939
if (const DagInit *Dag = dyn_cast<DagInit>(I))
19401940
if (const DefInit *DI = dyn_cast<DefInit>(Dag->getOperator()))
@@ -2855,7 +2855,7 @@ TreePattern::TreePattern(const Record *TheRec, const ListInit *RawPat,
28552855
bool isInput, CodeGenDAGPatterns &cdp)
28562856
: TheRecord(TheRec), CDP(cdp), isInputPattern(isInput), HasError(false),
28572857
Infer(*this) {
2858-
for (const Init *I : RawPat->getValues())
2858+
for (const Init *I : RawPat->getElements())
28592859
Trees.push_back(ParseTreePattern(I, ""));
28602860
}
28612861

@@ -3766,7 +3766,7 @@ static bool hasNullFragReference(const DagInit *DI) {
37663766
/// hasNullFragReference - Return true if any DAG in the list references
37673767
/// the null_frag operator.
37683768
static bool hasNullFragReference(const ListInit *LI) {
3769-
for (const Init *I : LI->getValues()) {
3769+
for (const Init *I : LI->getElements()) {
37703770
const DagInit *DI = dyn_cast<DagInit>(I);
37713771
assert(DI && "non-dag in an instruction Pattern list?!");
37723772
if (hasNullFragReference(DI))

‎llvm/utils/TableGen/Common/CodeGenRegisters.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -771,7 +771,7 @@ CodeGenRegisterClass::CodeGenRegisterClass(CodeGenRegBank &RegBank,
771771

772772
// Alternative allocation orders may be subsets.
773773
SetTheory::RecSet Order;
774-
for (auto [Idx, AltOrderElem] : enumerate(AltOrders->getValues())) {
774+
for (auto [Idx, AltOrderElem] : enumerate(AltOrders->getElements())) {
775775
RegBank.getSets().evaluate(AltOrderElem, Order, R->getLoc());
776776
Orders[1 + Idx].append(Order.begin(), Order.end());
777777
// Verify that all altorder members are regclass members.

0 commit comments

Comments
 (0)