diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-06-15 17:16:56 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-06-15 17:16:56 +0000 |
| commit | 587525468d4fc61902965476051019b20fd1621e (patch) | |
| tree | 4ed1b556244d886d7f2e63b1d26d35b9ed1e0cca /clang/unittests/Basic/VirtualFileSystemTest.cpp | |
| parent | f94c108395a3eb0ad785a2433877dc21d79cf6bd (diff) | |
| download | llvm-587525468d4fc61902965476051019b20fd1621e.zip llvm-587525468d4fc61902965476051019b20fd1621e.tar.gz llvm-587525468d4fc61902965476051019b20fd1621e.tar.bz2 | |
[BasicAA] Don't call isKnownNonEqual if we might be have gone through a PHINode.
This is a fix for the test case in PR32314.
Basic Alias Analysis can ask if two nodes are known non-equal after looking through a phi node to find a GEP. isAddOfNonZero saw an add of a constant from the same phi and said that its output couldn't be equal. But Basic Alias Analysis was really asking about the value from the previous loop iteration.
This patch at least makes that case not happen anymore, I'm not sure if there were still other ways this can fail. As was discussed in the bug, it looks like fixing BasicAA would be difficult so this patch seemed like a possible workaround
Differential Revision: https://reviews.llvm.org/D33136
llvm-svn: 305481
Diffstat (limited to 'clang/unittests/Basic/VirtualFileSystemTest.cpp')
0 files changed, 0 insertions, 0 deletions