diff options
| author | Davide Italiano <davide@freebsd.org> | 2017-02-18 03:02:44 +0000 |
|---|---|---|
| committer | Davide Italiano <davide@freebsd.org> | 2017-02-18 03:02:44 +0000 |
| commit | 982bf827b57a727ceeec9bd34c25ccb2b0133a9d (patch) | |
| tree | 896e1b4f76d1d2a23bf364a97c068c3038d837c7 /llvm/lib/IR/Verifier.cpp | |
| parent | bd4cd0c063ca432f4cca52c331fadd58abfb8b2e (diff) | |
| download | bcm5719-llvm-982bf827b57a727ceeec9bd34c25ccb2b0133a9d.tar.gz bcm5719-llvm-982bf827b57a727ceeec9bd34c25ccb2b0133a9d.zip | |
[IR/Verifier] Don't visit DISubprograms more than needed.
Before this patch we happened to visit twice, one when scanning
MDNodes and the other one while visiting the function. Remove
the explicit call to visitDISubprogram there, so we don't emit
the same error twice in case the verifier fail and we save some
time when running it.
Thanks to Justin Bogner for the report and Adrian for the quick
review!
PR: 31995
llvm-svn: 295537
Diffstat (limited to 'llvm/lib/IR/Verifier.cpp')
| -rw-r--r-- | llvm/lib/IR/Verifier.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp index 9c2da10c9e1..c14e2bbdee4 100644 --- a/llvm/lib/IR/Verifier.cpp +++ b/llvm/lib/IR/Verifier.cpp @@ -2119,8 +2119,6 @@ void Verifier::visitFunction(const Function &F) { if (!N) return; - visitDISubprogram(*N); - // Check that all !dbg attachments lead to back to N (or, at least, another // subprogram that describes the same function). // |
