Skip to content

Commit f173dc7

Browse files
committed
C++: Use shorter names for new IR predicates. This should hopefully guide users to use these predicates by default.
1 parent 6ead6c6 commit f173dc7

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

cpp/ql/src/semmle/code/cpp/ir/implementation/aliased_ssa/Instruction.qll

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -593,7 +593,7 @@ class InitializeParameterInstruction extends VariableInstruction {
593593
* Holds if this instruction initializes the parameter with index `index`, or
594594
* if `index` is `-1` and this instruction initializes `this`.
595595
*/
596-
final predicate isParameterOrQualifierIndex(int index) {
596+
final predicate hasIndex(int index) {
597597
index >= 0 and index = this.getParameter().getIndex()
598598
or
599599
index = -1 and this.getIRVariable() instanceof IRThisVariable
@@ -617,7 +617,7 @@ class InitializeIndirectionInstruction extends VariableInstruction {
617617
* index `index`, or if `index` is `-1` and this instruction initializes the memory
618618
* pointed to by `this`.
619619
*/
620-
final predicate isParameterOrQualifierIndex(int index) {
620+
final predicate hasIndex(int index) {
621621
index >= 0 and index = this.getParameter().getIndex()
622622
or
623623
index = -1 and this.getIRVariable() instanceof IRThisVariable
@@ -801,7 +801,7 @@ class ReturnIndirectionInstruction extends VariableInstruction {
801801
* Holds if this instruction is the return indirection for the parameter with index `index`, or
802802
* if this instruction is the return indirection for `this` and `index` is `-1`.
803803
*/
804-
final predicate isParameterOrThisIndirection(int index) {
804+
final predicate hasIndex(int index) {
805805
index >= 0 and index = this.getParameter().getIndex()
806806
or
807807
index = -1 and this.isThisIndirection()
@@ -1622,7 +1622,7 @@ class CallInstruction extends Instruction {
16221622
* Gets the argument operand at the specified index, or `this` if `index` is `-1`.
16231623
*/
16241624
pragma[noinline]
1625-
final ArgumentOperand getPositionalOrThisArgumentOperand(int index) {
1625+
final ArgumentOperand getArgumentOperand(int index) {
16261626
index >= 0 and result = getPositionalArgumentOperand(index)
16271627
or
16281628
index = -1 and result = getThisArgumentOperand()
@@ -1632,9 +1632,7 @@ class CallInstruction extends Instruction {
16321632
* Gets the argument at the specified index, or `this` if `index` is `-1`.
16331633
*/
16341634
pragma[noinline]
1635-
final Instruction getPositionalOrThisArgument(int index) {
1636-
result = getPositionalOrThisArgumentOperand(index).getDef()
1637-
}
1635+
final Instruction getArgument(int index) { result = getArgumentOperand(index).getDef() }
16381636

16391637
/**
16401638
* Gets the number of arguments of the call, including the `this` pointer, if any.

0 commit comments

Comments
 (0)