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; } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits