diff options
| author | Zvi Rackover <zvi.rackover@intel.com> | 2017-04-18 14:55:43 +0000 |
|---|---|---|
| committer | Zvi Rackover <zvi.rackover@intel.com> | 2017-04-18 14:55:43 +0000 |
| commit | d942397e24f76850420d0f70e66bdbb79e49d300 (patch) | |
| tree | 7e754fdb6b061a00150d54a84032ee6a68d36d83 /llvm/lib/Transforms | |
| parent | fb1d915ab2dff98eac756cf882b872f9a4f34882 (diff) | |
| download | bcm5719-llvm-d942397e24f76850420d0f70e66bdbb79e49d300.tar.gz bcm5719-llvm-d942397e24f76850420d0f70e66bdbb79e49d300.zip | |
LoopRerollPass: Prefer Value::hasOneUse() over Value::getNumUses(). NFC.
getNumUses() can be more expensive as it iterates over all list's elements.
llvm-svn: 300558
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/LoopRerollPass.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/LoopRerollPass.cpp b/llvm/lib/Transforms/Scalar/LoopRerollPass.cpp index 86058fe0b1a..d54ad58fe1d 100644 --- a/llvm/lib/Transforms/Scalar/LoopRerollPass.cpp +++ b/llvm/lib/Transforms/Scalar/LoopRerollPass.cpp @@ -557,7 +557,7 @@ bool LoopReroll::isLoopControlIV(Loop *L, Instruction *IV) { Instruction *UUser = dyn_cast<Instruction>(UU); // Skip SExt if we are extending an nsw value // TODO: Allow ZExt too - if (BO->hasNoSignedWrap() && UUser && UUser->getNumUses() == 1 && + if (BO->hasNoSignedWrap() && UUser && UUser->hasOneUse() && isa<SExtInst>(UUser)) UUser = dyn_cast<Instruction>(*(UUser->user_begin())); if (!isCompareUsedByBranch(UUser)) |
