[PATCH] D139706: [RISCV][VP] expand vp intrinscs if no +v feature

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h:245
     using VPLegalization = TargetTransformInfo::VPLegalization;
+    if (!ST->hasStdExtV())
+      return VPLegalization(VPLegalization::Discard, VPLegalization::Convert);
----------------
craig.topper wrote:
> craig.topper wrote:
> > What if the target has Zve*?
> This should be hasInstructionsV
Oops I meant `hasVInstructions()`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139706