diff options
author | Florian Hahn <flo@fhahn.com> | 2025-06-25 09:31:02 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-06-25 09:31:02 +0100 |
commit | 17c5c19902a8b9270687186098e66565afbef64b (patch) | |
tree | b30a9301a51501b1260b47994a9c13de76596930 /llvm/lib/IR/Verifier.cpp | |
parent | 4ac4726d00644f6c6b0e2de1df0d00deed0015bf (diff) | |
download | llvm-17c5c19902a8b9270687186098e66565afbef64b.zip llvm-17c5c19902a8b9270687186098e66565afbef64b.tar.gz llvm-17c5c19902a8b9270687186098e66565afbef64b.tar.bz2 |
[Verifier] Always verify all assume bundles. (#145586)
For some reason, some of the checks for specific assumbe bundle elements
exit early if the check pass, meaning we don't verify other entries.
Replace the early returns with early continues.
This also requires removing some tests that are currently rejected. They will
be added back as part of https://github.com/llvm/llvm-project/pull/128436.
PR: https://github.com/llvm/llvm-project/pull/145586
Diffstat (limited to 'llvm/lib/IR/Verifier.cpp')
-rw-r--r-- | llvm/lib/IR/Verifier.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp index 7126134..70beb03 100644 --- a/llvm/lib/IR/Verifier.cpp +++ b/llvm/lib/IR/Verifier.cpp @@ -5517,7 +5517,7 @@ void Verifier::visitIntrinsicCall(Intrinsic::ID ID, CallBase &Call) { Call.getOperand(Elem.Begin + 1)->getType()->isPointerTy(), "arguments to separate_storage assumptions should be pointers", Call); - return; + continue; } Check(Elem.Tag->getKey() == "ignore" || Attribute::isExistingAttribute(Elem.Tag->getKey()), @@ -5534,7 +5534,7 @@ void Verifier::visitIntrinsicCall(Intrinsic::ID ID, CallBase &Call) { if (ArgCount == 3) Check(Call.getOperand(Elem.Begin + 2)->getType()->isIntegerTy(), "third argument should be an integer if present", Call); - return; + continue; } Check(ArgCount <= 2, "too many arguments", Call); if (Kind == Attribute::None) |