diff options
author | Alon Zakai <azakai@google.com> | 2022-09-28 11:30:56 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-09-28 18:30:56 +0000 |
commit | 7044da7fe6754c54bd7f4930eaa208e3081cabe3 (patch) | |
tree | d3b6169b8a56eb8e7eb2957b4f05f8c317468639 /test/gtest/possible-contents.cpp | |
parent | 7aa2cb9990e7dd3de1ef5831ef8fad348734aa70 (diff) | |
download | binaryen-7044da7fe6754c54bd7f4930eaa208e3081cabe3.tar.gz binaryen-7044da7fe6754c54bd7f4930eaa208e3081cabe3.tar.bz2 binaryen-7044da7fe6754c54bd7f4930eaa208e3081cabe3.zip |
[GUFA] Fix haveIntersection on comparing nullable with non-nullable (#5089)
We compared types and not heap types, so a difference in nullability
confused us. But at that point in the code, we've ruled out nulls, so we
should focus on heap types only.
Diffstat (limited to 'test/gtest/possible-contents.cpp')
-rw-r--r-- | test/gtest/possible-contents.cpp | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/test/gtest/possible-contents.cpp b/test/gtest/possible-contents.cpp index 1e1f0e5c9..933e220bb 100644 --- a/test/gtest/possible-contents.cpp +++ b/test/gtest/possible-contents.cpp @@ -280,9 +280,16 @@ TEST_F(PossibleContentsTest, TestIntersection) { assertHaveIntersection(exactFuncSignatureType, exactFuncSignatureType); assertHaveIntersection(i32Zero, i32One); // TODO: this could be inferred false + // Exact types only differing by nullability can intersect (not on the null, + // but on something else). + assertHaveIntersection(exactAnyref, exactNonNullAnyref); + // Due to subtyping, an intersection might exist. assertHaveIntersection(funcGlobal, funcGlobal); assertHaveIntersection(funcGlobal, exactFuncSignatureType); + assertHaveIntersection(nonNullFuncGlobal, exactFuncSignatureType); + assertHaveIntersection(funcGlobal, exactNonNullFuncSignatureType); + assertHaveIntersection(nonNullFuncGlobal, exactNonNullFuncSignatureType); // Neither is a subtype of the other, but nulls are possible, so a null can be // the intersection. |