diff options
author | Philip Reames <listmail@philipreames.com> | 2018-03-23 23:41:47 +0000 |
---|---|---|
committer | Philip Reames <listmail@philipreames.com> | 2018-03-23 23:41:47 +0000 |
commit | 6a1f3446b5f2896c22a52dca732ba30c15535cd2 (patch) | |
tree | f2167c8ad3612352fcffe81a9b5d2b4aba3f2d0a /llvm/lib/Transforms/Scalar | |
parent | e27b410661f5e3a8ead2645eb27567a376668e0f (diff) | |
download | bcm5719-llvm-6a1f3446b5f2896c22a52dca732ba30c15535cd2.tar.gz bcm5719-llvm-6a1f3446b5f2896c22a52dca732ba30c15535cd2.zip |
[GuardWidening] Group code by class [NFC]
llvm-svn: 328387
Diffstat (limited to 'llvm/lib/Transforms/Scalar')
-rw-r--r-- | llvm/lib/Transforms/Scalar/GuardWidening.cpp | 79 |
1 files changed, 40 insertions, 39 deletions
diff --git a/llvm/lib/Transforms/Scalar/GuardWidening.cpp b/llvm/lib/Transforms/Scalar/GuardWidening.cpp index c4aeccb85ca..200df9ded40 100644 --- a/llvm/lib/Transforms/Scalar/GuardWidening.cpp +++ b/llvm/lib/Transforms/Scalar/GuardWidening.cpp @@ -212,32 +212,6 @@ public: /// The entry point for this pass. bool run(); }; - -struct GuardWideningLegacyPass : public FunctionPass { - static char ID; - GuardWideningPass Impl; - - GuardWideningLegacyPass() : FunctionPass(ID) { - initializeGuardWideningLegacyPassPass(*PassRegistry::getPassRegistry()); - } - - bool runOnFunction(Function &F) override { - if (skipFunction(F)) - return false; - return GuardWideningImpl( - getAnalysis<DominatorTreeWrapperPass>().getDomTree(), - getAnalysis<PostDominatorTreeWrapperPass>().getPostDomTree(), - getAnalysis<LoopInfoWrapperPass>().getLoopInfo()).run(); - } - - void getAnalysisUsage(AnalysisUsage &AU) const override { - AU.setPreservesCFG(); - AU.addRequired<DominatorTreeWrapperPass>(); - AU.addRequired<PostDominatorTreeWrapperPass>(); - AU.addRequired<LoopInfoWrapperPass>(); - } -}; - } bool GuardWideningImpl::run() { @@ -651,19 +625,6 @@ bool GuardWideningImpl::combineRangeChecks( return RangeChecksOut.size() != OldCount; } -PreservedAnalyses GuardWideningPass::run(Function &F, - FunctionAnalysisManager &AM) { - auto &DT = AM.getResult<DominatorTreeAnalysis>(F); - auto &LI = AM.getResult<LoopAnalysis>(F); - auto &PDT = AM.getResult<PostDominatorTreeAnalysis>(F); - if (!GuardWideningImpl(DT, PDT, LI).run()) - return PreservedAnalyses::all(); - - PreservedAnalyses PA; - PA.preserveSet<CFGAnalyses>(); - return PA; -} - #ifndef NDEBUG StringRef GuardWideningImpl::scoreTypeToString(WideningScore WS) { switch (WS) { @@ -681,6 +642,46 @@ StringRef GuardWideningImpl::scoreTypeToString(WideningScore WS) { } #endif +PreservedAnalyses GuardWideningPass::run(Function &F, + FunctionAnalysisManager &AM) { + auto &DT = AM.getResult<DominatorTreeAnalysis>(F); + auto &LI = AM.getResult<LoopAnalysis>(F); + auto &PDT = AM.getResult<PostDominatorTreeAnalysis>(F); + if (!GuardWideningImpl(DT, PDT, LI).run()) + return PreservedAnalyses::all(); + + PreservedAnalyses PA; + PA.preserveSet<CFGAnalyses>(); + return PA; +} + +namespace { +struct GuardWideningLegacyPass : public FunctionPass { + static char ID; + GuardWideningPass Impl; + + GuardWideningLegacyPass() : FunctionPass(ID) { + initializeGuardWideningLegacyPassPass(*PassRegistry::getPassRegistry()); + } + + bool runOnFunction(Function &F) override { + if (skipFunction(F)) + return false; + return GuardWideningImpl( + getAnalysis<DominatorTreeWrapperPass>().getDomTree(), + getAnalysis<PostDominatorTreeWrapperPass>().getPostDomTree(), + getAnalysis<LoopInfoWrapperPass>().getLoopInfo()).run(); + } + + void getAnalysisUsage(AnalysisUsage &AU) const override { + AU.setPreservesCFG(); + AU.addRequired<DominatorTreeWrapperPass>(); + AU.addRequired<PostDominatorTreeWrapperPass>(); + AU.addRequired<LoopInfoWrapperPass>(); + } +}; +} + char GuardWideningLegacyPass::ID = 0; INITIALIZE_PASS_BEGIN(GuardWideningLegacyPass, "guard-widening", "Widen guards", |