michaelmaitland added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.h:162
+ bool hasVInstructionsF16Mininal() const {
+ return HasStdExtZvfhmin || HasStdExtZvfh;
+ }
----------------
jacquesguan wrote:
> craig.topper wrote:
> > Doesn't HasStdExtZvfh already imply HasStdExtZvfhmin?
> The v spec doesn't metion this.
I think the spec conveys this when it says `The Zvfhmin extension depends on
the Zve32f extension.`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150253/new/
https://reviews.llvm.org/D150253
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
