[PATCH] D36392: [ImplicitNullCheck] Fix the bug when dependent instruction accesses memory

[Serguei Katkov via Phabricator via llvm-commits]

skatkov added a comment.

Thank you Sanjoy!
You asked about when we allowed stores. DependenceMI did not check for stores but it was hidden in canHandle. when I implemented INC based on stores and added alias analysis I removed the check from canHandle but did not add it to DependenceMI. so it was my fault.


https://reviews.llvm.org/D36392