[PATCH] D101938: [RISCV] Initial version of a demand based vsetvli insertion pass.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D101938#2743763 <https://reviews.llvm.org/D101938#2743763>, @arcbbb wrote:

> LGTM.
>
> I think https://reviews.llvm.org/D100932 is still useful here.
> How do you think if I move it into `RISCVInsertVSETVLI.cpp` instead?

I agree it's useful, but we should probably keep it in RISCVCleanupVSETVLI.cpp so we can keep the skipFunction() check in it's runOnMachineFunction and the getOptLevel() check in RISCVTargetMachine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101938