[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