[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.

Patch updated:

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.


https://reviews.llvm.org/D36592

Files:
  lib/Transforms/Scalar/BDCE.cpp
  test/Transforms/BDCE/invalidate-assumptions.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36592.110728.patch
Type: text/x-patch
Size: 6432 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170811/d9634e16/attachment.bin>


More information about the llvm-commits mailing list