aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/IR/Verifier.cpp
diff options
context:
space:
mode:
authorFlorian Hahn <flo@fhahn.com>2025-06-25 09:31:02 +0100
committerGitHub <noreply@github.com>2025-06-25 09:31:02 +0100
commit17c5c19902a8b9270687186098e66565afbef64b (patch)
treeb30a9301a51501b1260b47994a9c13de76596930 /llvm/lib/IR/Verifier.cpp
parent4ac4726d00644f6c6b0e2de1df0d00deed0015bf (diff)
downloadllvm-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.cpp4
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)