[Triple] Add llvm::Triple::isRISCV{32,64}

[smd]

Summary

[1/9] patch series to port HWASAN for riscv64

[craig.topper]

LGTM. This makes sense. It may also simplify some of https://reviews.llvm.org/D128612

[MaskRay]

llvm/include/llvm/ADT/Triple.h

862

The order "32-bit RISC-V" is more common.

[smd]

Addressing comments

llvm/include/llvm/ADT/Triple.h

862

Fixed, thanks

[craig.topper]

Can we re-title this revision to something like [Triple] Add isRISCV32 and isRISCV64? First reading of the phrase "add explicit check" makes me think there's an if statement being added somewhere.

[smd]

In D131339#3711544, @craig.topper wrote:

Can we re-title this revision to something like [Triple] Add isRISCV32 and isRISCV64? First reading of the phrase "add explicit check" makes me think there's an if statement being added somewhere.

Sure, should fixed now.

llvm/include/llvm/ADT/Triple.h

bool isPPC32() const {
     return getArch() == Triple::ppc || getArch() == Triple::ppcle;
   }
 
   /// Tests whether the target is 64-bit PowerPC (little and big endian).
   bool isPPC64() const {
     return getArch() == Triple::ppc64 || getArch() == Triple::ppc64le;
   }
 
+  /// Tests whether the target is 32-bit RISC-V.
+  bool isRISCV32() const { return getArch() == Triple::riscv32; }
+
+  /// Tests whether the target is 64-bit RISC-V.
+  bool isRISCV64() const { return getArch() == Triple::riscv64; }
+
   /// Tests whether the target is RISC-V (32- and 64-bit).
-  bool isRISCV() const {
-    return getArch() == Triple::riscv32 || getArch() == Triple::riscv64;
-  }
+  bool isRISCV() const { return isRISCV32() || isRISCV64(); }
 
   /// Tests whether the target is 32-bit SPARC (little and big endian).
   bool isSPARC32() const {
     return getArch() == Triple::sparc || getArch() == Triple::sparcel;
   }
 
   /// Tests whether the target is 64-bit SPARC (big endian).
   bool isSPARC64() const { return getArch() == Triple::sparcv9; }