Skip to content

Commit b535725

Browse files
nate-chandlernate-chandler
committed
[NFC] SIL: Renamed SpecifyTestInst.
The instruction's spelling was changed to `specify_test` but the class name wasn't updated from `TestSpecificationInst`. Update it here.
1 parent 906d261 commit b535725

18 files changed

+35
-38
lines changed

‎include/swift/SIL/ParseTestSpecification.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ struct Arguments {
201201
struct UnparsedSpecification {
202202
/// The string which specifies the test.
203203
///
204-
/// Not a StringRef because the TestSpecificationInst whose payload is of
204+
/// Not a StringRef because the SpecifyTestInst whose payload is of
205205
/// interest gets deleted.
206206
std::string string;
207207
/// The next non-debug instruction.

‎include/swift/SIL/SILBuilder.h

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1065,11 +1065,10 @@ class SILBuilder {
10651065
return createDebugValue(Loc, src, Var);
10661066
}
10671067

1068-
TestSpecificationInst *
1069-
createTestSpecificationInst(SILLocation Loc,
1070-
StringRef ArgumentsSpecification) {
1071-
return insert(TestSpecificationInst::create(
1072-
getSILDebugLocation(Loc), ArgumentsSpecification, getModule()));
1068+
SpecifyTestInst *createSpecifyTestInst(SILLocation Loc,
1069+
StringRef ArgumentsSpecification) {
1070+
return insert(SpecifyTestInst::create(getSILDebugLocation(Loc),
1071+
ArgumentsSpecification, getModule()));
10731072
}
10741073

10751074
UnownedCopyValueInst *createUnownedCopyValue(SILLocation Loc,

‎include/swift/SIL/SILCloner.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3124,10 +3124,9 @@ void SILCloner<ImplClass>::visitIncrementProfilerCounterInst(
31243124
}
31253125

31263126
template <typename ImplClass>
3127-
void SILCloner<ImplClass>::visitTestSpecificationInst(
3128-
TestSpecificationInst *Inst) {
3127+
void SILCloner<ImplClass>::visitSpecifyTestInst(SpecifyTestInst *Inst) {
31293128
getBuilder().setCurrentDebugScope(getOpScope(Inst->getDebugScope()));
3130-
recordClonedInstruction(Inst, getBuilder().createTestSpecificationInst(
3129+
recordClonedInstruction(Inst, getBuilder().createSpecifyTestInst(
31313130
getOpLocation(Inst->getLoc()),
31323131
Inst->getArgumentsSpecification()));
31333132
}

‎include/swift/SIL/SILInstruction.h

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5309,22 +5309,21 @@ class DebugValueInst final
53095309
}
53105310
};
53115311

5312-
class TestSpecificationInst final
5313-
: public InstructionBase<SILInstructionKind::TestSpecificationInst,
5312+
class SpecifyTestInst final
5313+
: public InstructionBase<SILInstructionKind::SpecifyTestInst,
53145314
NonValueInstruction>,
5315-
private llvm::TrailingObjects<TestSpecificationInst, char> {
5315+
private llvm::TrailingObjects<SpecifyTestInst, char> {
53165316
friend TrailingObjects;
53175317
friend SILBuilder;
53185318

53195319
llvm::StringMap<SILValue> values;
53205320
unsigned ArgumentsSpecificationLength;
53215321

5322-
TestSpecificationInst(SILDebugLocation Loc,
5323-
unsigned ArgumentsSpecificationLength)
5322+
SpecifyTestInst(SILDebugLocation Loc, unsigned ArgumentsSpecificationLength)
53245323
: InstructionBase(Loc),
53255324
ArgumentsSpecificationLength(ArgumentsSpecificationLength) {}
53265325

5327-
static TestSpecificationInst *
5326+
static SpecifyTestInst *
53285327
create(SILDebugLocation Loc, StringRef argumentsSpecification, SILModule &M);
53295328

53305329
public:

‎include/swift/SIL/SILNodes.def

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -842,7 +842,7 @@ NON_VALUE_INST(DebugValueInst, debug_value,
842842
SILInstruction, None, DoesNotRelease)
843843
NON_VALUE_INST(DebugStepInst, debug_step,
844844
SILInstruction, MayHaveSideEffects, DoesNotRelease)
845-
NON_VALUE_INST(TestSpecificationInst, specify_test,
845+
NON_VALUE_INST(SpecifyTestInst, specify_test,
846846
SILInstruction, None, DoesNotRelease)
847847
#define NEVER_OR_SOMETIMES_LOADABLE_CHECKED_REF_STORAGE(Name, name, ...) \
848848
NON_VALUE_INST(Store##Name##Inst, store_##name, \

‎lib/IRGen/IRGenSIL.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1440,7 +1440,7 @@ class IRGenSILFunction :
14401440
void visitLinearFunctionExtractInst(LinearFunctionExtractInst *i);
14411441
void visitDifferentiabilityWitnessFunctionInst(
14421442
DifferentiabilityWitnessFunctionInst *i);
1443-
void visitTestSpecificationInst(TestSpecificationInst *i) {
1443+
void visitSpecifyTestInst(SpecifyTestInst *i) {
14441444
llvm_unreachable("test-only instruction in Lowered SIL?!");
14451445
}
14461446

‎lib/SIL/IR/OperandOwnership.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ SHOULD_NEVER_VISIT_INST(ReleaseValueAddr)
125125
SHOULD_NEVER_VISIT_INST(StrongRelease)
126126
SHOULD_NEVER_VISIT_INST(GetAsyncContinuation)
127127
SHOULD_NEVER_VISIT_INST(IncrementProfilerCounter)
128-
SHOULD_NEVER_VISIT_INST(TestSpecification)
128+
SHOULD_NEVER_VISIT_INST(SpecifyTest)
129129
SHOULD_NEVER_VISIT_INST(ScalarPackIndex)
130130
SHOULD_NEVER_VISIT_INST(Vector)
131131

‎lib/SIL/IR/SILInstructions.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -546,15 +546,15 @@ IncrementProfilerCounterInst *IncrementProfilerCounterInst::create(
546546
return Inst;
547547
}
548548

549-
TestSpecificationInst *
550-
TestSpecificationInst::create(SILDebugLocation Loc,
551-
StringRef ArgumentsSpecification, SILModule &M) {
549+
SpecifyTestInst *SpecifyTestInst::create(SILDebugLocation Loc,
550+
StringRef ArgumentsSpecification,
551+
SILModule &M) {
552552
auto ArgumentsSpecificationLength = ArgumentsSpecification.size();
553553
auto Size = totalSizeToAlloc<char>(ArgumentsSpecificationLength);
554-
auto Buffer = M.allocateInst(Size, alignof(TestSpecificationInst));
554+
auto Buffer = M.allocateInst(Size, alignof(SpecifyTestInst));
555555

556556
auto *Inst =
557-
::new (Buffer) TestSpecificationInst(Loc, ArgumentsSpecificationLength);
557+
::new (Buffer) SpecifyTestInst(Loc, ArgumentsSpecificationLength);
558558
std::uninitialized_copy(ArgumentsSpecification.begin(),
559559
ArgumentsSpecification.end(),
560560
Inst->getTrailingObjects<char>());

‎lib/SIL/IR/SILPrinter.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2679,7 +2679,7 @@ class SILPrinter : public SILInstructionVisitor<SILPrinter> {
26792679
*this << getIDAndType(RI->getOperand());
26802680
}
26812681

2682-
void visitTestSpecificationInst(TestSpecificationInst *TSI) {
2682+
void visitSpecifyTestInst(SpecifyTestInst *TSI) {
26832683
*this << QuotedString(TSI->getArgumentsSpecification());
26842684
}
26852685

‎lib/SIL/Parser/ParseSIL.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3410,7 +3410,7 @@ bool SILParser::parseSpecificSILInstruction(SILBuilder &B,
34103410
ResultVal = B.createDebugStep(InstLoc);
34113411
break;
34123412

3413-
case SILInstructionKind::TestSpecificationInst: {
3413+
case SILInstructionKind::SpecifyTestInst: {
34143414
// Parse the specification string.
34153415
if (P.Tok.getKind() != tok::string_literal) {
34163416
P.diagnose(P.Tok, diag::expected_sil_specify_test_body);
@@ -3421,7 +3421,7 @@ bool SILParser::parseSpecificSILInstruction(SILBuilder &B,
34213421
auto ArgumentsSpecification =
34223422
P.Tok.getText().drop_front(numQuotes).drop_back(numQuotes).trim();
34233423
P.consumeToken(tok::string_literal);
3424-
auto *tsi = B.createTestSpecificationInst(InstLoc, ArgumentsSpecification);
3424+
auto *tsi = B.createSpecifyTestInst(InstLoc, ArgumentsSpecification);
34253425
SmallVector<StringRef, 4> components;
34263426
test::getTestSpecificationComponents(ArgumentsSpecification, components);
34273427
for (auto component : components) {

0 commit comments

Comments
 (0)