[PATCH] D93282: [RISCV] V does not imply F.

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 15 02:11:25 PST 2020


frasercrmck added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoV.td:741
 
+let Predicates = [HasStdExtVAndF] in {
 // Vector Single-Width Floating-Point Add/Subtract Instructions
----------------
craig.topper wrote:
> Can't this just be [HasStdExtV, HasStdExtF]?
I'd vote for separating them too


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93282



More information about the llvm-commits mailing list