projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 9b65ffb) | patch
[X86] In X86DAGToDAGISel::PreprocessISelDAG, make sure we don't access N after we...
authorCraig Topper <craig.topper@intel.com>
Wed, 27 Jun 2018 20:58:46 +0000 (20:58 +0000)
committerCraig Topper <craig.topper@intel.com>
Wed, 27 Jun 2018 20:58:46 +0000 (20:58 +0000)
commit880e34ed45aa1bfae2648d5d3426e6525fafecff
tree0367b3d6cc2e5521a62d76b8dc1dde84c6a9a91etree | snapshot
parent9b65ffb0976c5b9590959a6439d338fc5ec0dcebcommit | diff
[X86] In X86DAGToDAGISel::PreprocessISelDAG, make sure we don't access N after we delete it.

If we turn X86ISD::AND into ISD::AND, we delete N. But we were continuing onto the next block of code even though N no longer existed.

Just happened to notice it. I assume asan didn't notice it because we explicitly unpoison deleted nodes and give them a DELETE_NODE opcode.

llvm-svn: 335787
llvm/lib/Target/X86/X86ISelDAGToDAG.cpp diff | blob | history
Domain: System / Toolchain;