[PATCH] D25685: [DAG] use isConstOrConstSplat in ComputeNumSignBits to optimize SRA

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 17 11:29:57 PDT 2016


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - I think isConstOrConstSplatFP should probably moved over at the same time as isConstOrConstSplat (separate preliminary commit) followed by the SRA fix - make sense?

We can then investigate adding a "AllowUndefs" args to both of them in a followup commit and start the fun job of consolidating all those predicates.....


https://reviews.llvm.org/D25685





More information about the llvm-commits mailing list