[clang] f8ec44b - [4/15][Clang][RISCV][NFC] Remove unnecessary logic under RVVIntrinsic::computeBuiltinTypes
via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 24 00:09:18 PST 2023
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
More information about the cfe-commits
mailing list