diff options
author | Bill Wendling <isanbard@gmail.com> | 2012-01-31 06:57:53 +0000 |
---|---|---|
committer | Bill Wendling <isanbard@gmail.com> | 2012-01-31 06:57:53 +0000 |
commit | 5d8ab0f02b3af1a19b46ccc6aa4782451e9dab8b (patch) | |
tree | 76134a0b76b3de60cc539d5ae23b43419ec1d7f1 | |
parent | cac50c5ab8bc2e8a55eb91a4b36d46e1c20c9c64 (diff) | |
download | external_llvm-5d8ab0f02b3af1a19b46ccc6aa4782451e9dab8b.zip external_llvm-5d8ab0f02b3af1a19b46ccc6aa4782451e9dab8b.tar.gz external_llvm-5d8ab0f02b3af1a19b46ccc6aa4782451e9dab8b.tar.bz2 |
Cache the size of the vector instead of calling .size() all over the place.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@149368 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/Scalar/GVN.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index e2617ed..6e928b7 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/lib/Transforms/Scalar/GVN.cpp @@ -1278,14 +1278,14 @@ bool GVN::processNonLocalLoad(LoadInst *LI) { // If we had to process more than one hundred blocks to find the // dependencies, this load isn't worth worrying about. Optimizing // it will be too expensive. - if (Deps.size() > 100) + unsigned NumDeps = Deps.size(); + if (NumDeps > 100) return false; // If we had a phi translation failure, we'll have a single entry which is a // clobber in the current block. Reject this early. - if (Deps.size() == 1 - && !Deps[0].getResult().isDef() && !Deps[0].getResult().isClobber()) - { + if (NumDeps == 1 && + !Deps[0].getResult().isDef() && !Deps[0].getResult().isClobber()) { DEBUG( dbgs() << "GVN: non-local load "; WriteAsOperand(dbgs(), LI); @@ -1301,7 +1301,7 @@ bool GVN::processNonLocalLoad(LoadInst *LI) { SmallVector<AvailableValueInBlock, 16> ValuesPerBlock; SmallVector<BasicBlock*, 16> UnavailableBlocks; - for (unsigned i = 0, e = Deps.size(); i != e; ++i) { + for (unsigned i = 0, e = NumDeps; i != e; ++i) { BasicBlock *DepBB = Deps[i].getBB(); MemDepResult DepInfo = Deps[i].getResult(); |