Skip to content

Commit 399d47d

Browse files
committed
C++: Cleanup
1 parent 5e1c67f commit 399d47d

File tree

1 file changed

+8
-21
lines changed
  • cpp/ql/lib/semmle/code/cpp/ir/implementation/aliased_ssa/internal

1 file changed

+8
-21
lines changed

cpp/ql/lib/semmle/code/cpp/ir/implementation/aliased_ssa/internal/AliasedSSA.qll

Lines changed: 8 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -157,23 +157,14 @@ private newtype TMemoryLocation =
157157
) and
158158
languageType = type.getCanonicalLanguageType()
159159
} or
160-
TEntireAllocationMemoryLocation(Allocation var, boolean isMayAccess) {
161-
(
162-
var instanceof IndirectParameterAllocation or
163-
var instanceof DynamicAllocation
164-
) and
165-
(isMayAccess = false or isMayAccess = true)
160+
TEntireAllocationMemoryLocation(Allocation var, Boolean isMayAccess) {
161+
var instanceof IndirectParameterAllocation or
162+
var instanceof DynamicAllocation
166163
} or
167164
TGroupedMemoryLocation(VariableGroup vg, Boolean isMayAccess, Boolean isAll) or
168-
TUnknownMemoryLocation(IRFunction irFunc, boolean isMayAccess) {
169-
isMayAccess = false or isMayAccess = true
170-
} or
171-
TAllNonLocalMemory(IRFunction irFunc, boolean isMayAccess) {
172-
isMayAccess = false or isMayAccess = true
173-
} or
174-
TAllAliasedMemory(IRFunction irFunc, boolean isMayAccess) {
175-
isMayAccess = false or isMayAccess = true
176-
}
165+
TUnknownMemoryLocation(IRFunction irFunc, Boolean isMayAccess) or
166+
TAllNonLocalMemory(IRFunction irFunc, Boolean isMayAccess) or
167+
TAllAliasedMemory(IRFunction irFunc, Boolean isMayAccess)
177168

178169
/**
179170
* Represents the memory location accessed by a memory operand or memory result. In this implementation, the location is
@@ -249,12 +240,8 @@ abstract class AllocationMemoryLocation extends MemoryLocation {
249240
Allocation var;
250241
boolean isMayAccess;
251242

252-
AllocationMemoryLocation() {
253-
this instanceof TMemoryLocation and
254-
isMayAccess = false
255-
or
256-
isMayAccess = true // Just ensures that `isMayAccess` is bound.
257-
}
243+
bindingset[isMayAccess]
244+
AllocationMemoryLocation() { any() }
258245

259246
final override VirtualVariable getVirtualVariable() {
260247
if allocationEscapes(var)

0 commit comments

Comments
 (0)