[PATCH] D141768: [WIP][11/N][Clang][RISCV][NFC] Remove Policy::PolicyType::Omit
Yueh-Ting (eop) Chen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Jan 14 11:19:52 PST 2023
eopXD updated this revision to Diff 489298.
eopXD added a comment.
Rebase to latset main.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141768/new/
https://reviews.llvm.org/D141768
Files:
clang/include/clang/Support/RISCVVIntrinsicUtils.h
clang/lib/Support/RISCVVIntrinsicUtils.cpp
Index: clang/lib/Support/RISCVVIntrinsicUtils.cpp
===================================================================
--- clang/lib/Support/RISCVVIntrinsicUtils.cpp
+++ clang/lib/Support/RISCVVIntrinsicUtils.cpp
@@ -969,9 +969,8 @@
}
llvm::SmallVector<Policy> RVVIntrinsic::getSupportedUnMaskedPolicies() {
- return {
- Policy(Policy::PolicyType::Undisturbed, Policy::PolicyType::Omit), // TU
- Policy(Policy::PolicyType::Agnostic, Policy::PolicyType::Omit)}; // TA
+ return {Policy(Policy::PolicyType::Undisturbed), // TU
+ Policy(Policy::PolicyType::Agnostic)}; // TA
}
llvm::SmallVector<Policy>
@@ -1029,28 +1028,33 @@
BuiltinName += "_ta";
}
} else {
- if (PolicyAttrs.isTUMAPolicy() && !HasMaskPolicy)
- appendPolicySuffix("_tum");
- else if (PolicyAttrs.isTAMAPolicy() && !HasMaskPolicy)
- appendPolicySuffix("_tam");
- else if (PolicyAttrs.isMUPolicy() && !HasTailPolicy)
- appendPolicySuffix("_mu");
- else if (PolicyAttrs.isMAPolicy() && !HasTailPolicy)
- appendPolicySuffix("_ma");
- else if (PolicyAttrs.isTUMUPolicy())
- appendPolicySuffix("_tumu");
- else if (PolicyAttrs.isTAMUPolicy())
- appendPolicySuffix("_tamu");
- else if (PolicyAttrs.isTUMAPolicy())
- appendPolicySuffix("_tuma");
- else if (PolicyAttrs.isTAMAPolicy())
- appendPolicySuffix("_tama");
- else if (PolicyAttrs.isTUPolicy() && !IsMasked)
- appendPolicySuffix("_tu");
- else if (PolicyAttrs.isTAPolicy() && !IsMasked)
- appendPolicySuffix("_ta");
- else
- llvm_unreachable("Unhandled policy condition");
+ if (IsMasked) {
+ if (PolicyAttrs.isTUMAPolicy() && !HasMaskPolicy)
+ appendPolicySuffix("_tum");
+ else if (PolicyAttrs.isTAMAPolicy() && !HasMaskPolicy)
+ appendPolicySuffix("_tam");
+ else if (PolicyAttrs.isMUPolicy() && !HasTailPolicy)
+ appendPolicySuffix("_mu");
+ else if (PolicyAttrs.isMAPolicy() && !HasTailPolicy)
+ appendPolicySuffix("_ma");
+ else if (PolicyAttrs.isTUMUPolicy())
+ appendPolicySuffix("_tumu");
+ else if (PolicyAttrs.isTAMUPolicy())
+ appendPolicySuffix("_tamu");
+ else if (PolicyAttrs.isTUMAPolicy())
+ appendPolicySuffix("_tuma");
+ else if (PolicyAttrs.isTAMAPolicy())
+ appendPolicySuffix("_tama");
+ else
+ llvm_unreachable("Unhandled policy condition");
+ } else {
+ if (PolicyAttrs.isTUPolicy())
+ appendPolicySuffix("_tu");
+ else if (PolicyAttrs.isTAPolicy())
+ appendPolicySuffix("_ta");
+ else
+ llvm_unreachable("Unhandled policy condition");
+ }
}
}
Index: clang/include/clang/Support/RISCVVIntrinsicUtils.h
===================================================================
--- clang/include/clang/Support/RISCVVIntrinsicUtils.h
+++ clang/include/clang/Support/RISCVVIntrinsicUtils.h
@@ -97,13 +97,13 @@
enum PolicyType {
Undisturbed,
Agnostic,
- Omit, // No policy required.
};
PolicyType TailPolicy = Agnostic;
PolicyType MaskPolicy = Undisturbed;
Policy() : IsUnspecified(true) {}
- Policy(PolicyType _TailPolicy, PolicyType _MaskPolicy)
- : TailPolicy(_TailPolicy), MaskPolicy(_MaskPolicy) {}
+ Policy(PolicyType TailPolicy) : TailPolicy(TailPolicy) {}
+ Policy(PolicyType TailPolicy, PolicyType MaskPolicy)
+ : TailPolicy(TailPolicy), MaskPolicy(MaskPolicy) {}
bool isTAMAPolicy() const {
return TailPolicy == Agnostic && MaskPolicy == Agnostic;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D141768.489298.patch
Type: text/x-patch
Size: 3533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230114/55bb7c76/attachment-0001.bin>
More information about the cfe-commits
mailing list