craig.topper added inline comments.
================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:744
static const char *ImpliedExtsZfh[] = {"zfhmin"};
+static const char *ImpliedExtsZve64d[] = {"zve64f", "d"};
+static const char *ImpliedExtsZve64f[] = {"zve64x", "zve32f"};
----------------
I think we need to drop the implication of "f" here and "d" on zve64f. Move
them to V
================
Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.h:195
+ bool hasVInstructionsF32() const { return HasStdExtZve32f && HasStdExtF; }
+ // FIXME: Consider Zfinx in the future
+ bool hasVInstructionsF64() const { return HasStdExtZve64d && HasStdExtD; }
----------------
Zfinx -> Zdinx
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112408/new/
https://reviews.llvm.org/D112408
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
