[llvm] 54e8cfe - [RISCV] Simplify some predicate functions in RISCVSubtarget.h. NFC

[Craig Topper via llvm-commits]

Author: Craig Topper
Date: 2023-07-30T12:37:50-07:00
New Revision: 54e8cfe6d6d8add38056bf7e294c75f0d9ff3986

URL: https://github.com/llvm/llvm-project/commit/54e8cfe6d6d8add38056bf7e294c75f0d9ff3986
DIFF: https://github.com/llvm/llvm-project/commit/54e8cfe6d6d8add38056bf7e294c75f0d9ff3986.diff

LOG: [RISCV] Simplify some predicate functions in RISCVSubtarget.h. NFC

Remove some redundancy.

HasStdExtZve32f implies HasStdExtF
HasStdExtZve64d implies HasStdExtD
HasStdExtZvfbfwma implies HasStdExtZvfbfmin

Added: 
    

Modified: 
    llvm/lib/Target/RISCV/RISCVSubtarget.h

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVSubtarget.h b/llvm/lib/Target/RISCV/RISCVSubtarget.h
index 8ec87f6cfc8e6b..599d60a28019b5 100644
--- a/llvm/lib/Target/RISCV/RISCVSubtarget.h
+++ b/llvm/lib/Target/RISCV/RISCVSubtarget.h
@@ -124,7 +124,7 @@ class RISCVSubtarget : public RISCVGenSubtargetInfo {
     return hasStdExtZfhOrZfhmin() || hasStdExtZhinxOrZhinxmin();
   }
   bool hasHalfFPLoadStoreMove() const {
-    return HasStdExtZfh || HasStdExtZfhmin || HasStdExtZfbfmin;
+    return hasStdExtZfhOrZfhmin() || HasStdExtZfbfmin;
   }
   bool is64Bit() const { return IsRV64; }
   MVT getXLenVT() const { return XLenVT; }
@@ -167,13 +167,9 @@ class RISCVSubtarget : public RISCVGenSubtargetInfo {
   bool hasVInstructions() const { return HasStdExtZve32x; }
   bool hasVInstructionsI64() const { return HasStdExtZve64x; }
   bool hasVInstructionsF16() const { return HasStdExtZvfh; }
-  bool hasVInstructionsBF16() const { 
-    return HasStdExtZvfbfmin || HasStdExtZvfbfwma; 
-  }
-  // FIXME: Consider Zfinx in the future
-  bool hasVInstructionsF32() const { return HasStdExtZve32f && HasStdExtF; }
-  // FIXME: Consider Zdinx in the future
-  bool hasVInstructionsF64() const { return HasStdExtZve64d && HasStdExtD; }
+  bool hasVInstructionsBF16() const { return HasStdExtZvfbfmin; }
+  bool hasVInstructionsF32() const { return HasStdExtZve32f; }
+  bool hasVInstructionsF64() const { return HasStdExtZve64d; }
   // F16 and F64 both require F32.
   bool hasVInstructionsAnyF() const { return hasVInstructionsF32(); }
   bool hasVInstructionsFullMultiply() const { return HasStdExtV; }