[clang] 2a16e1f - [7/15][Clang][RISCV][NFC] Correct the default value for Policy to TAMU

via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 24 01:21:00 PST 2023


Author: eopXD
Date: 2023-01-24T01:20:55-08:00
New Revision: 2a16e1ff7d2735001dbe40e607823857f4bedd0e

URL: https://github.com/llvm/llvm-project/commit/2a16e1ff7d2735001dbe40e607823857f4bedd0e
DIFF: https://github.com/llvm/llvm-project/commit/2a16e1ff7d2735001dbe40e607823857f4bedd0e.diff

LOG: [7/15][Clang][RISCV][NFC] Correct the default value for Policy to TAMU

The default value is set to `Omit`, but in fact the value is then
assigned in `updateNamesAndPolicy`, which is set to TUMU when masked and
TAMU when unmasked. This commit demonstrates so and further remove
another dependency of `Omit`.

This is the 7th commit of a patch-set that aims to change the default policy
for RVV intrinsics from TAMU to TAMA.

Please refer to the cover letter in the 1st commit (D141573) for an
overview.

Reviewed By: craig.topper, kito-cheng

Differential Revision: https://reviews.llvm.org/D141759

Added: 
    

Modified: 
    clang/include/clang/Support/RISCVVIntrinsicUtils.h

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Support/RISCVVIntrinsicUtils.h b/clang/include/clang/Support/RISCVVIntrinsicUtils.h
index e64bab60cad2..57faab0fae59 100644
--- a/clang/include/clang/Support/RISCVVIntrinsicUtils.h
+++ b/clang/include/clang/Support/RISCVVIntrinsicUtils.h
@@ -99,8 +99,8 @@ struct Policy {
     Agnostic,
     Omit, // No policy required.
   };
-  PolicyType TailPolicy = Omit;
-  PolicyType MaskPolicy = Omit;
+  PolicyType TailPolicy = Agnostic;
+  PolicyType MaskPolicy = Undisturbed;
   bool HasTailPolicy, HasMaskPolicy;
   Policy(bool HasTailPolicy, bool HasMaskPolicy)
       : IsUnspecified(true), HasTailPolicy(HasTailPolicy),


        


More information about the cfe-commits mailing list