diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2021-07-12 21:25:46 +0200 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2021-07-12 21:57:26 +0200 |
commit | 6ac32872ee2fa02bdce54b25641d9a21cfbcfa9a (patch) | |
tree | a14f9547dcb3a79574dc37cbbe2977737e9ddb02 /llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp | |
parent | 279c3a5be3b4f3030584033823f19752325d08ce (diff) | |
download | llvm-6ac32872ee2fa02bdce54b25641d9a21cfbcfa9a.zip llvm-6ac32872ee2fa02bdce54b25641d9a21cfbcfa9a.tar.gz llvm-6ac32872ee2fa02bdce54b25641d9a21cfbcfa9a.tar.bz2 |
[Attributes] Replace doesAttrKindHaveArgument() (NFC)
This is now the same as isIntAttrKind(), so use that instead, as
it does not require manual maintenance. The naming is also more
accurate in that both int and type attributes have an argument,
but this method was only targeting int attributes.
I initially wanted to tighten the AttrBuilder assertion, but we
have some in-tree uses that would violate it.
Diffstat (limited to 'llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp')
-rw-r--r-- | llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp b/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp index f8c6603..d689e04 100644 --- a/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp +++ b/llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp @@ -152,7 +152,7 @@ struct AssumeBuilderState { } if (auto *Arg = dyn_cast<Argument>(RK.WasOn)) { if (Arg->hasAttribute(RK.AttrKind) && - (!Attribute::doesAttrKindHaveArgument(RK.AttrKind) || + (!Attribute::isIntAttrKind(RK.AttrKind) || Arg->getAttribute(RK.AttrKind).getValueAsInt() >= RK.ArgValue)) return false; return true; @@ -422,7 +422,7 @@ struct AssumeSimplify { if (auto *Arg = dyn_cast_or_null<Argument>(RK.WasOn)) { bool HasSameKindAttr = Arg->hasAttribute(RK.AttrKind); if (HasSameKindAttr) - if (!Attribute::doesAttrKindHaveArgument(RK.AttrKind) || + if (!Attribute::isIntAttrKind(RK.AttrKind) || Arg->getAttribute(RK.AttrKind).getValueAsInt() >= RK.ArgValue) { RemoveFromAssume(); |