[PATCH] D140347: TargetLowering: Teach DemandedBits about VSCALE

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/vscale-demanded-bits.ll:47
+
+declare void @llvm.masked.store.nxv8i8.p0(<vscale x 8 x i8>, ptr nocapture, i32 immarg, <vscale x 8 x i1>)
----------------
compnerd wrote:
> craig.topper wrote:
> > Don't you need a vscale_range attribute to hit the bug?
> I thought so, except, removing the attributes somehow still reproduces the difference?  I don't think that the reduction is the best, and I'm happy to restore the attribute if you like.
I thought in my testing that it won't make it to the APInt multiply without the attribute?


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

https://reviews.llvm.org/D140347