diff options
author | Owen Anderson <resistor@mac.com> | 2007-06-21 01:59:05 +0000 |
---|---|---|
committer | Owen Anderson <resistor@mac.com> | 2007-06-21 01:59:05 +0000 |
commit | 27876a3ff9a6b991b81448287fa8894355331968 (patch) | |
tree | bc85d1770b624d27c1a54ab5b7256c0e1c5c2883 /llvm/lib/Transforms/Scalar/GVNPRE.cpp | |
parent | fd5683ad7a95d897f741000475c353d4342e4f4d (diff) | |
download | bcm5719-llvm-27876a3ff9a6b991b81448287fa8894355331968.tar.gz bcm5719-llvm-27876a3ff9a6b991b81448287fa8894355331968.zip |
Eliminate a redundant check. This speeds up optimization of 253.perlbmk from 13.5 seconds to 10.9 seconds.
llvm-svn: 37683
Diffstat (limited to 'llvm/lib/Transforms/Scalar/GVNPRE.cpp')
-rw-r--r-- | llvm/lib/Transforms/Scalar/GVNPRE.cpp | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/llvm/lib/Transforms/Scalar/GVNPRE.cpp b/llvm/lib/Transforms/Scalar/GVNPRE.cpp index ef70f679bcf..b1b096e3504 100644 --- a/llvm/lib/Transforms/Scalar/GVNPRE.cpp +++ b/llvm/lib/Transforms/Scalar/GVNPRE.cpp @@ -887,13 +887,9 @@ bool GVNPRE::buildsets_anticin(BasicBlock* BB, // However, for opaques, such as constants within PHI nodes, it is // possible that they have not yet received a number. Make sure they do // so now. - uint32_t valNum = 0; - if (isa<BinaryOperator>(*I) || isa<CmpInst>(*I)) - valNum = VN.lookup(*I); - else - valNum = VN.lookup_or_add(*I); - if (find_leader(anticIn, valNum) == 0) - val_insert(anticIn, *I); + if (!isa<BinaryOperator>(*I) && !isa<CmpInst>(*I)) + VN.lookup_or_add(*I); + val_insert(anticIn, *I); } clean(anticIn); |