[PATCH] D25691: [DAGCombiner] Add vector demanded elements support to computeKnownBits

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 23 14:31:35 PDT 2016


RKSimon added a comment.

In https://reviews.llvm.org/D25691#577325, @zvi wrote:

> It would be nice if both instcombine and dagcombine shared the same DemandedBits infrastructure, though i don't have a plan to achieve this.


Agreed, we should be trying to add similar features were possible. After this patch and we've matured this a little further on the DAGCombiner side implementing an InstCombine equivalent should be quite a bit quicker.


Repository:
  rL LLVM

https://reviews.llvm.org/D25691





More information about the llvm-commits mailing list