Skip to content

Commit bc5dd0a

Browse files
LiedtkeV8-internal LUCI CQ
authored andcommitted
[cleanup] Remove duplicate check for nil
I guess, I was a bit overly cautious when writing that condition... Change-Id: Ic0b99d65b50b1a97cf75f668fd325b4cc137316c Reviewed-on: https://chrome-internal-review.googlesource.com/c/v8/fuzzilli/+/8468856 Reviewed-by: Dominik Klemba <[email protected]> Commit-Queue: Dominik Klemba <[email protected]> Auto-Submit: Matthias Liedtke <[email protected]>
1 parent e0ccfda commit bc5dd0a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Sources/Fuzzilli/FuzzIL/TypeSystem.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -649,7 +649,7 @@ public struct ILType: Hashable {
649649
let commonProperties = self.properties.intersection(other.properties)
650650
let commonMethods = self.methods.intersection(other.methods)
651651
let signature = self.signature == other.signature ? self.signature : nil // TODO: this is overly coarse, we could also see if one signature subsumes the other, then take the subsuming one.
652-
let receiver = other.receiver != nil && other.receiver != nil ? self.receiver?.intersection(with: other.receiver!) : nil
652+
let receiver = other.receiver != nil ? self.receiver?.intersection(with: other.receiver!) : nil
653653
var group = self.group == other.group ? self.group : nil
654654
let wasmExt = self.wasmType != nil && other.wasmType != nil ? self.wasmType!.union(other.wasmType!) : nil
655655
// Object groups are used to describe certain wasm types. If the WasmTypeExtension is lost,
@@ -740,7 +740,7 @@ public struct ILType: Hashable {
740740
return .nothing
741741
}
742742

743-
let receiver = self.receiver != nil && other.receiver != nil ? self.receiver?.union(with: other.receiver!) : self.receiver ?? other.receiver
743+
let receiver = self.receiver != nil && other.receiver != nil ? self.receiver!.union(with: other.receiver!) : self.receiver ?? other.receiver
744744

745745
// If either value is nil, the result is the non-nil value. If both are non-nil, the result
746746
// is their intersection if valid, otherwise .nothing is returned.

0 commit comments

Comments
 (0)