[PATCH] D53987: [ValueTracking] peek through 2-input shuffles in ComputeNumSignBits

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 172408.
spatel added a comment.

Patch updated:
Copy the DAG logic completely to make sure undefs are handled properly and allow further enhancements.
I added a unit-test to show that the existing code had a bug. 
Also, I fixed the 2-undef bug and added a unit-test with:
https://reviews.llvm.org/rL345988


https://reviews.llvm.org/D53987

Files:
  lib/Analysis/ValueTracking.cpp
  test/Transforms/InstCombine/logical-select.ll
  unittests/Analysis/ValueTrackingTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53987.172408.patch
Type: text/x-patch
Size: 4728 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181102/c28f3b70/attachment.bin>