Author: eopXD Date: 2023-01-24T00:09:14-08:00 New Revision: f8ec44b2a0e29fa6edf3cc3b48c42e862a112039
URL: https://github.com/llvm/llvm-project/commit/f8ec44b2a0e29fa6edf3cc3b48c42e862a112039 DIFF: https://github.com/llvm/llvm-project/commit/f8ec44b2a0e29fa6edf3cc3b48c42e862a112039.diff LOG: [4/15][Clang][RISCV][NFC] Remove unnecessary logic under RVVIntrinsic::computeBuiltinTypes `PolicyAttrs::IsUnspecified` is modified in `RVV::Intrinsic::computeBuiltInTypes` and used under `RVVIntrinsic::updateNamesAndPolicy`. Suprisingly the modification in `RVV::Intrinsic::computeBuiltInTypes` is completely unnecessary. This commit removes the redundant logic. This is the 4th 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 Differential Revision: https://reviews.llvm.org/D141577 Added: Modified: clang/lib/Support/RISCVVIntrinsicUtils.cpp Removed: ################################################################################ diff --git a/clang/lib/Support/RISCVVIntrinsicUtils.cpp b/clang/lib/Support/RISCVVIntrinsicUtils.cpp index 18b8ca54b7b6a..d67221ee942c8 100644 --- a/clang/lib/Support/RISCVVIntrinsicUtils.cpp +++ b/clang/lib/Support/RISCVVIntrinsicUtils.cpp @@ -918,12 +918,6 @@ llvm::SmallVector<PrototypeDescriptor> RVVIntrinsic::computeBuiltinTypes( // Update PolicyAttrs if need (TA or TAMA) for compute builtin types. if (PolicyAttrs.isMAPolicy()) PolicyAttrs.TailPolicy = Policy::PolicyType::Agnostic; // TAMA - if (PolicyAttrs.isUnspecified()) { - if (!IsMasked) { - PolicyAttrs.IsUnspecified = false; - PolicyAttrs.TailPolicy = Policy::PolicyType::Agnostic; // TA - } - } bool HasPassthruOp = DefaultScheme == PolicyScheme::HasPassthruOperand; if (IsMasked) { // If HasMaskedOffOperand, insert result type as first input operand if _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
