[PATCH] D38967: [SelectionDAG] Don't subject ISD:Constant to the depth limit in TargetLowering::SimplifyDemandedBits.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 16 14:19:10 PDT 2017


RKSimon added a comment.

In https://reviews.llvm.org/D38967#898866, @craig.topper wrote:

> Doesn't look like SimplifyDemandedBits or computeKnownBits currently handling ConstantFP. We probably don't cross any fp->integer boundaries when recursing. I definitely see an early out in ISD::BITCAST handling if its a cast from FP.


I think it's mostly as we don't have a use case for handling known bits of floats, so we just early out.


https://reviews.llvm.org/D38967





More information about the llvm-commits mailing list