[RISCV] Add SiFive extension support

[4vtomat]

Summary

Add SiFive extension support
Depends on D147934

[asb]

clang/include/clang/Support/RISCVVIntrinsicUtils.h

465

Nit: It would better match the surrounding capitalisation to call this RVV_REQ_Xsfvcp

[craig.topper]

Is there a different patch with the .td for these intrinsics?

[4vtomat]

In D147935#4259687, @craig.topper wrote:

Is there a different patch with the .td for these intrinsics?

Yes, it's in this patch: D148223

clang/include/clang/Support/RISCVVIntrinsicUtils.h

465

Got it! Thanks!

[4vtomat]

Resolved Alex's comment.

[craig.topper]

LGTM

[asb]

LGTM

clang/include/clang/Support/RISCVVIntrinsicUtils.h

 };
 
 // RVVRequire should be sync'ed with target features, but only
 // required features used in riscv_vector.td.
 enum RVVRequire : uint8_t {
   RVV_REQ_None = 0,
   RVV_REQ_RV64 = 1 << 0,
   RVV_REQ_FullMultiply = 1 << 1,
+  RVV_REQ_Xsfvcp = 1 << 2,
 
-  LLVM_MARK_AS_BITMASK_ENUM(RVV_REQ_FullMultiply)
+  LLVM_MARK_AS_BITMASK_ENUM(RVV_REQ_Xsfvcp)
 };
 
 // Raw RVV intrinsic info, used to expand later.
 // This struct is highly compact for minimized code size.
 struct RVVIntrinsicRecord {
   // Intrinsic name, e.g. vadd_vv
   const char *Name;
 

clang/utils/TableGen/RISCVVEmitter.cpp

for (auto *R : RV) {
 
     SR.Log2LMULMask = Log2LMULMask;
 
     SR.RequiredExtensions = 0;
     for (auto RequiredFeature : RequiredFeatures) {
       RVVRequire RequireExt = StringSwitch<RVVRequire>(RequiredFeature)
                                   .Case("RV64", RVV_REQ_RV64)
                                   .Case("FullMultiply", RVV_REQ_FullMultiply)
+                                  .Case("Xsfvcp", RVV_REQ_Xsfvcp)
                                   .Default(RVV_REQ_None);
       assert(RequireExt != RVV_REQ_None && "Unrecognized required feature?");
       SR.RequiredExtensions |= RequireExt;
     }
 
     SR.NF = NF;
     SR.HasMasked = HasMasked;
     SR.HasVL = HasVL;