diff options
| author | Nicolai Haehnle <nhaehnle@gmail.com> | 2018-03-21 17:13:10 +0000 |
|---|---|---|
| committer | Nicolai Haehnle <nhaehnle@gmail.com> | 2018-03-21 17:13:10 +0000 |
| commit | 87aec1b194bd1ae1a3aff49cb3d49cb82edcd7bc (patch) | |
| tree | 7ffa7aa2314c9f75b9c0dac00943ad62fe756b4f | |
| parent | 420e28c78c9fe51345686c0ac48b64a06a1907c1 (diff) | |
| download | bcm5719-llvm-87aec1b194bd1ae1a3aff49cb3d49cb82edcd7bc.tar.gz bcm5719-llvm-87aec1b194bd1ae1a3aff49cb3d49cb82edcd7bc.zip | |
TableGen: Remove redundant loop in ListInit::resolveReferences
Summary:
Recursive lookups are handled by the Resolver, so the loop was purely
a waste of runtime.
Change-Id: I2bd23a68b478aea0bbac1a86ca7635adffa28688
Reviewers: arsenm, craig.topper, tra, MartinO
Subscribers: wdng, llvm-commits
Differential Revision: https://reviews.llvm.org/D44624
llvm-svn: 328118
| -rw-r--r-- | llvm/lib/TableGen/Record.cpp | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/llvm/lib/TableGen/Record.cpp b/llvm/lib/TableGen/Record.cpp index 81c75cd847d..30889b1e6e4 100644 --- a/llvm/lib/TableGen/Record.cpp +++ b/llvm/lib/TableGen/Record.cpp @@ -634,13 +634,8 @@ Init *ListInit::resolveReferences(Resolver &R) const { bool Changed = false; for (Init *CurElt : getValues()) { - Init *E; - - do { - E = CurElt; - CurElt = CurElt->resolveReferences(R); - Changed |= E != CurElt; - } while (E != CurElt); + Init *E = CurElt->resolveReferences(R); + Changed |= E != CurElt; Resolved.push_back(E); } |

