diff --git a/clang/include/clang/Support/RISCVVIntrinsicUtils.h b/clang/include/clang/Support/RISCVVIntrinsicUtils.h --- a/clang/include/clang/Support/RISCVVIntrinsicUtils.h +++ b/clang/include/clang/Support/RISCVVIntrinsicUtils.h @@ -93,7 +93,7 @@ }; struct Policy { - bool PolicyNone = false; + bool IsUnspecified = false; enum PolicyType { Undisturbed, Agnostic, @@ -102,7 +102,7 @@ PolicyType TailPolicy = Omit; PolicyType MaskPolicy = Omit; bool IntrinsicWithoutMU = false; - Policy() : PolicyNone(true) {} + Policy() : IsUnspecified(true) {} Policy(PolicyType _TailPolicy, PolicyType _MaskPolicy, bool _IntrinsicWithoutMU = false) : TailPolicy(_TailPolicy), MaskPolicy(_MaskPolicy), @@ -150,11 +150,11 @@ return MaskPolicy == Undisturbed && TailPolicy == Omit; } - bool isPolicyNonePolicy() const { return PolicyNone; } + bool isUnspecified() const { return IsUnspecified; } bool operator==(const Policy &Other) const { - return PolicyNone == Other.PolicyNone && TailPolicy == Other.TailPolicy && - MaskPolicy == Other.MaskPolicy && + return IsUnspecified == Other.IsUnspecified && + TailPolicy == Other.TailPolicy && MaskPolicy == Other.MaskPolicy && IntrinsicWithoutMU == Other.IntrinsicWithoutMU; } @@ -434,7 +434,7 @@ return IntrinsicTypes; } Policy getPolicyAttrs() const { - assert(PolicyAttrs.PolicyNone == false); + assert(PolicyAttrs.IsUnspecified == false); return PolicyAttrs; } unsigned getPolicyAttrsBits() const { @@ -444,7 +444,7 @@ // constexpr unsigned TAIL_AGNOSTIC_MASK_AGNOSTIC = 3; // FIXME: how about value 2 // int PolicyAttrs = TAIL_UNDISTURBED; - assert(PolicyAttrs.PolicyNone == false); + assert(PolicyAttrs.IsUnspecified == false); if (PolicyAttrs.isTUMAPolicy()) return 2; diff --git a/clang/lib/Support/RISCVVIntrinsicUtils.cpp b/clang/lib/Support/RISCVVIntrinsicUtils.cpp --- a/clang/lib/Support/RISCVVIntrinsicUtils.cpp +++ b/clang/lib/Support/RISCVVIntrinsicUtils.cpp @@ -917,9 +917,9 @@ // Update PolicyAttrs if need (TA or TAMA) for compute builtin types. if (PolicyAttrs.isMAPolicy()) PolicyAttrs.TailPolicy = Policy::PolicyType::Agnostic; // TAMA - if (PolicyAttrs.isPolicyNonePolicy()) { + if (PolicyAttrs.isUnspecified()) { if (!IsMasked) { - PolicyAttrs.PolicyNone = false; + PolicyAttrs.IsUnspecified = false; PolicyAttrs.TailPolicy = Policy::PolicyType::Agnostic; // TA } } @@ -1022,8 +1022,8 @@ OverloadedName += suffix; }; - if (PolicyAttrs.isPolicyNonePolicy()) { - PolicyAttrs.PolicyNone = false; + if (PolicyAttrs.isUnspecified()) { + PolicyAttrs.IsUnspecified = false; if (IsMasked) { Name += "_m"; // FIXME: Currently _m default policy implementation is different with