[PATCH] D112408: [RISCV] Add the zve extension according to the v1.0 spec

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 12:20:06 PDT 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCV.td:225
+def HasStdExtVIntegerEEW32
+    : Predicate<"Subtarget->hasStdExtV() || SubTarget->hasStdExtZve32x()">,
+      AssemblerPredicate<
----------------
StdExtV depends on Zve64d which depends on FeatureStdExtZve64f which depends on FeatureStdExtZve32f which depends on FeatureStdExtZve32x. Do we need to check both hasStdExtV and hasStdExtZve32x here or could we just check hasStdExtZve32x?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112408



More information about the llvm-commits mailing list