[clang] d63e492 - [10/15][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy`

via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 24 07:12:40 PST 2023


Author: eopXD
Date: 2023-01-24T07:12:34-08:00
New Revision: d63e492562f255684f0e2b2e4eb14baf17a485d7

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

LOG: [10/15][Clang][RISCV][NFC] Don't need to check for `MaskPolicy` in `isTAPolicy` and `isTUPolicy`

Caller of the two utilities is always companied with a predicate to
check for `!IsMask`, so we don't need to check for the mask policy
here.

This also removes dependency to `Policy::PolicyType::Omit`, which will
be removed.

This is the 10th 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: kito-cheng

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

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 57faab0fae59b..e1571d795158c 100644
--- a/clang/include/clang/Support/RISCVVIntrinsicUtils.h
+++ b/clang/include/clang/Support/RISCVVIntrinsicUtils.h
@@ -126,13 +126,9 @@ struct Policy {
     return TailPolicy == Undisturbed && MaskPolicy == Undisturbed;
   }
 
-  bool isTAPolicy() const {
-    return TailPolicy == Agnostic && MaskPolicy == Omit;
-  }
+  bool isTAPolicy() const { return TailPolicy == Agnostic; }
 
-  bool isTUPolicy() const {
-    return TailPolicy == Undisturbed && MaskPolicy == Omit;
-  }
+  bool isTUPolicy() const { return TailPolicy == Undisturbed; }
 
   bool isMAPolicy() const { return MaskPolicy == Agnostic; }
 


        


More information about the cfe-commits mailing list