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

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 8 21:10:21 PDT 2017

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.


More information about the llvm-commits mailing list