Skip to content

Commit a9af28e

Browse files
committed
C++: Drive-by fix. This is not needed anymore after github#16345
1 parent 43c7ac3 commit a9af28e

File tree

1 file changed

+5
-14
lines changed

1 file changed

+5
-14
lines changed

cpp/ql/lib/semmle/code/cpp/ir/dataflow/internal/SsaInternals.qll

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -726,20 +726,11 @@ private predicate indirectConversionFlowStep(Node nFrom, Node nTo) {
726726
nodeToDefOrUse(nTo, sv, bb2, i2, _) and
727727
adjacentDefRead(bb2, i2, sv, _, _)
728728
) and
729-
(
730-
exists(Operand op1, Operand op2, int indirectionIndex, Instruction instr |
731-
hasOperandAndIndex(nFrom, op1, pragma[only_bind_into](indirectionIndex)) and
732-
hasOperandAndIndex(nTo, op2, pragma[only_bind_into](indirectionIndex)) and
733-
instr = op2.getDef() and
734-
conversionFlow(op1, instr, _, _)
735-
)
736-
or
737-
exists(Operand op1, Operand op2, int indirectionIndex, Instruction instr |
738-
hasOperandAndIndex(nFrom, op1, pragma[only_bind_into](indirectionIndex)) and
739-
hasOperandAndIndex(nTo, op2, indirectionIndex - 1) and
740-
instr = op2.getDef() and
741-
isDereference(instr, op1, _)
742-
)
729+
exists(Operand op1, Operand op2, int indirectionIndex, Instruction instr |
730+
hasOperandAndIndex(nFrom, op1, pragma[only_bind_into](indirectionIndex)) and
731+
hasOperandAndIndex(nTo, op2, pragma[only_bind_into](indirectionIndex)) and
732+
instr = op2.getDef() and
733+
conversionFlow(op1, instr, _, _)
743734
)
744735
}
745736

0 commit comments

Comments
 (0)