[PATCH] D59535: [SelectionDAG] Compute known bits of CopyFromReg

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 11 05:47:40 PDT 2019


lebedev.ri added a comment.

In D59535#1461651 <https://reviews.llvm.org/D59535#1461651>, @dmgreen wrote:

> Thanks. I've reverted in r358113 and will attempt to add some extra tests in.


Yes, **please** add tests (ideally for x86 too.).

>> We need a test for each of these cases. Otherwise we risk having the same problem all over again.
> 
> Definitely agree.
> 
> I guess this is something that will be fixed in global isel. In the meantime, perhaps we could do with something that is more specific about not being visible though, more than just a bitcast.  Happy to help if I can, just let me know.

D60294 <https://reviews.llvm.org/D60294> caught my attention, that may be a //related// fix.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59535/new/

https://reviews.llvm.org/D59535





More information about the llvm-commits mailing list