[PATCH] D36592: [BDCE] clear poison generators after turning a value into zero (PR33695, PR34037)
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 11 08:05:45 PDT 2017
spatel updated this revision to Diff 110728.
spatel marked 2 inline comments as done.
spatel added a comment.
1. Use a loop (worklist) rather than recursion to avoid blowing up the stack.
2. Add an exclusion list based on opcodes to stop the search.
3. Add a test to show that the exclusion list works for a call inst.
4. Add a TODO comment with Nuno's suggestion from the bug report to make this better.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6432 bytes
Desc: not available
More information about the llvm-commits