2002-05-11 00:44:58 +02:00
|
|
|
//===-- LICM.cpp - Loop Invariant Code Motion Pass ------------------------===//
|
2005-04-22 01:48:37 +02:00
|
|
|
//
|
2003-10-20 21:43:21 +02:00
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
2007-12-29 21:36:04 +01:00
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
2005-04-22 01:48:37 +02:00
|
|
|
//
|
2003-10-20 21:43:21 +02:00
|
|
|
//===----------------------------------------------------------------------===//
|
2002-05-11 00:44:58 +02:00
|
|
|
//
|
2003-12-09 18:18:00 +01:00
|
|
|
// This pass performs loop invariant code motion, attempting to remove as much
|
|
|
|
// code from the body of a loop as possible. It does this by either hoisting
|
|
|
|
// code into the preheader block, or by sinking code to the exit blocks if it is
|
|
|
|
// safe. This pass also promotes must-aliased memory locations in the loop to
|
2003-12-19 08:22:45 +01:00
|
|
|
// live in registers, thus hoisting and sinking "invariant" loads and stores.
|
2003-12-09 18:18:00 +01:00
|
|
|
//
|
|
|
|
// This pass uses alias analysis for two purposes:
|
2003-02-24 04:52:32 +01:00
|
|
|
//
|
2004-05-23 23:20:19 +02:00
|
|
|
// 1. Moving loop invariant loads and calls out of loops. If we can determine
|
|
|
|
// that a load or call inside of a loop never aliases anything stored to,
|
|
|
|
// we can hoist it or sink it like any other instruction.
|
2003-02-24 04:52:32 +01:00
|
|
|
// 2. Scalar Promotion of Memory - If there is a store instruction inside of
|
|
|
|
// the loop, we try to move the store to happen AFTER the loop instead of
|
|
|
|
// inside of the loop. This can only happen if a few conditions are true:
|
|
|
|
// A. The pointer stored through is loop invariant
|
|
|
|
// B. There are no stores or loads in the loop which _may_ alias the
|
|
|
|
// pointer. There are no calls in the loop which mod/ref the pointer.
|
|
|
|
// If these conditions are true, we can promote the loads and stores in the
|
|
|
|
// loop of the pointer to use a temporary alloca'd variable. We then use
|
2010-08-29 08:43:52 +02:00
|
|
|
// the SSAUpdater to construct the appropriate SSA form for the value.
|
2002-05-11 00:44:58 +02:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2016-07-13 00:37:48 +02:00
|
|
|
#include "llvm/Transforms/Scalar/LICM.h"
|
2012-12-03 17:50:05 +01:00
|
|
|
#include "llvm/ADT/Statistic.h"
|
2002-08-22 23:39:55 +02:00
|
|
|
#include "llvm/Analysis/AliasAnalysis.h"
|
2003-03-04 00:32:45 +01:00
|
|
|
#include "llvm/Analysis/AliasSetTracker.h"
|
[PM/AA] Rebuild LLVM's alias analysis infrastructure in a way compatible
with the new pass manager, and no longer relying on analysis groups.
This builds essentially a ground-up new AA infrastructure stack for
LLVM. The core ideas are the same that are used throughout the new pass
manager: type erased polymorphism and direct composition. The design is
as follows:
- FunctionAAResults is a type-erasing alias analysis results aggregation
interface to walk a single query across a range of results from
different alias analyses. Currently this is function-specific as we
always assume that aliasing queries are *within* a function.
- AAResultBase is a CRTP utility providing stub implementations of
various parts of the alias analysis result concept, notably in several
cases in terms of other more general parts of the interface. This can
be used to implement only a narrow part of the interface rather than
the entire interface. This isn't really ideal, this logic should be
hoisted into FunctionAAResults as currently it will cause
a significant amount of redundant work, but it faithfully models the
behavior of the prior infrastructure.
- All the alias analysis passes are ported to be wrapper passes for the
legacy PM and new-style analysis passes for the new PM with a shared
result object. In some cases (most notably CFL), this is an extremely
naive approach that we should revisit when we can specialize for the
new pass manager.
- BasicAA has been restructured to reflect that it is much more
fundamentally a function analysis because it uses dominator trees and
loop info that need to be constructed for each function.
All of the references to getting alias analysis results have been
updated to use the new aggregation interface. All the preservation and
other pass management code has been updated accordingly.
The way the FunctionAAResultsWrapperPass works is to detect the
available alias analyses when run, and add them to the results object.
This means that we should be able to continue to respect when various
passes are added to the pipeline, for example adding CFL or adding TBAA
passes should just cause their results to be available and to get folded
into this. The exception to this rule is BasicAA which really needs to
be a function pass due to using dominator trees and loop info. As
a consequence, the FunctionAAResultsWrapperPass directly depends on
BasicAA and always includes it in the aggregation.
This has significant implications for preserving analyses. Generally,
most passes shouldn't bother preserving FunctionAAResultsWrapperPass
because rebuilding the results just updates the set of known AA passes.
The exception to this rule are LoopPass instances which need to preserve
all the function analyses that the loop pass manager will end up
needing. This means preserving both BasicAAWrapperPass and the
aggregating FunctionAAResultsWrapperPass.
Now, when preserving an alias analysis, you do so by directly preserving
that analysis. This is only necessary for non-immutable-pass-provided
alias analyses though, and there are only three of interest: BasicAA,
GlobalsAA (formerly GlobalsModRef), and SCEVAA. Usually BasicAA is
preserved when needed because it (like DominatorTree and LoopInfo) is
marked as a CFG-only pass. I've expanded GlobalsAA into the preserved
set everywhere we previously were preserving all of AliasAnalysis, and
I've added SCEVAA in the intersection of that with where we preserve
SCEV itself.
One significant challenge to all of this is that the CGSCC passes were
actually using the alias analysis implementations by taking advantage of
a pretty amazing set of loop holes in the old pass manager's analysis
management code which allowed analysis groups to slide through in many
cases. Moving away from analysis groups makes this problem much more
obvious. To fix it, I've leveraged the flexibility the design of the new
PM components provides to just directly construct the relevant alias
analyses for the relevant functions in the IPO passes that need them.
This is a bit hacky, but should go away with the new pass manager, and
is already in many ways cleaner than the prior state.
Another significant challenge is that various facilities of the old
alias analysis infrastructure just don't fit any more. The most
significant of these is the alias analysis 'counter' pass. That pass
relied on the ability to snoop on AA queries at different points in the
analysis group chain. Instead, I'm planning to build printing
functionality directly into the aggregation layer. I've not included
that in this patch merely to keep it smaller.
Note that all of this needs a nearly complete rewrite of the AA
documentation. I'm planning to do that, but I'd like to make sure the
new design settles, and to flesh out a bit more of what it looks like in
the new pass manager first.
Differential Revision: http://reviews.llvm.org/D12080
llvm-svn: 247167
2015-09-09 19:55:00 +02:00
|
|
|
#include "llvm/Analysis/BasicAliasAnalysis.h"
|
2016-03-09 23:59:30 +01:00
|
|
|
#include "llvm/Analysis/CaptureTracking.h"
|
2010-09-01 01:00:16 +02:00
|
|
|
#include "llvm/Analysis/ConstantFolding.h"
|
[PM/AA] Rebuild LLVM's alias analysis infrastructure in a way compatible
with the new pass manager, and no longer relying on analysis groups.
This builds essentially a ground-up new AA infrastructure stack for
LLVM. The core ideas are the same that are used throughout the new pass
manager: type erased polymorphism and direct composition. The design is
as follows:
- FunctionAAResults is a type-erasing alias analysis results aggregation
interface to walk a single query across a range of results from
different alias analyses. Currently this is function-specific as we
always assume that aliasing queries are *within* a function.
- AAResultBase is a CRTP utility providing stub implementations of
various parts of the alias analysis result concept, notably in several
cases in terms of other more general parts of the interface. This can
be used to implement only a narrow part of the interface rather than
the entire interface. This isn't really ideal, this logic should be
hoisted into FunctionAAResults as currently it will cause
a significant amount of redundant work, but it faithfully models the
behavior of the prior infrastructure.
- All the alias analysis passes are ported to be wrapper passes for the
legacy PM and new-style analysis passes for the new PM with a shared
result object. In some cases (most notably CFL), this is an extremely
naive approach that we should revisit when we can specialize for the
new pass manager.
- BasicAA has been restructured to reflect that it is much more
fundamentally a function analysis because it uses dominator trees and
loop info that need to be constructed for each function.
All of the references to getting alias analysis results have been
updated to use the new aggregation interface. All the preservation and
other pass management code has been updated accordingly.
The way the FunctionAAResultsWrapperPass works is to detect the
available alias analyses when run, and add them to the results object.
This means that we should be able to continue to respect when various
passes are added to the pipeline, for example adding CFL or adding TBAA
passes should just cause their results to be available and to get folded
into this. The exception to this rule is BasicAA which really needs to
be a function pass due to using dominator trees and loop info. As
a consequence, the FunctionAAResultsWrapperPass directly depends on
BasicAA and always includes it in the aggregation.
This has significant implications for preserving analyses. Generally,
most passes shouldn't bother preserving FunctionAAResultsWrapperPass
because rebuilding the results just updates the set of known AA passes.
The exception to this rule are LoopPass instances which need to preserve
all the function analyses that the loop pass manager will end up
needing. This means preserving both BasicAAWrapperPass and the
aggregating FunctionAAResultsWrapperPass.
Now, when preserving an alias analysis, you do so by directly preserving
that analysis. This is only necessary for non-immutable-pass-provided
alias analyses though, and there are only three of interest: BasicAA,
GlobalsAA (formerly GlobalsModRef), and SCEVAA. Usually BasicAA is
preserved when needed because it (like DominatorTree and LoopInfo) is
marked as a CFG-only pass. I've expanded GlobalsAA into the preserved
set everywhere we previously were preserving all of AliasAnalysis, and
I've added SCEVAA in the intersection of that with where we preserve
SCEV itself.
One significant challenge to all of this is that the CGSCC passes were
actually using the alias analysis implementations by taking advantage of
a pretty amazing set of loop holes in the old pass manager's analysis
management code which allowed analysis groups to slide through in many
cases. Moving away from analysis groups makes this problem much more
obvious. To fix it, I've leveraged the flexibility the design of the new
PM components provides to just directly construct the relevant alias
analyses for the relevant functions in the IPO passes that need them.
This is a bit hacky, but should go away with the new pass manager, and
is already in many ways cleaner than the prior state.
Another significant challenge is that various facilities of the old
alias analysis infrastructure just don't fit any more. The most
significant of these is the alias analysis 'counter' pass. That pass
relied on the ability to snoop on AA queries at different points in the
analysis group chain. Instead, I'm planning to build printing
functionality directly into the aggregation layer. I've not included
that in this patch merely to keep it smaller.
Note that all of this needs a nearly complete rewrite of the AA
documentation. I'm planning to do that, but I'd like to make sure the
new design settles, and to flesh out a bit more of what it looks like in
the new pass manager first.
Differential Revision: http://reviews.llvm.org/D12080
llvm-svn: 247167
2015-09-09 19:55:00 +02:00
|
|
|
#include "llvm/Analysis/GlobalsModRef.h"
|
2016-03-10 00:07:53 +01:00
|
|
|
#include "llvm/Analysis/Loads.h"
|
2010-09-01 01:00:16 +02:00
|
|
|
#include "llvm/Analysis/LoopInfo.h"
|
|
|
|
#include "llvm/Analysis/LoopPass.h"
|
2016-03-09 23:59:30 +01:00
|
|
|
#include "llvm/Analysis/MemoryBuiltins.h"
|
2017-11-21 16:45:46 +01:00
|
|
|
#include "llvm/Analysis/MemorySSA.h"
|
2017-10-10 01:19:02 +02:00
|
|
|
#include "llvm/Analysis/OptimizationRemarkEmitter.h"
|
2014-01-24 02:59:49 +01:00
|
|
|
#include "llvm/Analysis/ScalarEvolution.h"
|
[PM/AA] Rebuild LLVM's alias analysis infrastructure in a way compatible
with the new pass manager, and no longer relying on analysis groups.
This builds essentially a ground-up new AA infrastructure stack for
LLVM. The core ideas are the same that are used throughout the new pass
manager: type erased polymorphism and direct composition. The design is
as follows:
- FunctionAAResults is a type-erasing alias analysis results aggregation
interface to walk a single query across a range of results from
different alias analyses. Currently this is function-specific as we
always assume that aliasing queries are *within* a function.
- AAResultBase is a CRTP utility providing stub implementations of
various parts of the alias analysis result concept, notably in several
cases in terms of other more general parts of the interface. This can
be used to implement only a narrow part of the interface rather than
the entire interface. This isn't really ideal, this logic should be
hoisted into FunctionAAResults as currently it will cause
a significant amount of redundant work, but it faithfully models the
behavior of the prior infrastructure.
- All the alias analysis passes are ported to be wrapper passes for the
legacy PM and new-style analysis passes for the new PM with a shared
result object. In some cases (most notably CFL), this is an extremely
naive approach that we should revisit when we can specialize for the
new pass manager.
- BasicAA has been restructured to reflect that it is much more
fundamentally a function analysis because it uses dominator trees and
loop info that need to be constructed for each function.
All of the references to getting alias analysis results have been
updated to use the new aggregation interface. All the preservation and
other pass management code has been updated accordingly.
The way the FunctionAAResultsWrapperPass works is to detect the
available alias analyses when run, and add them to the results object.
This means that we should be able to continue to respect when various
passes are added to the pipeline, for example adding CFL or adding TBAA
passes should just cause their results to be available and to get folded
into this. The exception to this rule is BasicAA which really needs to
be a function pass due to using dominator trees and loop info. As
a consequence, the FunctionAAResultsWrapperPass directly depends on
BasicAA and always includes it in the aggregation.
This has significant implications for preserving analyses. Generally,
most passes shouldn't bother preserving FunctionAAResultsWrapperPass
because rebuilding the results just updates the set of known AA passes.
The exception to this rule are LoopPass instances which need to preserve
all the function analyses that the loop pass manager will end up
needing. This means preserving both BasicAAWrapperPass and the
aggregating FunctionAAResultsWrapperPass.
Now, when preserving an alias analysis, you do so by directly preserving
that analysis. This is only necessary for non-immutable-pass-provided
alias analyses though, and there are only three of interest: BasicAA,
GlobalsAA (formerly GlobalsModRef), and SCEVAA. Usually BasicAA is
preserved when needed because it (like DominatorTree and LoopInfo) is
marked as a CFG-only pass. I've expanded GlobalsAA into the preserved
set everywhere we previously were preserving all of AliasAnalysis, and
I've added SCEVAA in the intersection of that with where we preserve
SCEV itself.
One significant challenge to all of this is that the CGSCC passes were
actually using the alias analysis implementations by taking advantage of
a pretty amazing set of loop holes in the old pass manager's analysis
management code which allowed analysis groups to slide through in many
cases. Moving away from analysis groups makes this problem much more
obvious. To fix it, I've leveraged the flexibility the design of the new
PM components provides to just directly construct the relevant alias
analyses for the relevant functions in the IPO passes that need them.
This is a bit hacky, but should go away with the new pass manager, and
is already in many ways cleaner than the prior state.
Another significant challenge is that various facilities of the old
alias analysis infrastructure just don't fit any more. The most
significant of these is the alias analysis 'counter' pass. That pass
relied on the ability to snoop on AA queries at different points in the
analysis group chain. Instead, I'm planning to build printing
functionality directly into the aggregation layer. I've not included
that in this patch merely to keep it smaller.
Note that all of this needs a nearly complete rewrite of the AA
documentation. I'm planning to do that, but I'd like to make sure the
new design settles, and to flesh out a bit more of what it looks like in
the new pass manager first.
Differential Revision: http://reviews.llvm.org/D12080
llvm-svn: 247167
2015-09-09 19:55:00 +02:00
|
|
|
#include "llvm/Analysis/ScalarEvolutionAliasAnalysis.h"
|
2015-03-23 20:32:43 +01:00
|
|
|
#include "llvm/Analysis/TargetLibraryInfo.h"
|
2018-03-21 23:34:23 +01:00
|
|
|
#include "llvm/Analysis/Utils/Local.h"
|
2011-12-15 00:49:11 +01:00
|
|
|
#include "llvm/Analysis/ValueTracking.h"
|
2014-03-04 12:45:46 +01:00
|
|
|
#include "llvm/IR/CFG.h"
|
2013-01-02 12:36:10 +01:00
|
|
|
#include "llvm/IR/Constants.h"
|
|
|
|
#include "llvm/IR/DataLayout.h"
|
|
|
|
#include "llvm/IR/DerivedTypes.h"
|
2014-01-13 10:26:24 +01:00
|
|
|
#include "llvm/IR/Dominators.h"
|
2013-01-02 12:36:10 +01:00
|
|
|
#include "llvm/IR/Instructions.h"
|
|
|
|
#include "llvm/IR/IntrinsicInst.h"
|
|
|
|
#include "llvm/IR/LLVMContext.h"
|
2013-01-05 17:44:07 +01:00
|
|
|
#include "llvm/IR/Metadata.h"
|
2014-03-04 13:09:19 +01:00
|
|
|
#include "llvm/IR/PredIteratorCache.h"
|
2004-09-02 00:55:40 +02:00
|
|
|
#include "llvm/Support/CommandLine.h"
|
|
|
|
#include "llvm/Support/Debug.h"
|
2012-12-03 17:50:05 +01:00
|
|
|
#include "llvm/Support/raw_ostream.h"
|
2016-05-05 02:54:54 +02:00
|
|
|
#include "llvm/Transforms/Scalar.h"
|
2017-01-11 10:43:56 +01:00
|
|
|
#include "llvm/Transforms/Scalar/LoopPassManager.h"
|
2017-11-03 17:24:53 +01:00
|
|
|
#include "llvm/Transforms/Utils/BasicBlockUtils.h"
|
2014-01-25 05:07:24 +01:00
|
|
|
#include "llvm/Transforms/Utils/LoopUtils.h"
|
2012-12-03 17:50:05 +01:00
|
|
|
#include "llvm/Transforms/Utils/SSAUpdater.h"
|
2002-05-11 00:44:58 +02:00
|
|
|
#include <algorithm>
|
2016-05-27 16:27:24 +02:00
|
|
|
#include <utility>
|
2003-12-09 18:18:00 +01:00
|
|
|
using namespace llvm;
|
2003-11-11 23:41:34 +01:00
|
|
|
|
2014-04-22 04:55:47 +02:00
|
|
|
#define DEBUG_TYPE "licm"
|
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
STATISTIC(NumSunk, "Number of instructions sunk out of loop");
|
|
|
|
STATISTIC(NumHoisted, "Number of instructions hoisted out of loop");
|
2006-12-19 22:40:18 +01:00
|
|
|
STATISTIC(NumMovedLoads, "Number of load insts hoisted or sunk");
|
|
|
|
STATISTIC(NumMovedCalls, "Number of call insts hoisted or sunk");
|
2016-05-05 02:54:54 +02:00
|
|
|
STATISTIC(NumPromoted, "Number of memory locations promoted to registers");
|
2006-12-19 22:40:18 +01:00
|
|
|
|
2017-02-21 21:53:48 +01:00
|
|
|
/// Memory promotion is enabled by default.
|
2008-05-13 02:00:25 +02:00
|
|
|
static cl::opt<bool>
|
2017-02-21 21:53:48 +01:00
|
|
|
DisablePromotion("disable-licm-promotion", cl::Hidden, cl::init(false),
|
2016-05-05 02:54:54 +02:00
|
|
|
cl::desc("Disable memory promotion in LICM pass"));
|
2003-02-24 04:52:32 +01:00
|
|
|
|
2017-02-02 14:22:03 +01:00
|
|
|
static cl::opt<uint32_t> MaxNumUsesTraversed(
|
|
|
|
"licm-max-num-uses-traversed", cl::Hidden, cl::init(8),
|
|
|
|
cl::desc("Max num uses visited for identifying load "
|
|
|
|
"invariance in loop using invariant start (default = 8)"));
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
static bool inSubLoop(BasicBlock *BB, Loop *CurLoop, LoopInfo *LI);
|
2017-12-15 21:33:24 +01:00
|
|
|
static bool isNotUsedOrFreeInLoop(const Instruction &I, const Loop *CurLoop,
|
|
|
|
const LoopSafetyInfo *SafetyInfo,
|
|
|
|
TargetTransformInfo *TTI, bool &FreeInLoop);
|
2016-01-28 16:51:58 +01:00
|
|
|
static bool hoist(Instruction &I, const DominatorTree *DT, const Loop *CurLoop,
|
2017-01-11 05:39:35 +01:00
|
|
|
const LoopSafetyInfo *SafetyInfo,
|
|
|
|
OptimizationRemarkEmitter *ORE);
|
2017-11-03 17:24:53 +01:00
|
|
|
static bool sink(Instruction &I, LoopInfo *LI, DominatorTree *DT,
|
2018-02-12 18:56:55 +01:00
|
|
|
const Loop *CurLoop, LoopSafetyInfo *SafetyInfo,
|
2017-12-15 21:33:24 +01:00
|
|
|
OptimizationRemarkEmitter *ORE, bool FreeInLoop);
|
2017-01-11 05:39:49 +01:00
|
|
|
static bool isSafeToExecuteUnconditionally(Instruction &Inst,
|
2015-05-13 03:12:18 +02:00
|
|
|
const DominatorTree *DT,
|
|
|
|
const Loop *CurLoop,
|
2016-06-10 22:03:17 +02:00
|
|
|
const LoopSafetyInfo *SafetyInfo,
|
2017-01-11 05:39:49 +01:00
|
|
|
OptimizationRemarkEmitter *ORE,
|
2015-05-22 04:14:05 +02:00
|
|
|
const Instruction *CtxI = nullptr);
|
2015-02-22 19:35:32 +01:00
|
|
|
static bool pointerInvalidatedByLoop(Value *V, uint64_t Size,
|
2016-05-05 02:54:54 +02:00
|
|
|
const AAMDNodes &AAInfo,
|
2015-02-22 19:35:32 +01:00
|
|
|
AliasSetTracker *CurAST);
|
2016-01-04 04:37:39 +01:00
|
|
|
static Instruction *
|
|
|
|
CloneInstructionInExitBlock(Instruction &I, BasicBlock &ExitBlock, PHINode &PN,
|
|
|
|
const LoopInfo *LI,
|
2016-06-10 22:03:17 +02:00
|
|
|
const LoopSafetyInfo *SafetyInfo);
|
2015-02-22 19:35:32 +01:00
|
|
|
|
2008-05-13 02:00:25 +02:00
|
|
|
namespace {
|
2016-07-13 00:37:48 +02:00
|
|
|
struct LoopInvariantCodeMotion {
|
|
|
|
bool runOnLoop(Loop *L, AliasAnalysis *AA, LoopInfo *LI, DominatorTree *DT,
|
2017-12-15 21:33:24 +01:00
|
|
|
TargetLibraryInfo *TLI, TargetTransformInfo *TTI,
|
|
|
|
ScalarEvolution *SE, MemorySSA *MSSA,
|
2017-01-11 05:39:35 +01:00
|
|
|
OptimizationRemarkEmitter *ORE, bool DeleteAST);
|
2016-07-13 00:37:48 +02:00
|
|
|
|
|
|
|
DenseMap<Loop *, AliasSetTracker *> &getLoopToAliasSetMap() {
|
|
|
|
return LoopToAliasSetMap;
|
|
|
|
}
|
|
|
|
|
|
|
|
private:
|
|
|
|
DenseMap<Loop *, AliasSetTracker *> LoopToAliasSetMap;
|
|
|
|
|
|
|
|
AliasSetTracker *collectAliasInfoForLoop(Loop *L, LoopInfo *LI,
|
|
|
|
AliasAnalysis *AA);
|
|
|
|
};
|
|
|
|
|
|
|
|
struct LegacyLICMPass : public LoopPass {
|
2016-05-05 02:54:54 +02:00
|
|
|
static char ID; // Pass identification, replacement for typeid
|
2016-07-13 00:37:48 +02:00
|
|
|
LegacyLICMPass() : LoopPass(ID) {
|
|
|
|
initializeLegacyLICMPassPass(*PassRegistry::getPassRegistry());
|
2016-05-05 02:54:54 +02:00
|
|
|
}
|
2007-05-01 23:15:47 +02:00
|
|
|
|
2016-07-13 00:37:48 +02:00
|
|
|
bool runOnLoop(Loop *L, LPPassManager &LPM) override {
|
2016-12-23 14:12:50 +01:00
|
|
|
if (skipLoop(L)) {
|
|
|
|
// If we have run LICM on a previous loop but now we are skipping
|
|
|
|
// (because we've hit the opt-bisect limit), we need to clear the
|
|
|
|
// loop alias information.
|
2016-12-23 16:02:35 +01:00
|
|
|
for (auto <AS : LICM.getLoopToAliasSetMap())
|
|
|
|
delete LTAS.second;
|
2016-12-23 14:12:50 +01:00
|
|
|
LICM.getLoopToAliasSetMap().clear();
|
2016-07-13 00:37:48 +02:00
|
|
|
return false;
|
2016-12-23 14:12:50 +01:00
|
|
|
}
|
2016-07-13 00:37:48 +02:00
|
|
|
|
|
|
|
auto *SE = getAnalysisIfAvailable<ScalarEvolutionWrapperPass>();
|
2017-11-21 16:45:46 +01:00
|
|
|
MemorySSA *MSSA = EnableMSSALoopDependency
|
|
|
|
? (&getAnalysis<MemorySSAWrapperPass>().getMSSA())
|
|
|
|
: nullptr;
|
2017-01-11 05:39:35 +01:00
|
|
|
// For the old PM, we can't use OptimizationRemarkEmitter as an analysis
|
|
|
|
// pass. Function analyses need to be preserved across loop transformations
|
|
|
|
// but ORE cannot be preserved (see comment before the pass definition).
|
|
|
|
OptimizationRemarkEmitter ORE(L->getHeader()->getParent());
|
2016-07-13 00:37:48 +02:00
|
|
|
return LICM.runOnLoop(L,
|
|
|
|
&getAnalysis<AAResultsWrapperPass>().getAAResults(),
|
|
|
|
&getAnalysis<LoopInfoWrapperPass>().getLoopInfo(),
|
|
|
|
&getAnalysis<DominatorTreeWrapperPass>().getDomTree(),
|
|
|
|
&getAnalysis<TargetLibraryInfoWrapperPass>().getTLI(),
|
2017-12-15 21:33:24 +01:00
|
|
|
&getAnalysis<TargetTransformInfoWrapperPass>().getTTI(
|
|
|
|
*L->getHeader()->getParent()),
|
2017-11-21 16:45:46 +01:00
|
|
|
SE ? &SE->getSE() : nullptr, MSSA, &ORE, false);
|
2016-07-13 00:37:48 +02:00
|
|
|
}
|
2002-05-11 00:44:58 +02:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
/// This transformation requires natural loop information & requires that
|
|
|
|
/// loop preheaders be inserted into the CFG...
|
|
|
|
///
|
|
|
|
void getAnalysisUsage(AnalysisUsage &AU) const override {
|
|
|
|
AU.setPreservesCFG();
|
|
|
|
AU.addRequired<TargetLibraryInfoWrapperPass>();
|
2017-11-21 16:45:46 +01:00
|
|
|
if (EnableMSSALoopDependency)
|
|
|
|
AU.addRequired<MemorySSAWrapperPass>();
|
2017-12-15 21:33:24 +01:00
|
|
|
AU.addRequired<TargetTransformInfoWrapperPass>();
|
2016-05-05 02:54:54 +02:00
|
|
|
getLoopAnalysisUsage(AU);
|
|
|
|
}
|
2002-05-11 00:44:58 +02:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
using llvm::Pass::doFinalization;
|
2012-12-04 06:41:27 +01:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
bool doFinalization() override {
|
2016-07-13 00:37:48 +02:00
|
|
|
assert(LICM.getLoopToAliasSetMap().empty() &&
|
|
|
|
"Didn't free loop alias sets");
|
2016-05-05 02:54:54 +02:00
|
|
|
return false;
|
|
|
|
}
|
2007-03-07 05:41:30 +01:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
private:
|
2016-07-13 00:37:48 +02:00
|
|
|
LoopInvariantCodeMotion LICM;
|
2002-05-11 00:44:58 +02:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
/// cloneBasicBlockAnalysis - Simple Analysis hook. Clone alias set info.
|
|
|
|
void cloneBasicBlockAnalysis(BasicBlock *From, BasicBlock *To,
|
|
|
|
Loop *L) override;
|
2007-07-31 10:01:41 +02:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
/// deleteAnalysisValue - Simple Analysis hook. Delete value V from alias
|
|
|
|
/// set.
|
|
|
|
void deleteAnalysisValue(Value *V, Loop *L) override;
|
2007-07-31 10:01:41 +02:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
/// Simple Analysis hook. Delete loop L from alias set map.
|
|
|
|
void deleteAnalysisLoop(Loop *L) override;
|
2016-07-12 08:25:32 +02:00
|
|
|
};
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
} // namespace
|
2016-07-12 00:45:24 +02:00
|
|
|
|
2017-01-11 07:23:21 +01:00
|
|
|
PreservedAnalyses LICMPass::run(Loop &L, LoopAnalysisManager &AM,
|
|
|
|
LoopStandardAnalysisResults &AR, LPMUpdater &) {
|
2016-07-13 00:37:48 +02:00
|
|
|
const auto &FAM =
|
2017-01-11 07:23:21 +01:00
|
|
|
AM.getResult<FunctionAnalysisManagerLoopProxy>(L, AR).getManager();
|
2016-07-13 00:37:48 +02:00
|
|
|
Function *F = L.getHeader()->getParent();
|
|
|
|
|
2017-01-11 05:39:35 +01:00
|
|
|
auto *ORE = FAM.getCachedResult<OptimizationRemarkEmitterAnalysis>(*F);
|
2017-01-11 07:23:21 +01:00
|
|
|
// FIXME: This should probably be optional rather than required.
|
|
|
|
if (!ORE)
|
|
|
|
report_fatal_error("LICM: OptimizationRemarkEmitterAnalysis not "
|
|
|
|
"cached at a higher level");
|
2016-07-13 00:37:48 +02:00
|
|
|
|
|
|
|
LoopInvariantCodeMotion LICM;
|
2017-12-15 21:33:24 +01:00
|
|
|
if (!LICM.runOnLoop(&L, &AR.AA, &AR.LI, &AR.DT, &AR.TLI, &AR.TTI, &AR.SE,
|
|
|
|
AR.MSSA, ORE, true))
|
2016-07-13 00:37:48 +02:00
|
|
|
return PreservedAnalyses::all();
|
|
|
|
|
2017-01-15 07:32:49 +01:00
|
|
|
auto PA = getLoopPassPreservedAnalyses();
|
|
|
|
PA.preserveSet<CFGAnalyses>();
|
|
|
|
return PA;
|
2016-07-13 00:37:48 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
char LegacyLICMPass::ID = 0;
|
|
|
|
INITIALIZE_PASS_BEGIN(LegacyLICMPass, "licm", "Loop Invariant Code Motion",
|
|
|
|
false, false)
|
[LPM] Factor all of the loop analysis usage updates into a common helper
routine.
We were getting this wrong in small ways and generally being very
inconsistent about it across loop passes. Instead, let's have a common
place where we do this. One minor downside is that this will require
some analyses like SCEV in more places than they are strictly needed.
However, this seems benign as these analyses are complete no-ops, and
without this consistency we can in many cases end up with the legacy
pass manager scheduling deciding to split up a loop pass pipeline in
order to run the function analysis half-way through. It is very, very
annoying to fix these without just being very pedantic across the board.
The only loop passes I've not updated here are ones that use
AU.setPreservesAll() such as IVUsers (an analysis) and the pass printer.
They seemed less relevant.
With this patch, almost all of the problems in PR24804 around loop pass
pipelines are fixed. The one remaining issue is that we run simplify-cfg
and instcombine in the middle of the loop pass pipeline. We've recently
added some loop variants of these passes that would seem substantially
cleaner to use, but this at least gets us much closer to the previous
state. Notably, the seven loop pass managers is down to three.
I've not updated the loop passes using LoopAccessAnalysis because that
analysis hasn't been fully wired into LoopSimplify/LCSSA, and it isn't
clear that those transforms want to support those forms anyways. They
all run late anyways, so this is harmless. Similarly, LSR is left alone
because it already carefully manages its forms and doesn't need to get
fused into a single loop pass manager with a bunch of other loop passes.
LoopReroll didn't use loop simplified form previously, and I've updated
the test case to match the trivially different output.
Finally, I've also factored all the pass initialization for the passes
that use this technique as well, so that should be done regularly and
reliably.
Thanks to James for the help reviewing and thinking about this stuff,
and Ben for help thinking about it as well!
Differential Revision: http://reviews.llvm.org/D17435
llvm-svn: 261316
2016-02-19 11:45:18 +01:00
|
|
|
INITIALIZE_PASS_DEPENDENCY(LoopPass)
|
2015-01-15 11:41:28 +01:00
|
|
|
INITIALIZE_PASS_DEPENDENCY(TargetLibraryInfoWrapperPass)
|
2017-12-15 21:33:24 +01:00
|
|
|
INITIALIZE_PASS_DEPENDENCY(TargetTransformInfoWrapperPass)
|
2017-11-21 16:45:46 +01:00
|
|
|
INITIALIZE_PASS_DEPENDENCY(MemorySSAWrapperPass)
|
2016-07-13 00:37:48 +02:00
|
|
|
INITIALIZE_PASS_END(LegacyLICMPass, "licm", "Loop Invariant Code Motion", false,
|
|
|
|
false)
|
2008-05-13 02:00:25 +02:00
|
|
|
|
2016-07-13 00:37:48 +02:00
|
|
|
Pass *llvm::createLICMPass() { return new LegacyLICMPass(); }
|
2002-05-11 00:44:58 +02:00
|
|
|
|
2007-07-31 18:52:25 +02:00
|
|
|
/// Hoist expressions out of the specified loop. Note, alias info for inner
|
2011-07-06 21:20:02 +02:00
|
|
|
/// loop is not preserved so it is not a good idea to run LICM multiple
|
2007-07-31 18:52:25 +02:00
|
|
|
/// times on one loop.
|
2016-07-13 00:37:48 +02:00
|
|
|
/// We should delete AST for inner loops in the new pass manager to avoid
|
|
|
|
/// memory leak.
|
2002-09-26 18:52:07 +02:00
|
|
|
///
|
2017-12-15 21:33:24 +01:00
|
|
|
bool LoopInvariantCodeMotion::runOnLoop(
|
|
|
|
Loop *L, AliasAnalysis *AA, LoopInfo *LI, DominatorTree *DT,
|
|
|
|
TargetLibraryInfo *TLI, TargetTransformInfo *TTI, ScalarEvolution *SE,
|
|
|
|
MemorySSA *MSSA, OptimizationRemarkEmitter *ORE, bool DeleteAST) {
|
2016-07-13 00:37:48 +02:00
|
|
|
bool Changed = false;
|
2011-12-02 02:26:24 +01:00
|
|
|
|
2014-02-11 13:52:27 +01:00
|
|
|
assert(L->isLCSSAForm(*DT) && "Loop is not in LCSSA form.");
|
|
|
|
|
2016-07-13 00:37:48 +02:00
|
|
|
AliasSetTracker *CurAST = collectAliasInfoForLoop(L, LI, AA);
|
2002-05-11 00:44:58 +02:00
|
|
|
|
2002-09-26 21:40:25 +02:00
|
|
|
// Get the preheader block to move instructions into...
|
2016-07-13 00:37:48 +02:00
|
|
|
BasicBlock *Preheader = L->getLoopPreheader();
|
2002-09-26 21:40:25 +02:00
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
// Compute loop safety information.
|
2016-06-10 22:03:17 +02:00
|
|
|
LoopSafetyInfo SafetyInfo;
|
2016-07-13 00:37:48 +02:00
|
|
|
computeLoopSafetyInfo(&SafetyInfo, L);
|
2012-09-04 12:25:04 +02:00
|
|
|
|
2002-05-11 00:44:58 +02:00
|
|
|
// We want to visit all of the instructions in this loop... that are not parts
|
|
|
|
// of our subloops (they have already had their invariants hoisted out of
|
|
|
|
// their loop, into this loop, so there is no need to process the BODIES of
|
|
|
|
// the subloops).
|
|
|
|
//
|
2002-09-29 23:46:09 +02:00
|
|
|
// Traverse the body of the loop in depth first order on the dominator tree so
|
|
|
|
// that we are guaranteed to see definitions before we see uses. This allows
|
2007-08-18 17:08:56 +02:00
|
|
|
// us to sink instructions in one pass, without iteration. After sinking
|
2003-12-19 08:22:45 +01:00
|
|
|
// instructions, we perform another pass to hoist them out of the loop.
|
2002-09-29 23:46:09 +02:00
|
|
|
//
|
2009-11-05 22:11:53 +01:00
|
|
|
if (L->hasDedicatedExits())
|
2017-12-15 21:33:24 +01:00
|
|
|
Changed |= sinkRegion(DT->getNode(L->getHeader()), AA, LI, DT, TLI, TTI, L,
|
2017-01-11 05:39:35 +01:00
|
|
|
CurAST, &SafetyInfo, ORE);
|
2009-11-05 22:11:53 +01:00
|
|
|
if (Preheader)
|
2016-07-13 00:37:48 +02:00
|
|
|
Changed |= hoistRegion(DT->getNode(L->getHeader()), AA, LI, DT, TLI, L,
|
2017-01-11 05:39:35 +01:00
|
|
|
CurAST, &SafetyInfo, ORE);
|
2002-05-11 00:44:58 +02:00
|
|
|
|
2003-02-24 04:52:32 +01:00
|
|
|
// Now that all loop invariants have been removed from the loop, promote any
|
2010-08-29 08:43:52 +02:00
|
|
|
// memory references to scalars that we can.
|
2016-12-29 23:51:22 +01:00
|
|
|
// Don't sink stores from loops without dedicated block exits. Exits
|
|
|
|
// containing indirect branches are not transformed by loop simplify,
|
|
|
|
// make sure we catch that. An additional load may be generated in the
|
|
|
|
// preheader for SSA updater, so also avoid sinking when no preheader
|
|
|
|
// is available.
|
|
|
|
if (!DisablePromotion && Preheader && L->hasDedicatedExits()) {
|
2016-12-30 00:11:19 +01:00
|
|
|
// Figure out the loop exits and their insertion points
|
2012-08-08 02:00:26 +02:00
|
|
|
SmallVector<BasicBlock *, 8> ExitBlocks;
|
2016-12-30 00:11:19 +01:00
|
|
|
L->getUniqueExitBlocks(ExitBlocks);
|
|
|
|
|
|
|
|
// We can't insert into a catchswitch.
|
|
|
|
bool HasCatchSwitch = llvm::any_of(ExitBlocks, [](BasicBlock *Exit) {
|
|
|
|
return isa<CatchSwitchInst>(Exit->getTerminator());
|
|
|
|
});
|
|
|
|
|
|
|
|
if (!HasCatchSwitch) {
|
|
|
|
SmallVector<Instruction *, 8> InsertPts;
|
|
|
|
InsertPts.reserve(ExitBlocks.size());
|
|
|
|
for (BasicBlock *ExitBlock : ExitBlocks)
|
|
|
|
InsertPts.push_back(&*ExitBlock->getFirstInsertionPt());
|
|
|
|
|
|
|
|
PredIteratorCache PIC;
|
|
|
|
|
|
|
|
bool Promoted = false;
|
|
|
|
|
|
|
|
// Loop over all of the alias sets in the tracker object.
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
for (AliasSet &AS : *CurAST) {
|
|
|
|
// We can promote this alias set if it has a store, if it is a "Must"
|
|
|
|
// alias set, if the pointer is loop invariant, and if we are not
|
|
|
|
// eliminating any volatile loads or stores.
|
|
|
|
if (AS.isForwardingAliasSet() || !AS.isMod() || !AS.isMustAlias() ||
|
|
|
|
AS.isVolatile() || !L->isLoopInvariant(AS.begin()->getValue()))
|
|
|
|
continue;
|
|
|
|
|
|
|
|
assert(
|
|
|
|
!AS.empty() &&
|
|
|
|
"Must alias set should have at least one pointer element in it!");
|
|
|
|
|
|
|
|
SmallSetVector<Value *, 8> PointerMustAliases;
|
|
|
|
for (const auto &ASI : AS)
|
|
|
|
PointerMustAliases.insert(ASI.getValue());
|
|
|
|
|
|
|
|
Promoted |= promoteLoopAccessesToScalars(PointerMustAliases, ExitBlocks,
|
|
|
|
InsertPts, PIC, LI, DT, TLI, L,
|
|
|
|
CurAST, &SafetyInfo, ORE);
|
|
|
|
}
|
2016-12-30 00:11:19 +01:00
|
|
|
|
|
|
|
// Once we have promoted values across the loop body we have to
|
|
|
|
// recursively reform LCSSA as any nested loop may now have values defined
|
|
|
|
// within the loop used in the outer loop.
|
|
|
|
// FIXME: This is really heavy handed. It would be a bit better to use an
|
|
|
|
// SSAUpdater strategy during promotion that was LCSSA aware and reformed
|
|
|
|
// it as it went.
|
|
|
|
if (Promoted)
|
|
|
|
formLCSSARecursively(*L, *DT, LI, SE);
|
|
|
|
|
|
|
|
Changed |= Promoted;
|
|
|
|
}
|
2014-02-01 14:35:14 +01:00
|
|
|
}
|
|
|
|
|
2014-02-11 13:52:27 +01:00
|
|
|
// Check that neither this loop nor its parent have had LCSSA broken. LICM is
|
|
|
|
// specifically moving instructions across the loop boundary and so it is
|
|
|
|
// especially in need of sanity checking here.
|
|
|
|
assert(L->isLCSSAForm(*DT) && "Loop not left in LCSSA form after LICM!");
|
|
|
|
assert((!L->getParentLoop() || L->getParentLoop()->isLCSSAForm(*DT)) &&
|
|
|
|
"Parent loop not left in LCSSA form after LICM!");
|
2014-01-25 05:07:24 +01:00
|
|
|
|
2010-08-29 19:46:00 +02:00
|
|
|
// If this loop is nested inside of another one, save the alias information
|
|
|
|
// for when we process the outer loop.
|
2016-07-13 00:37:48 +02:00
|
|
|
if (L->getParentLoop() && !DeleteAST)
|
2010-08-29 19:46:00 +02:00
|
|
|
LoopToAliasSetMap[L] = CurAST;
|
|
|
|
else
|
|
|
|
delete CurAST;
|
2016-05-03 19:50:11 +02:00
|
|
|
|
2016-07-13 00:37:48 +02:00
|
|
|
if (Changed && SE)
|
|
|
|
SE->forgetLoopDispositions(L);
|
2007-03-07 05:41:30 +01:00
|
|
|
return Changed;
|
2002-05-11 00:44:58 +02:00
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// Walk the specified region of the CFG (defined by all blocks dominated by
|
2016-05-05 02:54:54 +02:00
|
|
|
/// the specified block, and that are in the current loop) in reverse depth
|
2015-02-22 19:35:32 +01:00
|
|
|
/// first order w.r.t the DominatorTree. This allows us to visit uses before
|
|
|
|
/// definitions, allowing us to sink a loop body in one pass without iteration.
|
2003-12-19 08:22:45 +01:00
|
|
|
///
|
2015-03-10 03:37:25 +01:00
|
|
|
bool llvm::sinkRegion(DomTreeNode *N, AliasAnalysis *AA, LoopInfo *LI,
|
2017-12-15 21:33:24 +01:00
|
|
|
DominatorTree *DT, TargetLibraryInfo *TLI,
|
|
|
|
TargetTransformInfo *TTI, Loop *CurLoop,
|
2017-01-11 05:39:35 +01:00
|
|
|
AliasSetTracker *CurAST, LoopSafetyInfo *SafetyInfo,
|
|
|
|
OptimizationRemarkEmitter *ORE) {
|
2015-02-22 19:35:32 +01:00
|
|
|
|
|
|
|
// Verify inputs.
|
2016-05-05 02:54:54 +02:00
|
|
|
assert(N != nullptr && AA != nullptr && LI != nullptr && DT != nullptr &&
|
|
|
|
CurLoop != nullptr && CurAST != nullptr && SafetyInfo != nullptr &&
|
|
|
|
"Unexpected input to sinkRegion");
|
2015-02-22 19:35:32 +01:00
|
|
|
|
2017-07-20 05:27:02 +02:00
|
|
|
// We want to visit children before parents. We will enque all the parents
|
|
|
|
// before their children in the worklist and process the worklist in reverse
|
|
|
|
// order.
|
|
|
|
SmallVector<DomTreeNode *, 16> Worklist = collectChildrenInLoop(N, CurLoop);
|
2003-12-19 08:22:45 +01:00
|
|
|
|
2016-01-14 00:01:57 +01:00
|
|
|
bool Changed = false;
|
2017-07-20 05:27:02 +02:00
|
|
|
for (DomTreeNode *DTN : reverse(Worklist)) {
|
|
|
|
BasicBlock *BB = DTN->getBlock();
|
|
|
|
// Only need to process the contents of this block if it is not part of a
|
|
|
|
// subloop (which would already have been processed).
|
|
|
|
if (inSubLoop(BB, CurLoop, LI))
|
2010-08-29 20:22:25 +02:00
|
|
|
continue;
|
2005-04-22 01:48:37 +02:00
|
|
|
|
2017-07-20 05:27:02 +02:00
|
|
|
for (BasicBlock::iterator II = BB->end(); II != BB->begin();) {
|
|
|
|
Instruction &I = *--II;
|
|
|
|
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
// If the instruction is dead, we would try to sink it because it isn't
|
|
|
|
// used in the loop, instead, just delete it.
|
2017-07-20 05:27:02 +02:00
|
|
|
if (isInstructionTriviallyDead(&I, TLI)) {
|
2018-05-14 14:53:11 +02:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM deleting dead inst: " << I << '\n');
|
2018-03-18 16:59:19 +01:00
|
|
|
salvageDebugInfo(I);
|
2017-07-20 05:27:02 +02:00
|
|
|
++II;
|
|
|
|
CurAST->deleteValue(&I);
|
|
|
|
I.eraseFromParent();
|
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Check to see if we can sink this instruction to the exit blocks
|
|
|
|
// of the loop. We can do this if the all users of the instruction are
|
|
|
|
// outside of the loop. In this case, it doesn't even matter if the
|
|
|
|
// operands of the instruction are loop invariant.
|
|
|
|
//
|
2017-12-15 21:33:24 +01:00
|
|
|
bool FreeInLoop = false;
|
|
|
|
if (isNotUsedOrFreeInLoop(I, CurLoop, SafetyInfo, TTI, FreeInLoop) &&
|
2017-07-20 05:27:02 +02:00
|
|
|
canSinkOrHoistInst(I, AA, DT, CurLoop, CurAST, SafetyInfo, ORE)) {
|
2017-12-15 21:33:24 +01:00
|
|
|
if (sink(I, LI, DT, CurLoop, SafetyInfo, ORE, FreeInLoop)) {
|
|
|
|
if (!FreeInLoop) {
|
|
|
|
++II;
|
|
|
|
CurAST->deleteValue(&I);
|
|
|
|
I.eraseFromParent();
|
|
|
|
}
|
2017-11-03 17:24:53 +01:00
|
|
|
Changed = true;
|
|
|
|
}
|
2017-07-20 05:27:02 +02:00
|
|
|
}
|
2003-12-19 09:18:16 +01:00
|
|
|
}
|
2003-12-19 08:22:45 +01:00
|
|
|
}
|
2015-02-22 19:35:32 +01:00
|
|
|
return Changed;
|
2003-12-19 08:22:45 +01:00
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// Walk the specified region of the CFG (defined by all blocks dominated by
|
|
|
|
/// the specified block, and that are in the current loop) in depth first
|
|
|
|
/// order w.r.t the DominatorTree. This allows us to visit definitions before
|
|
|
|
/// uses, allowing us to hoist a loop body in one pass without iteration.
|
2002-09-29 23:46:09 +02:00
|
|
|
///
|
2015-03-10 03:37:25 +01:00
|
|
|
bool llvm::hoistRegion(DomTreeNode *N, AliasAnalysis *AA, LoopInfo *LI,
|
|
|
|
DominatorTree *DT, TargetLibraryInfo *TLI, Loop *CurLoop,
|
2017-01-11 05:39:35 +01:00
|
|
|
AliasSetTracker *CurAST, LoopSafetyInfo *SafetyInfo,
|
|
|
|
OptimizationRemarkEmitter *ORE) {
|
2015-02-22 19:35:32 +01:00
|
|
|
// Verify inputs.
|
2016-05-05 02:54:54 +02:00
|
|
|
assert(N != nullptr && AA != nullptr && LI != nullptr && DT != nullptr &&
|
|
|
|
CurLoop != nullptr && CurAST != nullptr && SafetyInfo != nullptr &&
|
|
|
|
"Unexpected input to hoistRegion");
|
2016-01-14 00:01:57 +01:00
|
|
|
|
2017-07-20 05:27:02 +02:00
|
|
|
// We want to visit parents before children. We will enque all the parents
|
|
|
|
// before their children in the worklist and process the worklist in order.
|
|
|
|
SmallVector<DomTreeNode *, 16> Worklist = collectChildrenInLoop(N, CurLoop);
|
2016-01-14 00:01:57 +01:00
|
|
|
|
|
|
|
bool Changed = false;
|
2017-07-20 05:27:02 +02:00
|
|
|
for (DomTreeNode *DTN : Worklist) {
|
|
|
|
BasicBlock *BB = DTN->getBlock();
|
|
|
|
// Only need to process the contents of this block if it is not part of a
|
|
|
|
// subloop (which would already have been processed).
|
2018-04-27 22:58:30 +02:00
|
|
|
if (inSubLoop(BB, CurLoop, LI))
|
|
|
|
continue;
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2018-05-04 23:35:00 +02:00
|
|
|
// Keep track of whether the prefix of instructions visited so far are such
|
|
|
|
// that the next instruction visited is guaranteed to execute if the loop
|
|
|
|
// is entered.
|
|
|
|
bool IsMustExecute = CurLoop->getHeader() == BB;
|
|
|
|
|
2018-04-27 22:58:30 +02:00
|
|
|
for (BasicBlock::iterator II = BB->begin(), E = BB->end(); II != E;) {
|
|
|
|
Instruction &I = *II++;
|
|
|
|
// Try constant folding this instruction. If all the operands are
|
|
|
|
// constants, it is technically hoistable, but it would be better to
|
|
|
|
// just fold it.
|
|
|
|
if (Constant *C = ConstantFoldInstruction(
|
|
|
|
&I, I.getModule()->getDataLayout(), TLI)) {
|
2018-05-14 14:53:11 +02:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM folding inst: " << I << " --> " << *C
|
|
|
|
<< '\n');
|
2018-04-27 22:58:30 +02:00
|
|
|
CurAST->copyValue(&I, C);
|
|
|
|
I.replaceAllUsesWith(C);
|
|
|
|
if (isInstructionTriviallyDead(&I, TLI)) {
|
|
|
|
CurAST->deleteValue(&I);
|
2017-07-20 05:27:02 +02:00
|
|
|
I.eraseFromParent();
|
|
|
|
}
|
2018-04-27 22:58:30 +02:00
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Attempt to remove floating point division out of the loop by
|
|
|
|
// converting it to a reciprocal multiplication.
|
|
|
|
if (I.getOpcode() == Instruction::FDiv &&
|
|
|
|
CurLoop->isLoopInvariant(I.getOperand(1)) &&
|
|
|
|
I.hasAllowReciprocal()) {
|
|
|
|
auto Divisor = I.getOperand(1);
|
|
|
|
auto One = llvm::ConstantFP::get(Divisor->getType(), 1.0);
|
|
|
|
auto ReciprocalDivisor = BinaryOperator::CreateFDiv(One, Divisor);
|
|
|
|
ReciprocalDivisor->setFastMathFlags(I.getFastMathFlags());
|
|
|
|
ReciprocalDivisor->insertBefore(&I);
|
|
|
|
|
|
|
|
auto Product =
|
|
|
|
BinaryOperator::CreateFMul(I.getOperand(0), ReciprocalDivisor);
|
|
|
|
Product->setFastMathFlags(I.getFastMathFlags());
|
|
|
|
Product->insertAfter(&I);
|
|
|
|
I.replaceAllUsesWith(Product);
|
|
|
|
I.eraseFromParent();
|
2017-04-11 04:22:54 +02:00
|
|
|
|
2018-04-27 22:58:30 +02:00
|
|
|
hoist(*ReciprocalDivisor, DT, CurLoop, SafetyInfo, ORE);
|
|
|
|
Changed = true;
|
|
|
|
continue;
|
2017-07-20 05:27:02 +02:00
|
|
|
}
|
2018-04-27 22:58:30 +02:00
|
|
|
|
|
|
|
// Try hoisting the instruction out to the preheader. We can only do
|
|
|
|
// this if all of the operands of the instruction are loop invariant and
|
|
|
|
// if it is safe to hoist the instruction.
|
|
|
|
//
|
|
|
|
if (CurLoop->hasLoopInvariantOperands(&I) &&
|
|
|
|
canSinkOrHoistInst(I, AA, DT, CurLoop, CurAST, SafetyInfo, ORE) &&
|
2018-05-04 23:35:00 +02:00
|
|
|
(IsMustExecute ||
|
|
|
|
isSafeToExecuteUnconditionally(
|
|
|
|
I, DT, CurLoop, SafetyInfo, ORE,
|
|
|
|
CurLoop->getLoopPreheader()->getTerminator()))) {
|
2018-04-27 22:58:30 +02:00
|
|
|
Changed |= hoist(I, DT, CurLoop, SafetyInfo, ORE);
|
2018-05-04 23:35:00 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (IsMustExecute)
|
|
|
|
IsMustExecute = isGuaranteedToTransferExecutionToSuccessor(&I);
|
2018-04-27 22:58:30 +02:00
|
|
|
}
|
2017-07-20 05:27:02 +02:00
|
|
|
}
|
2002-09-29 23:46:09 +02:00
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
return Changed;
|
|
|
|
}
|
|
|
|
|
2017-02-02 14:22:03 +01:00
|
|
|
// Return true if LI is invariant within scope of the loop. LI is invariant if
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
// CurLoop is dominated by an invariant.start representing the same memory
|
|
|
|
// location and size as the memory location LI loads from, and also the
|
|
|
|
// invariant.start has no uses.
|
2017-02-02 14:22:03 +01:00
|
|
|
static bool isLoadInvariantInLoop(LoadInst *LI, DominatorTree *DT,
|
|
|
|
Loop *CurLoop) {
|
|
|
|
Value *Addr = LI->getOperand(0);
|
|
|
|
const DataLayout &DL = LI->getModule()->getDataLayout();
|
|
|
|
const uint32_t LocSizeInBits = DL.getTypeSizeInBits(
|
|
|
|
cast<PointerType>(Addr->getType())->getElementType());
|
|
|
|
|
|
|
|
// if the type is i8 addrspace(x)*, we know this is the type of
|
|
|
|
// llvm.invariant.start operand
|
|
|
|
auto *PtrInt8Ty = PointerType::get(Type::getInt8Ty(LI->getContext()),
|
|
|
|
LI->getPointerAddressSpace());
|
|
|
|
unsigned BitcastsVisited = 0;
|
|
|
|
// Look through bitcasts until we reach the i8* type (this is invariant.start
|
|
|
|
// operand type).
|
|
|
|
while (Addr->getType() != PtrInt8Ty) {
|
|
|
|
auto *BC = dyn_cast<BitCastInst>(Addr);
|
|
|
|
// Avoid traversing high number of bitcast uses.
|
|
|
|
if (++BitcastsVisited > MaxNumUsesTraversed || !BC)
|
|
|
|
return false;
|
|
|
|
Addr = BC->getOperand(0);
|
|
|
|
}
|
|
|
|
|
|
|
|
unsigned UsesVisited = 0;
|
|
|
|
// Traverse all uses of the load operand value, to see if invariant.start is
|
|
|
|
// one of the uses, and whether it dominates the load instruction.
|
|
|
|
for (auto *U : Addr->users()) {
|
|
|
|
// Avoid traversing for Load operand with high number of users.
|
|
|
|
if (++UsesVisited > MaxNumUsesTraversed)
|
|
|
|
return false;
|
|
|
|
IntrinsicInst *II = dyn_cast<IntrinsicInst>(U);
|
|
|
|
// If there are escaping uses of invariant.start instruction, the load maybe
|
|
|
|
// non-invariant.
|
|
|
|
if (!II || II->getIntrinsicID() != Intrinsic::invariant_start ||
|
2017-05-17 00:38:40 +02:00
|
|
|
!II->use_empty())
|
2017-02-02 14:22:03 +01:00
|
|
|
continue;
|
|
|
|
unsigned InvariantSizeInBits =
|
|
|
|
cast<ConstantInt>(II->getArgOperand(0))->getSExtValue() * 8;
|
|
|
|
// Confirm the invariant.start location size contains the load operand size
|
|
|
|
// in bits. Also, the invariant.start should dominate the load, and we
|
|
|
|
// should not hoist the load out of a loop that contains this dominating
|
|
|
|
// invariant.start.
|
|
|
|
if (LocSizeInBits <= InvariantSizeInBits &&
|
|
|
|
DT->properlyDominates(II->getParent(), CurLoop->getHeader()))
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
Add Loop Sink pass to reverse the LICM based of basic block frequency.
Summary: LICM may hoist instructions to preheader speculatively. Before code generation, we need to sink down the hoisted instructions inside to loop if it's beneficial. This pass is a reverse of LICM: looking at instructions in preheader and sinks the instruction to basic blocks inside the loop body if basic block frequency is smaller than the preheader frequency.
Reviewers: hfinkel, davidxl, chandlerc
Subscribers: anna, modocache, mgorny, beanz, reames, dberlin, chandlerc, mcrosier, junbuml, sanjoy, mzolotukhin, llvm-commits
Differential Revision: https://reviews.llvm.org/D22778
llvm-svn: 285308
2016-10-27 18:30:08 +02:00
|
|
|
bool llvm::canSinkOrHoistInst(Instruction &I, AAResults *AA, DominatorTree *DT,
|
|
|
|
Loop *CurLoop, AliasSetTracker *CurAST,
|
2017-01-11 05:39:45 +01:00
|
|
|
LoopSafetyInfo *SafetyInfo,
|
|
|
|
OptimizationRemarkEmitter *ORE) {
|
2017-10-11 09:26:45 +02:00
|
|
|
// SafetyInfo is nullptr if we are checking for sinking from preheader to
|
|
|
|
// loop body.
|
|
|
|
const bool SinkingToLoopBody = !SafetyInfo;
|
2003-12-09 20:32:44 +01:00
|
|
|
// Loads have extra constraints we have to verify before we can hoist them.
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(&I)) {
|
2011-08-15 22:52:09 +02:00
|
|
|
if (!LI->isUnordered())
|
2017-10-11 09:26:45 +02:00
|
|
|
return false; // Don't sink/hoist volatile or ordered atomic loads!
|
2003-12-09 20:32:44 +01:00
|
|
|
|
2008-07-23 07:06:28 +02:00
|
|
|
// Loads from constant memory are always safe to move, even if they end up
|
|
|
|
// in the same alias set as something that ends up being modified.
|
2009-11-19 20:00:10 +01:00
|
|
|
if (AA->pointsToConstantMemory(LI->getOperand(0)))
|
2008-07-23 07:06:28 +02:00
|
|
|
return true;
|
2014-10-21 02:13:20 +02:00
|
|
|
if (LI->getMetadata(LLVMContext::MD_invariant_load))
|
2011-11-08 20:30:00 +01:00
|
|
|
return true;
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2017-10-11 09:26:45 +02:00
|
|
|
if (LI->isAtomic() && SinkingToLoopBody)
|
|
|
|
return false; // Don't sink unordered atomic loads to loop body.
|
|
|
|
|
2017-02-02 14:22:03 +01:00
|
|
|
// This checks for an invariant.start dominating the load.
|
|
|
|
if (isLoadInvariantInLoop(LI, DT, CurLoop))
|
|
|
|
return true;
|
|
|
|
|
2003-12-09 20:32:44 +01:00
|
|
|
// Don't hoist loads which have may-aliased stores in loop.
|
2010-10-20 00:54:46 +02:00
|
|
|
uint64_t Size = 0;
|
2004-11-26 22:20:09 +01:00
|
|
|
if (LI->getType()->isSized())
|
2016-09-02 03:59:27 +02:00
|
|
|
Size = I.getModule()->getDataLayout().getTypeStoreSize(LI->getType());
|
2014-07-24 14:16:19 +02:00
|
|
|
|
|
|
|
AAMDNodes AAInfo;
|
|
|
|
LI->getAAMetadata(AAInfo);
|
|
|
|
|
2017-01-11 05:39:45 +01:00
|
|
|
bool Invalidated =
|
|
|
|
pointerInvalidatedByLoop(LI->getOperand(0), Size, AAInfo, CurAST);
|
|
|
|
// Check loop-invariant address because this may also be a sinkable load
|
|
|
|
// whose address is not necessarily loop-invariant.
|
|
|
|
if (ORE && Invalidated && CurLoop->isLoopInvariant(LI->getPointerOperand()))
|
2017-10-11 19:12:59 +02:00
|
|
|
ORE->emit([&]() {
|
|
|
|
return OptimizationRemarkMissed(
|
|
|
|
DEBUG_TYPE, "LoadWithLoopInvariantAddressInvalidated", LI)
|
|
|
|
<< "failed to move load with loop-invariant address "
|
|
|
|
"because the loop may invalidate its value";
|
|
|
|
});
|
2017-01-11 05:39:45 +01:00
|
|
|
|
|
|
|
return !Invalidated;
|
2004-03-15 05:11:30 +01:00
|
|
|
} else if (CallInst *CI = dyn_cast<CallInst>(&I)) {
|
2011-05-27 20:37:52 +02:00
|
|
|
// Don't sink or hoist dbg info; it's legal, but not useful.
|
2016-09-02 03:59:27 +02:00
|
|
|
if (isa<DbgInfoIntrinsic>(I))
|
2011-05-27 20:37:52 +02:00
|
|
|
return false;
|
|
|
|
|
2016-01-04 04:37:39 +01:00
|
|
|
// Don't sink calls which can throw.
|
|
|
|
if (CI->mayThrow())
|
|
|
|
return false;
|
|
|
|
|
2011-05-27 20:37:52 +02:00
|
|
|
// Handle simple cases by querying alias analysis.
|
2015-07-23 01:15:57 +02:00
|
|
|
FunctionModRefBehavior Behavior = AA->getModRefBehavior(CI);
|
|
|
|
if (Behavior == FMRB_DoesNotAccessMemory)
|
2007-12-01 08:51:45 +01:00
|
|
|
return true;
|
2010-11-09 20:58:21 +01:00
|
|
|
if (AliasAnalysis::onlyReadsMemory(Behavior)) {
|
2015-09-22 00:27:59 +02:00
|
|
|
// A readonly argmemonly function only reads from memory pointed to by
|
|
|
|
// it's arguments with arbitrary offsets. If we can prove there are no
|
|
|
|
// writes to this memory in the loop, we can hoist or sink.
|
|
|
|
if (AliasAnalysis::onlyAccessesArgPointees(Behavior)) {
|
|
|
|
for (Value *Op : CI->arg_operands())
|
|
|
|
if (Op->getType()->isPointerTy() &&
|
|
|
|
pointerInvalidatedByLoop(Op, MemoryLocation::UnknownSize,
|
|
|
|
AAMDNodes(), CurAST))
|
|
|
|
return false;
|
|
|
|
return true;
|
|
|
|
}
|
2007-12-01 08:51:45 +01:00
|
|
|
// If this call only reads from memory and there are no writes to memory
|
|
|
|
// in the loop, we can hoist or sink the call as appropriate.
|
|
|
|
bool FoundMod = false;
|
2016-01-08 23:59:42 +01:00
|
|
|
for (AliasSet &AS : *CurAST) {
|
2007-12-01 08:51:45 +01:00
|
|
|
if (!AS.isForwardingAliasSet() && AS.isMod()) {
|
|
|
|
FoundMod = true;
|
|
|
|
break;
|
2004-03-15 05:11:30 +01:00
|
|
|
}
|
|
|
|
}
|
2016-05-05 02:54:54 +02:00
|
|
|
if (!FoundMod)
|
|
|
|
return true;
|
2004-03-15 05:11:30 +01:00
|
|
|
}
|
|
|
|
|
2012-09-04 12:25:04 +02:00
|
|
|
// FIXME: This should use mod/ref information to see if we can hoist or
|
|
|
|
// sink the call.
|
2016-09-02 03:59:27 +02:00
|
|
|
|
2004-03-15 05:11:30 +01:00
|
|
|
return false;
|
2003-12-09 20:32:44 +01:00
|
|
|
}
|
|
|
|
|
2016-09-02 03:59:27 +02:00
|
|
|
// Only these instructions are hoistable/sinkable.
|
|
|
|
if (!isa<BinaryOperator>(I) && !isa<CastInst>(I) && !isa<SelectInst>(I) &&
|
|
|
|
!isa<GetElementPtrInst>(I) && !isa<CmpInst>(I) &&
|
|
|
|
!isa<InsertElementInst>(I) && !isa<ExtractElementInst>(I) &&
|
|
|
|
!isa<ShuffleVectorInst>(I) && !isa<ExtractValueInst>(I) &&
|
|
|
|
!isa<InsertValueInst>(I))
|
|
|
|
return false;
|
|
|
|
|
2017-10-11 09:26:45 +02:00
|
|
|
// If we are checking for sinking from preheader to loop body it will be
|
|
|
|
// always safe as there is no speculative execution.
|
|
|
|
if (SinkingToLoopBody)
|
2016-10-03 20:52:08 +02:00
|
|
|
return true;
|
|
|
|
|
2016-09-02 03:59:27 +02:00
|
|
|
// TODO: Plumb the context instruction through to make hoisting and sinking
|
|
|
|
// more powerful. Hoisting of loads already works due to the special casing
|
|
|
|
// above.
|
|
|
|
return isSafeToExecuteUnconditionally(I, DT, CurLoop, SafetyInfo, nullptr);
|
2003-12-10 07:41:05 +01:00
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// Returns true if a PHINode is a trivially replaceable with an
|
2014-01-25 05:07:24 +01:00
|
|
|
/// Instruction.
|
2015-02-22 19:35:32 +01:00
|
|
|
/// This is true when all incoming values are that instruction.
|
|
|
|
/// This pattern occurs most often with LCSSA PHI nodes.
|
2014-01-25 05:07:24 +01:00
|
|
|
///
|
2015-05-12 22:05:20 +02:00
|
|
|
static bool isTriviallyReplacablePHI(const PHINode &PN, const Instruction &I) {
|
2015-05-12 22:05:31 +02:00
|
|
|
for (const Value *IncValue : PN.incoming_values())
|
|
|
|
if (IncValue != &I)
|
2014-01-25 05:07:24 +01:00
|
|
|
return false;
|
|
|
|
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2017-12-15 21:33:24 +01:00
|
|
|
/// Return true if the instruction is free in the loop.
|
|
|
|
static bool isFreeInLoop(const Instruction &I, const Loop *CurLoop,
|
|
|
|
const TargetTransformInfo *TTI) {
|
|
|
|
|
|
|
|
if (const GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(&I)) {
|
|
|
|
if (TTI->getUserCost(GEP) != TargetTransformInfo::TCC_Free)
|
|
|
|
return false;
|
|
|
|
// For a GEP, we cannot simply use getUserCost because currently it
|
|
|
|
// optimistically assume that a GEP will fold into addressing mode
|
|
|
|
// regardless of its users.
|
|
|
|
const BasicBlock *BB = GEP->getParent();
|
|
|
|
for (const User *U : GEP->users()) {
|
|
|
|
const Instruction *UI = cast<Instruction>(U);
|
|
|
|
if (CurLoop->contains(UI) &&
|
|
|
|
(BB != UI->getParent() ||
|
|
|
|
(!isa<StoreInst>(UI) && !isa<LoadInst>(UI))))
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
} else
|
|
|
|
return TTI->getUserCost(&I) == TargetTransformInfo::TCC_Free;
|
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// Return true if the only users of this instruction are outside of
|
|
|
|
/// the loop. If this is true, we can sink the instruction to the exit
|
|
|
|
/// blocks of the loop.
|
2003-12-10 07:41:05 +01:00
|
|
|
///
|
2017-12-15 21:33:24 +01:00
|
|
|
/// We also return true if the instruction could be folded away in lowering.
|
|
|
|
/// (e.g., a GEP can be folded into a load as an addressing mode in the loop).
|
|
|
|
static bool isNotUsedOrFreeInLoop(const Instruction &I, const Loop *CurLoop,
|
|
|
|
const LoopSafetyInfo *SafetyInfo,
|
|
|
|
TargetTransformInfo *TTI, bool &FreeInLoop) {
|
2016-01-04 04:37:39 +01:00
|
|
|
const auto &BlockColors = SafetyInfo->BlockColors;
|
2017-12-15 21:33:24 +01:00
|
|
|
bool IsFree = isFreeInLoop(I, CurLoop, TTI);
|
2015-05-13 03:12:18 +02:00
|
|
|
for (const User *U : I.users()) {
|
|
|
|
const Instruction *UI = cast<Instruction>(U);
|
|
|
|
if (const PHINode *PN = dyn_cast<PHINode>(UI)) {
|
2016-01-04 04:37:39 +01:00
|
|
|
const BasicBlock *BB = PN->getParent();
|
|
|
|
// We cannot sink uses in catchswitches.
|
|
|
|
if (isa<CatchSwitchInst>(BB->getTerminator()))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// We need to sink a callsite to a unique funclet. Avoid sinking if the
|
|
|
|
// phi use is too muddled.
|
|
|
|
if (isa<CallInst>(I))
|
|
|
|
if (!BlockColors.empty() &&
|
|
|
|
BlockColors.find(const_cast<BasicBlock *>(BB))->second.size() != 1)
|
|
|
|
return false;
|
2003-12-11 23:23:32 +01:00
|
|
|
}
|
2014-01-25 05:07:24 +01:00
|
|
|
|
2017-12-15 21:33:24 +01:00
|
|
|
if (CurLoop->contains(UI)) {
|
|
|
|
if (IsFree) {
|
|
|
|
FreeInLoop = true;
|
|
|
|
continue;
|
|
|
|
}
|
2014-01-25 05:07:24 +01:00
|
|
|
return false;
|
2017-12-15 21:33:24 +01:00
|
|
|
}
|
2003-12-11 23:23:32 +01:00
|
|
|
}
|
2003-12-10 07:41:05 +01:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2016-01-04 04:37:39 +01:00
|
|
|
static Instruction *
|
|
|
|
CloneInstructionInExitBlock(Instruction &I, BasicBlock &ExitBlock, PHINode &PN,
|
|
|
|
const LoopInfo *LI,
|
2016-06-10 22:03:17 +02:00
|
|
|
const LoopSafetyInfo *SafetyInfo) {
|
2016-01-04 04:37:39 +01:00
|
|
|
Instruction *New;
|
|
|
|
if (auto *CI = dyn_cast<CallInst>(&I)) {
|
|
|
|
const auto &BlockColors = SafetyInfo->BlockColors;
|
|
|
|
|
|
|
|
// Sinking call-sites need to be handled differently from other
|
|
|
|
// instructions. The cloned call-site needs a funclet bundle operand
|
|
|
|
// appropriate for it's location in the CFG.
|
|
|
|
SmallVector<OperandBundleDef, 1> OpBundles;
|
|
|
|
for (unsigned BundleIdx = 0, BundleEnd = CI->getNumOperandBundles();
|
|
|
|
BundleIdx != BundleEnd; ++BundleIdx) {
|
|
|
|
OperandBundleUse Bundle = CI->getOperandBundleAt(BundleIdx);
|
|
|
|
if (Bundle.getTagID() == LLVMContext::OB_funclet)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
OpBundles.emplace_back(Bundle);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!BlockColors.empty()) {
|
|
|
|
const ColorVector &CV = BlockColors.find(&ExitBlock)->second;
|
|
|
|
assert(CV.size() == 1 && "non-unique color for exit block!");
|
|
|
|
BasicBlock *BBColor = CV.front();
|
|
|
|
Instruction *EHPad = BBColor->getFirstNonPHI();
|
|
|
|
if (EHPad->isEHPad())
|
|
|
|
OpBundles.emplace_back("funclet", EHPad);
|
|
|
|
}
|
|
|
|
|
|
|
|
New = CallInst::Create(CI, OpBundles);
|
|
|
|
} else {
|
|
|
|
New = I.clone();
|
|
|
|
}
|
|
|
|
|
2014-06-25 11:17:21 +02:00
|
|
|
ExitBlock.getInstList().insert(ExitBlock.getFirstInsertionPt(), New);
|
2016-05-05 02:54:54 +02:00
|
|
|
if (!I.getName().empty())
|
|
|
|
New->setName(I.getName() + ".le");
|
2014-06-25 11:17:21 +02:00
|
|
|
|
|
|
|
// Build LCSSA PHI nodes for any in-loop operands. Note that this is
|
|
|
|
// particularly cheap because we can rip off the PHI node that we're
|
|
|
|
// replacing for the number and blocks of the predecessors.
|
|
|
|
// OPT: If this shows up in a profile, we can instead finish sinking all
|
|
|
|
// invariant instructions, and then walk their operands to re-establish
|
|
|
|
// LCSSA. That will eliminate creating PHI nodes just to nuke them when
|
|
|
|
// sinking bottom-up.
|
|
|
|
for (User::op_iterator OI = New->op_begin(), OE = New->op_end(); OI != OE;
|
|
|
|
++OI)
|
|
|
|
if (Instruction *OInst = dyn_cast<Instruction>(*OI))
|
|
|
|
if (Loop *OLoop = LI->getLoopFor(OInst->getParent()))
|
|
|
|
if (!OLoop->contains(&PN)) {
|
|
|
|
PHINode *OpPN =
|
|
|
|
PHINode::Create(OInst->getType(), PN.getNumIncomingValues(),
|
2015-10-13 21:26:58 +02:00
|
|
|
OInst->getName() + ".lcssa", &ExitBlock.front());
|
2014-06-25 11:17:21 +02:00
|
|
|
for (unsigned i = 0, e = PN.getNumIncomingValues(); i != e; ++i)
|
|
|
|
OpPN->addIncoming(OInst, PN.getIncomingBlock(i));
|
|
|
|
*OI = OpPN;
|
|
|
|
}
|
|
|
|
return New;
|
|
|
|
}
|
|
|
|
|
2017-11-03 17:24:53 +01:00
|
|
|
static Instruction *sinkThroughTriviallyReplacablePHI(
|
|
|
|
PHINode *TPN, Instruction *I, LoopInfo *LI,
|
|
|
|
SmallDenseMap<BasicBlock *, Instruction *, 32> &SunkCopies,
|
|
|
|
const LoopSafetyInfo *SafetyInfo, const Loop *CurLoop) {
|
|
|
|
assert(isTriviallyReplacablePHI(*TPN, *I) &&
|
|
|
|
"Expect only trivially replacalbe PHI");
|
|
|
|
BasicBlock *ExitBlock = TPN->getParent();
|
|
|
|
Instruction *New;
|
|
|
|
auto It = SunkCopies.find(ExitBlock);
|
|
|
|
if (It != SunkCopies.end())
|
|
|
|
New = It->second;
|
|
|
|
else
|
|
|
|
New = SunkCopies[ExitBlock] =
|
|
|
|
CloneInstructionInExitBlock(*I, *ExitBlock, *TPN, LI, SafetyInfo);
|
|
|
|
return New;
|
|
|
|
}
|
|
|
|
|
2018-02-12 18:56:55 +01:00
|
|
|
static bool canSplitPredecessors(PHINode *PN, LoopSafetyInfo *SafetyInfo) {
|
2017-11-03 17:24:53 +01:00
|
|
|
BasicBlock *BB = PN->getParent();
|
|
|
|
if (!BB->canSplitPredecessors())
|
|
|
|
return false;
|
2018-02-12 18:56:55 +01:00
|
|
|
// It's not impossible to split EHPad blocks, but if BlockColors already exist
|
|
|
|
// it require updating BlockColors for all offspring blocks accordingly. By
|
|
|
|
// skipping such corner case, we can make updating BlockColors after splitting
|
|
|
|
// predecessor fairly simple.
|
|
|
|
if (!SafetyInfo->BlockColors.empty() && BB->getFirstNonPHI()->isEHPad())
|
|
|
|
return false;
|
2017-11-03 17:24:53 +01:00
|
|
|
for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) {
|
|
|
|
BasicBlock *BBPred = *PI;
|
|
|
|
if (isa<IndirectBrInst>(BBPred->getTerminator()))
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void splitPredecessorsOfLoopExit(PHINode *PN, DominatorTree *DT,
|
2018-02-12 18:56:55 +01:00
|
|
|
LoopInfo *LI, const Loop *CurLoop,
|
|
|
|
LoopSafetyInfo *SafetyInfo) {
|
2017-11-03 17:24:53 +01:00
|
|
|
#ifndef NDEBUG
|
|
|
|
SmallVector<BasicBlock *, 32> ExitBlocks;
|
|
|
|
CurLoop->getUniqueExitBlocks(ExitBlocks);
|
|
|
|
SmallPtrSet<BasicBlock *, 32> ExitBlockSet(ExitBlocks.begin(),
|
|
|
|
ExitBlocks.end());
|
|
|
|
#endif
|
|
|
|
BasicBlock *ExitBB = PN->getParent();
|
|
|
|
assert(ExitBlockSet.count(ExitBB) && "Expect the PHI is in an exit block.");
|
|
|
|
|
|
|
|
// Split predecessors of the loop exit to make instructions in the loop are
|
|
|
|
// exposed to exit blocks through trivially replacable PHIs while keeping the
|
|
|
|
// loop in the canonical form where each predecessor of each exit block should
|
|
|
|
// be contained within the loop. For example, this will convert the loop below
|
|
|
|
// from
|
|
|
|
//
|
|
|
|
// LB1:
|
|
|
|
// %v1 =
|
|
|
|
// br %LE, %LB2
|
|
|
|
// LB2:
|
|
|
|
// %v2 =
|
|
|
|
// br %LE, %LB1
|
|
|
|
// LE:
|
|
|
|
// %p = phi [%v1, %LB1], [%v2, %LB2] <-- non-trivially replacable
|
|
|
|
//
|
|
|
|
// to
|
|
|
|
//
|
|
|
|
// LB1:
|
|
|
|
// %v1 =
|
|
|
|
// br %LE.split, %LB2
|
|
|
|
// LB2:
|
|
|
|
// %v2 =
|
|
|
|
// br %LE.split2, %LB1
|
|
|
|
// LE.split:
|
|
|
|
// %p1 = phi [%v1, %LB1] <-- trivially replacable
|
|
|
|
// br %LE
|
|
|
|
// LE.split2:
|
|
|
|
// %p2 = phi [%v2, %LB2] <-- trivially replacable
|
|
|
|
// br %LE
|
|
|
|
// LE:
|
|
|
|
// %p = phi [%p1, %LE.split], [%p2, %LE.split2]
|
|
|
|
//
|
2018-02-12 18:56:55 +01:00
|
|
|
auto &BlockColors = SafetyInfo->BlockColors;
|
2017-11-03 17:24:53 +01:00
|
|
|
SmallSetVector<BasicBlock *, 8> PredBBs(pred_begin(ExitBB), pred_end(ExitBB));
|
|
|
|
while (!PredBBs.empty()) {
|
|
|
|
BasicBlock *PredBB = *PredBBs.begin();
|
|
|
|
assert(CurLoop->contains(PredBB) &&
|
|
|
|
"Expect all predecessors are in the loop");
|
2018-02-12 18:56:55 +01:00
|
|
|
if (PN->getBasicBlockIndex(PredBB) >= 0) {
|
|
|
|
BasicBlock *NewPred = SplitBlockPredecessors(
|
|
|
|
ExitBB, PredBB, ".split.loop.exit", DT, LI, true);
|
|
|
|
// Since we do not allow splitting EH-block with BlockColors in
|
|
|
|
// canSplitPredecessors(), we can simply assign predecessor's color to
|
|
|
|
// the new block.
|
2018-03-23 18:36:18 +01:00
|
|
|
if (!BlockColors.empty()) {
|
|
|
|
// Grab a reference to the ColorVector to be inserted before getting the
|
|
|
|
// reference to the vector we are copying because inserting the new
|
|
|
|
// element in BlockColors might cause the map to be reallocated.
|
|
|
|
ColorVector &ColorsForNewBlock = BlockColors[NewPred];
|
|
|
|
ColorVector &ColorsForOldBlock = BlockColors[PredBB];
|
|
|
|
ColorsForNewBlock = ColorsForOldBlock;
|
|
|
|
}
|
2018-02-12 18:56:55 +01:00
|
|
|
}
|
2017-11-03 17:24:53 +01:00
|
|
|
PredBBs.remove(PredBB);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// When an instruction is found to only be used outside of the loop, this
|
|
|
|
/// function moves it to the exit blocks and patches up SSA form as needed.
|
2003-12-19 09:18:16 +01:00
|
|
|
/// This method is guaranteed to remove the original instruction from its
|
|
|
|
/// position, and may either delete it or move it to outside of the loop.
|
2003-12-10 07:41:05 +01:00
|
|
|
///
|
2017-11-03 17:24:53 +01:00
|
|
|
static bool sink(Instruction &I, LoopInfo *LI, DominatorTree *DT,
|
2018-02-12 18:56:55 +01:00
|
|
|
const Loop *CurLoop, LoopSafetyInfo *SafetyInfo,
|
2017-12-15 21:33:24 +01:00
|
|
|
OptimizationRemarkEmitter *ORE, bool FreeInLoop) {
|
2018-05-14 14:53:11 +02:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM sinking instruction: " << I << "\n");
|
2017-10-11 19:12:59 +02:00
|
|
|
ORE->emit([&]() {
|
|
|
|
return OptimizationRemark(DEBUG_TYPE, "InstSunk", &I)
|
|
|
|
<< "sinking " << ore::NV("Inst", &I);
|
|
|
|
});
|
2015-02-22 19:35:32 +01:00
|
|
|
bool Changed = false;
|
2016-05-05 02:54:54 +02:00
|
|
|
if (isa<LoadInst>(I))
|
|
|
|
++NumMovedLoads;
|
|
|
|
else if (isa<CallInst>(I))
|
|
|
|
++NumMovedCalls;
|
2003-12-10 21:43:29 +01:00
|
|
|
++NumSunk;
|
|
|
|
|
2017-11-03 17:24:53 +01:00
|
|
|
// Iterate over users to be ready for actual sinking. Replace users via
|
|
|
|
// unrechable blocks with undef and make all user PHIs trivially replcable.
|
|
|
|
SmallPtrSet<Instruction *, 8> VisitedUsers;
|
|
|
|
for (Value::user_iterator UI = I.user_begin(), UE = I.user_end(); UI != UE;) {
|
|
|
|
auto *User = cast<Instruction>(*UI);
|
|
|
|
Use &U = UI.getUse();
|
|
|
|
++UI;
|
2014-02-11 13:52:27 +01:00
|
|
|
|
2017-12-15 21:33:24 +01:00
|
|
|
if (VisitedUsers.count(User) || CurLoop->contains(User))
|
2017-11-03 17:24:53 +01:00
|
|
|
continue;
|
2014-06-25 09:54:58 +02:00
|
|
|
|
2014-09-02 18:22:00 +02:00
|
|
|
if (!DT->isReachableFromEntry(User->getParent())) {
|
2017-11-17 21:38:25 +01:00
|
|
|
U = UndefValue::get(I.getType());
|
2017-12-15 21:33:24 +01:00
|
|
|
Changed = true;
|
2014-09-02 18:22:00 +02:00
|
|
|
continue;
|
|
|
|
}
|
2017-11-03 17:24:53 +01:00
|
|
|
|
2014-02-11 13:52:27 +01:00
|
|
|
// The user must be a PHI node.
|
2014-09-02 18:22:00 +02:00
|
|
|
PHINode *PN = cast<PHINode>(User);
|
2014-02-11 13:52:27 +01:00
|
|
|
|
2015-07-12 05:53:05 +02:00
|
|
|
// Surprisingly, instructions can be used outside of loops without any
|
|
|
|
// exits. This can only happen in PHI nodes if the incoming block is
|
|
|
|
// unreachable.
|
|
|
|
BasicBlock *BB = PN->getIncomingBlock(U);
|
|
|
|
if (!DT->isReachableFromEntry(BB)) {
|
|
|
|
U = UndefValue::get(I.getType());
|
2017-12-15 21:33:24 +01:00
|
|
|
Changed = true;
|
2015-07-12 05:53:05 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2017-11-03 17:24:53 +01:00
|
|
|
VisitedUsers.insert(PN);
|
|
|
|
if (isTriviallyReplacablePHI(*PN, I))
|
|
|
|
continue;
|
|
|
|
|
2018-02-12 18:56:55 +01:00
|
|
|
if (!canSplitPredecessors(PN, SafetyInfo))
|
2017-12-15 21:33:24 +01:00
|
|
|
return Changed;
|
2014-02-11 13:52:27 +01:00
|
|
|
|
2017-11-03 17:24:53 +01:00
|
|
|
// Split predecessors of the PHI so that we can make users trivially
|
|
|
|
// replacable.
|
2018-02-12 18:56:55 +01:00
|
|
|
splitPredecessorsOfLoopExit(PN, DT, LI, CurLoop, SafetyInfo);
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2017-11-03 17:24:53 +01:00
|
|
|
// Should rebuild the iterators, as they may be invalidated by
|
|
|
|
// splitPredecessorsOfLoopExit().
|
|
|
|
UI = I.user_begin();
|
|
|
|
UE = I.user_end();
|
|
|
|
}
|
|
|
|
|
2017-12-15 21:33:24 +01:00
|
|
|
if (VisitedUsers.empty())
|
|
|
|
return Changed;
|
|
|
|
|
2017-11-03 17:24:53 +01:00
|
|
|
#ifndef NDEBUG
|
|
|
|
SmallVector<BasicBlock *, 32> ExitBlocks;
|
|
|
|
CurLoop->getUniqueExitBlocks(ExitBlocks);
|
|
|
|
SmallPtrSet<BasicBlock *, 32> ExitBlockSet(ExitBlocks.begin(),
|
|
|
|
ExitBlocks.end());
|
|
|
|
#endif
|
|
|
|
|
|
|
|
// Clones of this instruction. Don't create more than one per exit block!
|
|
|
|
SmallDenseMap<BasicBlock *, Instruction *, 32> SunkCopies;
|
|
|
|
|
|
|
|
// If this instruction is only used outside of the loop, then all users are
|
|
|
|
// PHI nodes in exit blocks due to LCSSA form. Just RAUW them with clones of
|
|
|
|
// the instruction.
|
2017-12-15 21:33:24 +01:00
|
|
|
SmallSetVector<User*, 8> Users(I.user_begin(), I.user_end());
|
|
|
|
for (auto *UI : Users) {
|
|
|
|
auto *User = cast<Instruction>(UI);
|
|
|
|
|
|
|
|
if (CurLoop->contains(User))
|
|
|
|
continue;
|
|
|
|
|
|
|
|
PHINode *PN = cast<PHINode>(User);
|
2017-11-03 17:24:53 +01:00
|
|
|
assert(ExitBlockSet.count(PN->getParent()) &&
|
|
|
|
"The LCSSA PHI is not in an exit block!");
|
|
|
|
// The PHI must be trivially replacable.
|
|
|
|
Instruction *New = sinkThroughTriviallyReplacablePHI(PN, &I, LI, SunkCopies,
|
|
|
|
SafetyInfo, CurLoop);
|
2014-02-11 13:52:27 +01:00
|
|
|
PN->replaceAllUsesWith(New);
|
|
|
|
PN->eraseFromParent();
|
2017-12-15 21:33:24 +01:00
|
|
|
Changed = true;
|
2003-12-10 07:41:05 +01:00
|
|
|
}
|
2015-02-22 19:35:32 +01:00
|
|
|
return Changed;
|
2003-12-10 07:41:05 +01:00
|
|
|
}
|
2002-09-29 23:46:09 +02:00
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// When an instruction is found to only use loop invariant operands that
|
|
|
|
/// is safe to hoist, this instruction is called to do the dirty work.
|
2002-09-26 18:52:07 +02:00
|
|
|
///
|
2016-01-28 16:51:58 +01:00
|
|
|
static bool hoist(Instruction &I, const DominatorTree *DT, const Loop *CurLoop,
|
2017-01-11 05:39:35 +01:00
|
|
|
const LoopSafetyInfo *SafetyInfo,
|
|
|
|
OptimizationRemarkEmitter *ORE) {
|
2016-01-28 16:51:58 +01:00
|
|
|
auto *Preheader = CurLoop->getLoopPreheader();
|
2018-05-14 14:53:11 +02:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": " << I
|
|
|
|
<< "\n");
|
2017-10-11 19:12:59 +02:00
|
|
|
ORE->emit([&]() {
|
|
|
|
return OptimizationRemark(DEBUG_TYPE, "Hoisted", &I) << "hoisting "
|
|
|
|
<< ore::NV("Inst", &I);
|
|
|
|
});
|
2016-01-28 16:51:58 +01:00
|
|
|
|
|
|
|
// Metadata can be dependent on conditions we are hoisting above.
|
|
|
|
// Conservatively strip all metadata on the instruction unless we were
|
|
|
|
// guaranteed to execute I if we entered the loop, in which case the metadata
|
|
|
|
// is valid in the loop preheader.
|
|
|
|
if (I.hasMetadataOtherThanDebugLoc() &&
|
|
|
|
// The check on hasMetadataOtherThanDebugLoc is to prevent us from burning
|
|
|
|
// time in isGuaranteedToExecute if we don't actually have anything to
|
|
|
|
// drop. It is a compile time optimization, not required for correctness.
|
|
|
|
!isGuaranteedToExecute(I, DT, CurLoop, SafetyInfo))
|
|
|
|
I.dropUnknownNonDebugMetadata();
|
|
|
|
|
2010-08-29 20:18:40 +02:00
|
|
|
// Move the new node to the Preheader, before its terminator.
|
|
|
|
I.moveBefore(Preheader->getTerminator());
|
2005-04-22 01:48:37 +02:00
|
|
|
|
2017-01-06 19:38:57 +01:00
|
|
|
// Do not retain debug locations when we are moving instructions to different
|
|
|
|
// basic blocks, because we want to avoid jumpy line tables. Calls, however,
|
|
|
|
// need to retain their debug locs because they may be inlined.
|
|
|
|
// FIXME: How do we retain source locations without causing poor debugging
|
|
|
|
// behavior?
|
|
|
|
if (!isa<CallInst>(I))
|
|
|
|
I.setDebugLoc(DebugLoc());
|
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
if (isa<LoadInst>(I))
|
|
|
|
++NumMovedLoads;
|
|
|
|
else if (isa<CallInst>(I))
|
|
|
|
++NumMovedCalls;
|
2002-09-26 18:38:03 +02:00
|
|
|
++NumHoisted;
|
2015-02-22 19:35:32 +01:00
|
|
|
return true;
|
2002-05-11 00:44:58 +02:00
|
|
|
}
|
|
|
|
|
2015-05-18 20:07:00 +02:00
|
|
|
/// Only sink or hoist an instruction if it is not a trapping instruction,
|
|
|
|
/// or if the instruction is known not to trap when moved to the preheader.
|
2015-02-22 19:35:32 +01:00
|
|
|
/// or if it is a trapping instruction and is guaranteed to execute.
|
2017-01-11 05:39:49 +01:00
|
|
|
static bool isSafeToExecuteUnconditionally(Instruction &Inst,
|
2015-05-13 03:12:18 +02:00
|
|
|
const DominatorTree *DT,
|
|
|
|
const Loop *CurLoop,
|
2016-06-10 22:03:17 +02:00
|
|
|
const LoopSafetyInfo *SafetyInfo,
|
2017-01-11 05:39:49 +01:00
|
|
|
OptimizationRemarkEmitter *ORE,
|
2015-05-22 04:14:05 +02:00
|
|
|
const Instruction *CtxI) {
|
2016-07-03 01:47:27 +02:00
|
|
|
if (isSafeToSpeculativelyExecute(&Inst, CtxI, DT))
|
2009-07-17 06:28:42 +02:00
|
|
|
return true;
|
2005-04-22 01:48:37 +02:00
|
|
|
|
2017-01-11 05:39:49 +01:00
|
|
|
bool GuaranteedToExecute =
|
|
|
|
isGuaranteedToExecute(Inst, DT, CurLoop, SafetyInfo);
|
|
|
|
|
|
|
|
if (!GuaranteedToExecute) {
|
|
|
|
auto *LI = dyn_cast<LoadInst>(&Inst);
|
|
|
|
if (LI && CurLoop->isLoopInvariant(LI->getPointerOperand()))
|
2017-10-11 19:12:59 +02:00
|
|
|
ORE->emit([&]() {
|
|
|
|
return OptimizationRemarkMissed(
|
|
|
|
DEBUG_TYPE, "LoadWithLoopInvariantAddressCondExecuted", LI)
|
|
|
|
<< "failed to hoist load with loop-invariant address "
|
|
|
|
"because load is conditionally executed";
|
|
|
|
});
|
2017-01-11 05:39:49 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
return GuaranteedToExecute;
|
2011-07-20 23:37:47 +02:00
|
|
|
}
|
|
|
|
|
2011-01-15 01:12:35 +01:00
|
|
|
namespace {
|
2016-05-05 02:54:54 +02:00
|
|
|
class LoopPromoter : public LoadAndStorePromoter {
|
|
|
|
Value *SomePtr; // Designated pointer to store to.
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
const SmallSetVector<Value *, 8> &PointerMustAliases;
|
2016-05-05 02:54:54 +02:00
|
|
|
SmallVectorImpl<BasicBlock *> &LoopExitBlocks;
|
|
|
|
SmallVectorImpl<Instruction *> &LoopInsertPts;
|
|
|
|
PredIteratorCache &PredCache;
|
|
|
|
AliasSetTracker &AST;
|
|
|
|
LoopInfo &LI;
|
|
|
|
DebugLoc DL;
|
|
|
|
int Alignment;
|
2017-02-14 02:38:31 +01:00
|
|
|
bool UnorderedAtomic;
|
2016-05-05 02:54:54 +02:00
|
|
|
AAMDNodes AATags;
|
2014-02-11 13:52:27 +01:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
Value *maybeInsertLCSSAPHI(Value *V, BasicBlock *BB) const {
|
|
|
|
if (Instruction *I = dyn_cast<Instruction>(V))
|
|
|
|
if (Loop *L = LI.getLoopFor(I->getParent()))
|
|
|
|
if (!L->contains(BB)) {
|
|
|
|
// We need to create an LCSSA PHI node for the incoming value and
|
|
|
|
// store that.
|
|
|
|
PHINode *PN = PHINode::Create(I->getType(), PredCache.size(BB),
|
|
|
|
I->getName() + ".lcssa", &BB->front());
|
|
|
|
for (BasicBlock *Pred : PredCache.get(BB))
|
|
|
|
PN->addIncoming(I, Pred);
|
|
|
|
return PN;
|
|
|
|
}
|
|
|
|
return V;
|
|
|
|
}
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
public:
|
|
|
|
LoopPromoter(Value *SP, ArrayRef<const Instruction *> Insts, SSAUpdater &S,
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
const SmallSetVector<Value *, 8> &PMA,
|
2016-05-05 02:54:54 +02:00
|
|
|
SmallVectorImpl<BasicBlock *> &LEB,
|
|
|
|
SmallVectorImpl<Instruction *> &LIP, PredIteratorCache &PIC,
|
|
|
|
AliasSetTracker &ast, LoopInfo &li, DebugLoc dl, int alignment,
|
2017-02-14 02:38:31 +01:00
|
|
|
bool UnorderedAtomic, const AAMDNodes &AATags)
|
2016-05-05 02:54:54 +02:00
|
|
|
: LoadAndStorePromoter(Insts, S), SomePtr(SP), PointerMustAliases(PMA),
|
|
|
|
LoopExitBlocks(LEB), LoopInsertPts(LIP), PredCache(PIC), AST(ast),
|
2017-02-14 02:38:31 +01:00
|
|
|
LI(li), DL(std::move(dl)), Alignment(alignment),
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
UnorderedAtomic(UnorderedAtomic), AATags(AATags) {}
|
2016-05-05 02:54:54 +02:00
|
|
|
|
|
|
|
bool isInstInList(Instruction *I,
|
|
|
|
const SmallVectorImpl<Instruction *> &) const override {
|
|
|
|
Value *Ptr;
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(I))
|
|
|
|
Ptr = LI->getOperand(0);
|
|
|
|
else
|
|
|
|
Ptr = cast<StoreInst>(I)->getPointerOperand();
|
|
|
|
return PointerMustAliases.count(Ptr);
|
|
|
|
}
|
2011-01-15 01:12:35 +01:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
void doExtraRewritesBeforeFinalDeletion() const override {
|
|
|
|
// Insert stores after in the loop exit blocks. Each exit block gets a
|
|
|
|
// store of the live-out values that feed them. Since we've already told
|
|
|
|
// the SSA updater about the defs in the loop and the preheader
|
|
|
|
// definition, it is all set and we can start using it.
|
|
|
|
for (unsigned i = 0, e = LoopExitBlocks.size(); i != e; ++i) {
|
|
|
|
BasicBlock *ExitBlock = LoopExitBlocks[i];
|
|
|
|
Value *LiveInValue = SSA.GetValueInMiddleOfBlock(ExitBlock);
|
|
|
|
LiveInValue = maybeInsertLCSSAPHI(LiveInValue, ExitBlock);
|
|
|
|
Value *Ptr = maybeInsertLCSSAPHI(SomePtr, ExitBlock);
|
|
|
|
Instruction *InsertPos = LoopInsertPts[i];
|
|
|
|
StoreInst *NewSI = new StoreInst(LiveInValue, Ptr, InsertPos);
|
2017-02-14 02:38:31 +01:00
|
|
|
if (UnorderedAtomic)
|
|
|
|
NewSI->setOrdering(AtomicOrdering::Unordered);
|
2016-05-05 02:54:54 +02:00
|
|
|
NewSI->setAlignment(Alignment);
|
|
|
|
NewSI->setDebugLoc(DL);
|
|
|
|
if (AATags)
|
|
|
|
NewSI->setAAMetadata(AATags);
|
2011-01-15 01:12:35 +01:00
|
|
|
}
|
2016-05-05 02:54:54 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void replaceLoadWithValue(LoadInst *LI, Value *V) const override {
|
|
|
|
// Update alias analysis.
|
|
|
|
AST.copyValue(LI, V);
|
|
|
|
}
|
|
|
|
void instructionDeleted(Instruction *I) const override { AST.deleteValue(I); }
|
|
|
|
};
|
2017-10-26 23:00:15 +02:00
|
|
|
|
|
|
|
|
|
|
|
/// Return true iff we can prove that a caller of this function can not inspect
|
|
|
|
/// the contents of the provided object in a well defined program.
|
|
|
|
bool isKnownNonEscaping(Value *Object, const TargetLibraryInfo *TLI) {
|
|
|
|
if (isa<AllocaInst>(Object))
|
|
|
|
// Since the alloca goes out of scope, we know the caller can't retain a
|
|
|
|
// reference to it and be well defined. Thus, we don't need to check for
|
|
|
|
// capture.
|
|
|
|
return true;
|
|
|
|
|
|
|
|
// For all other objects we need to know that the caller can't possibly
|
|
|
|
// have gotten a reference to the object. There are two components of
|
|
|
|
// that:
|
|
|
|
// 1) Object can't be escaped by this function. This is what
|
|
|
|
// PointerMayBeCaptured checks.
|
|
|
|
// 2) Object can't have been captured at definition site. For this, we
|
|
|
|
// need to know the return value is noalias. At the moment, we use a
|
|
|
|
// weaker condition and handle only AllocLikeFunctions (which are
|
|
|
|
// known to be noalias). TODO
|
|
|
|
return isAllocLikeFn(Object, TLI) &&
|
|
|
|
!PointerMayBeCaptured(Object, true, true);
|
|
|
|
}
|
|
|
|
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
} // namespace
|
2011-01-15 01:12:35 +01:00
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// Try to promote memory values to scalars by sinking stores out of the
|
|
|
|
/// loop and moving loads to before the loop. We do this by looping over
|
|
|
|
/// the stores in the loop, looking for stores to Must pointers which are
|
|
|
|
/// loop invariant.
|
2003-02-24 04:52:32 +01:00
|
|
|
///
|
2016-05-05 02:54:54 +02:00
|
|
|
bool llvm::promoteLoopAccessesToScalars(
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
const SmallSetVector<Value *, 8> &PointerMustAliases,
|
|
|
|
SmallVectorImpl<BasicBlock *> &ExitBlocks,
|
2016-05-05 02:54:54 +02:00
|
|
|
SmallVectorImpl<Instruction *> &InsertPts, PredIteratorCache &PIC,
|
|
|
|
LoopInfo *LI, DominatorTree *DT, const TargetLibraryInfo *TLI,
|
2017-01-11 05:39:35 +01:00
|
|
|
Loop *CurLoop, AliasSetTracker *CurAST, LoopSafetyInfo *SafetyInfo,
|
|
|
|
OptimizationRemarkEmitter *ORE) {
|
2015-02-22 19:35:32 +01:00
|
|
|
// Verify inputs.
|
2016-05-05 02:54:54 +02:00
|
|
|
assert(LI != nullptr && DT != nullptr && CurLoop != nullptr &&
|
|
|
|
CurAST != nullptr && SafetyInfo != nullptr &&
|
2015-02-22 19:35:32 +01:00
|
|
|
"Unexpected Input to promoteLoopAccessesToScalars");
|
2016-01-14 00:01:57 +01:00
|
|
|
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
Value *SomePtr = *PointerMustAliases.begin();
|
2016-05-05 02:54:54 +02:00
|
|
|
BasicBlock *Preheader = CurLoop->getLoopPreheader();
|
2005-04-22 01:48:37 +02:00
|
|
|
|
2018-03-13 20:38:45 +01:00
|
|
|
// It is not safe to promote a load/store from the loop if the load/store is
|
2010-08-29 08:43:52 +02:00
|
|
|
// conditional. For example, turning:
|
2003-02-24 04:52:32 +01:00
|
|
|
//
|
2010-08-29 08:43:52 +02:00
|
|
|
// for () { if (c) *P += 1; }
|
2003-02-24 04:52:32 +01:00
|
|
|
//
|
2010-08-29 08:43:52 +02:00
|
|
|
// into:
|
|
|
|
//
|
|
|
|
// tmp = *P; for () { if (c) tmp +=1; } *P = tmp;
|
|
|
|
//
|
|
|
|
// is not safe, because *P may only be valid to access if 'c' is true.
|
2011-07-06 21:20:02 +02:00
|
|
|
//
|
2016-03-09 23:59:30 +01:00
|
|
|
// The safety property divides into two parts:
|
2016-12-30 01:39:00 +01:00
|
|
|
// p1) The memory may not be dereferenceable on entry to the loop. In this
|
2016-03-09 23:59:30 +01:00
|
|
|
// case, we can't insert the required load in the preheader.
|
2016-12-30 01:39:00 +01:00
|
|
|
// p2) The memory model does not allow us to insert a store along any dynamic
|
2016-03-09 23:59:30 +01:00
|
|
|
// path which did not originally have one.
|
|
|
|
//
|
2016-12-30 01:39:00 +01:00
|
|
|
// If at least one store is guaranteed to execute, both properties are
|
|
|
|
// satisfied, and promotion is legal.
|
2017-01-05 21:42:06 +01:00
|
|
|
//
|
2016-12-30 01:39:00 +01:00
|
|
|
// This, however, is not a necessary condition. Even if no store/load is
|
2017-01-05 21:42:06 +01:00
|
|
|
// guaranteed to execute, we can still establish these properties.
|
|
|
|
// We can establish (p1) by proving that hoisting the load into the preheader
|
|
|
|
// is safe (i.e. proving dereferenceability on all paths through the loop). We
|
2016-12-30 01:39:00 +01:00
|
|
|
// can use any access within the alias set to prove dereferenceability,
|
2016-03-09 23:59:30 +01:00
|
|
|
// since they're all must alias.
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
//
|
|
|
|
// There are two ways establish (p2):
|
2017-01-05 21:42:06 +01:00
|
|
|
// a) Prove the location is thread-local. In this case the memory model
|
2016-12-30 01:39:00 +01:00
|
|
|
// requirement does not apply, and stores are safe to insert.
|
2017-01-05 21:42:06 +01:00
|
|
|
// b) Prove a store dominates every exit block. In this case, if an exit
|
|
|
|
// blocks is reached, the original dynamic path would have taken us through
|
|
|
|
// the store, so inserting a store into the exit block is safe. Note that this
|
|
|
|
// is different from the store being guaranteed to execute. For instance,
|
|
|
|
// if an exception is thrown on the first iteration of the loop, the original
|
|
|
|
// store is never executed, but the exit blocks are not executed either.
|
2016-12-30 01:39:00 +01:00
|
|
|
|
|
|
|
bool DereferenceableInPH = false;
|
|
|
|
bool SafeToInsertStore = false;
|
2016-03-09 23:59:30 +01:00
|
|
|
|
2016-05-05 02:54:54 +02:00
|
|
|
SmallVector<Instruction *, 64> LoopUses;
|
2010-08-29 08:43:52 +02:00
|
|
|
|
2011-07-06 21:19:55 +02:00
|
|
|
// We start with an alignment of one and try to find instructions that allow
|
|
|
|
// us to prove better alignment.
|
|
|
|
unsigned Alignment = 1;
|
2017-02-14 02:38:31 +01:00
|
|
|
// Keep track of which types of access we see
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
bool SawUnorderedAtomic = false;
|
2017-02-14 02:38:31 +01:00
|
|
|
bool SawNotAtomic = false;
|
2014-07-24 14:16:19 +02:00
|
|
|
AAMDNodes AATags;
|
2016-05-05 02:54:54 +02:00
|
|
|
|
2016-03-09 23:59:30 +01:00
|
|
|
const DataLayout &MDL = Preheader->getModule()->getDataLayout();
|
|
|
|
|
2017-10-26 23:00:15 +02:00
|
|
|
bool IsKnownThreadLocalObject = false;
|
2017-04-24 20:25:07 +02:00
|
|
|
if (SafetyInfo->MayThrow) {
|
2016-06-06 00:13:52 +02:00
|
|
|
// If a loop can throw, we have to insert a store along each unwind edge.
|
|
|
|
// That said, we can't actually make the unwind edge explicit. Therefore,
|
2017-10-26 23:00:15 +02:00
|
|
|
// we have to prove that the store is dead along the unwind edge. We do
|
|
|
|
// this by proving that the caller can't have a reference to the object
|
|
|
|
// after return and thus can't possibly load from the object.
|
2017-01-19 20:31:40 +01:00
|
|
|
Value *Object = GetUnderlyingObject(SomePtr, MDL);
|
2017-10-26 23:00:15 +02:00
|
|
|
if (!isKnownNonEscaping(Object, TLI))
|
|
|
|
return false;
|
|
|
|
// Subtlety: Alloca's aren't visible to callers, but *are* potentially
|
|
|
|
// visible to other threads if captured and used during their lifetimes.
|
|
|
|
IsKnownThreadLocalObject = !isa<AllocaInst>(Object);
|
2016-06-06 00:13:52 +02:00
|
|
|
}
|
|
|
|
|
2010-08-29 08:43:52 +02:00
|
|
|
// Check that all of the pointers in the alias set have the same type. We
|
|
|
|
// cannot (yet) promote a memory location that is loaded and stored in
|
2014-07-24 14:16:19 +02:00
|
|
|
// different sizes. While we are at it, collect alignment and AA info.
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
for (Value *ASIV : PointerMustAliases) {
|
2008-05-22 02:53:38 +02:00
|
|
|
// Check that all of the pointers in the alias set have the same type. We
|
|
|
|
// cannot (yet) promote a memory location that is loaded and stored in
|
|
|
|
// different sizes.
|
2010-08-29 08:43:52 +02:00
|
|
|
if (SomePtr->getType() != ASIV->getType())
|
2016-12-30 01:43:22 +01:00
|
|
|
return false;
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2014-03-09 04:16:01 +01:00
|
|
|
for (User *U : ASIV->users()) {
|
2010-08-29 08:43:52 +02:00
|
|
|
// Ignore instructions that are outside the loop.
|
2014-03-09 04:16:01 +01:00
|
|
|
Instruction *UI = dyn_cast<Instruction>(U);
|
|
|
|
if (!UI || !CurLoop->contains(UI))
|
2008-05-22 02:53:38 +02:00
|
|
|
continue;
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2010-08-29 08:43:52 +02:00
|
|
|
// If there is an non-load/store instruction in the loop, we can't promote
|
|
|
|
// it.
|
2017-01-11 05:39:49 +01:00
|
|
|
if (LoadInst *Load = dyn_cast<LoadInst>(UI)) {
|
2016-01-08 23:05:03 +01:00
|
|
|
assert(!Load->isVolatile() && "AST broken");
|
2017-02-14 02:38:31 +01:00
|
|
|
if (!Load->isUnordered())
|
2016-12-30 01:43:22 +01:00
|
|
|
return false;
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
|
2017-02-14 02:38:31 +01:00
|
|
|
SawUnorderedAtomic |= Load->isAtomic();
|
|
|
|
SawNotAtomic |= !Load->isAtomic();
|
2016-03-09 23:59:30 +01:00
|
|
|
|
2016-12-30 01:39:00 +01:00
|
|
|
if (!DereferenceableInPH)
|
|
|
|
DereferenceableInPH = isSafeToExecuteUnconditionally(
|
2017-01-11 05:39:49 +01:00
|
|
|
*Load, DT, CurLoop, SafetyInfo, ORE, Preheader->getTerminator());
|
2016-01-08 23:05:03 +01:00
|
|
|
} else if (const StoreInst *Store = dyn_cast<StoreInst>(UI)) {
|
2010-12-19 06:57:25 +01:00
|
|
|
// Stores *of* the pointer are not interesting, only stores *to* the
|
|
|
|
// pointer.
|
2014-03-09 04:16:01 +01:00
|
|
|
if (UI->getOperand(1) != ASIV)
|
2010-12-19 06:57:25 +01:00
|
|
|
continue;
|
2016-01-08 23:05:03 +01:00
|
|
|
assert(!Store->isVolatile() && "AST broken");
|
2017-02-14 02:38:31 +01:00
|
|
|
if (!Store->isUnordered())
|
2016-12-30 01:43:22 +01:00
|
|
|
return false;
|
2011-07-06 21:19:55 +02:00
|
|
|
|
2017-02-14 02:38:31 +01:00
|
|
|
SawUnorderedAtomic |= Store->isAtomic();
|
|
|
|
SawNotAtomic |= !Store->isAtomic();
|
|
|
|
|
2016-12-30 01:39:00 +01:00
|
|
|
// If the store is guaranteed to execute, both properties are satisfied.
|
|
|
|
// We may want to check if a store is guaranteed to execute even if we
|
|
|
|
// already know that promotion is safe, since it may have higher
|
|
|
|
// alignment than any other guaranteed stores, in which case we can
|
|
|
|
// raise the alignment on the promoted store.
|
2016-01-08 23:05:03 +01:00
|
|
|
unsigned InstAlignment = Store->getAlignment();
|
2016-12-30 01:39:00 +01:00
|
|
|
if (!InstAlignment)
|
|
|
|
InstAlignment =
|
|
|
|
MDL.getABITypeAlignment(Store->getValueOperand()->getType());
|
|
|
|
|
|
|
|
if (!DereferenceableInPH || !SafeToInsertStore ||
|
|
|
|
(InstAlignment > Alignment)) {
|
2015-02-22 19:35:32 +01:00
|
|
|
if (isGuaranteedToExecute(*UI, DT, CurLoop, SafetyInfo)) {
|
2016-12-30 01:39:00 +01:00
|
|
|
DereferenceableInPH = true;
|
|
|
|
SafeToInsertStore = true;
|
|
|
|
Alignment = std::max(Alignment, InstAlignment);
|
2011-07-20 23:37:47 +02:00
|
|
|
}
|
2016-06-24 14:38:45 +02:00
|
|
|
}
|
2016-03-09 23:59:30 +01:00
|
|
|
|
2017-01-05 21:42:06 +01:00
|
|
|
// If a store dominates all exit blocks, it is safe to sink.
|
|
|
|
// As explained above, if an exit block was executed, a dominating
|
2018-03-31 00:22:31 +02:00
|
|
|
// store must have been executed at least once, so we are not
|
2017-01-05 21:42:06 +01:00
|
|
|
// introducing stores on paths that did not have them.
|
|
|
|
// Note that this only looks at explicit exit blocks. If we ever
|
|
|
|
// start sinking stores into unwind edges (see above), this will break.
|
|
|
|
if (!SafeToInsertStore)
|
|
|
|
SafeToInsertStore = llvm::all_of(ExitBlocks, [&](BasicBlock *Exit) {
|
|
|
|
return DT->dominates(Store->getParent(), Exit);
|
|
|
|
});
|
|
|
|
|
2016-12-30 01:39:00 +01:00
|
|
|
// If the store is not guaranteed to execute, we may still get
|
|
|
|
// deref info through it.
|
|
|
|
if (!DereferenceableInPH) {
|
|
|
|
DereferenceableInPH = isDereferenceableAndAlignedPointer(
|
2016-05-05 02:54:54 +02:00
|
|
|
Store->getPointerOperand(), Store->getAlignment(), MDL,
|
2016-07-03 01:47:27 +02:00
|
|
|
Preheader->getTerminator(), DT);
|
2016-03-09 23:59:30 +01:00
|
|
|
}
|
2011-07-20 23:37:47 +02:00
|
|
|
} else
|
2016-12-30 01:43:22 +01:00
|
|
|
return false; // Not a load or store.
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2014-07-24 14:16:19 +02:00
|
|
|
// Merge the AA tags.
|
2012-12-31 09:37:17 +01:00
|
|
|
if (LoopUses.empty()) {
|
2014-07-24 14:16:19 +02:00
|
|
|
// On the first load/store, just take its AA tags.
|
|
|
|
UI->getAAMetadata(AATags);
|
|
|
|
} else if (AATags) {
|
|
|
|
UI->getAAMetadata(AATags, /* Merge = */ true);
|
2012-12-31 09:37:17 +01:00
|
|
|
}
|
2014-03-09 04:16:01 +01:00
|
|
|
|
|
|
|
LoopUses.push_back(UI);
|
2008-05-22 02:53:38 +02:00
|
|
|
}
|
2010-08-29 08:43:52 +02:00
|
|
|
}
|
2011-07-06 21:19:55 +02:00
|
|
|
|
2017-02-14 02:38:31 +01:00
|
|
|
// If we found both an unordered atomic instruction and a non-atomic memory
|
|
|
|
// access, bail. We can't blindly promote non-atomic to atomic since we
|
|
|
|
// might not be able to lower the result. We can't downgrade since that
|
|
|
|
// would violate memory model. Also, align 0 is an error for atomics.
|
|
|
|
if (SawUnorderedAtomic && SawNotAtomic)
|
|
|
|
return false;
|
2016-12-30 01:39:00 +01:00
|
|
|
|
|
|
|
// If we couldn't prove we can hoist the load, bail.
|
|
|
|
if (!DereferenceableInPH)
|
2016-12-30 01:43:22 +01:00
|
|
|
return false;
|
2016-12-30 01:39:00 +01:00
|
|
|
|
|
|
|
// We know we can hoist the load, but don't have a guaranteed store.
|
|
|
|
// Check whether the location is thread-local. If it is, then we can insert
|
|
|
|
// stores along paths which originally didn't have them without violating the
|
|
|
|
// memory model.
|
|
|
|
if (!SafeToInsertStore) {
|
2017-10-26 23:00:15 +02:00
|
|
|
if (IsKnownThreadLocalObject)
|
2017-01-19 20:31:40 +01:00
|
|
|
SafeToInsertStore = true;
|
|
|
|
else {
|
|
|
|
Value *Object = GetUnderlyingObject(SomePtr, MDL);
|
|
|
|
SafeToInsertStore =
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
(isAllocLikeFn(Object, TLI) || isa<AllocaInst>(Object)) &&
|
|
|
|
!PointerMayBeCaptured(Object, true, true);
|
2017-01-19 20:31:40 +01:00
|
|
|
}
|
2016-03-09 23:59:30 +01:00
|
|
|
}
|
2016-12-30 00:11:19 +01:00
|
|
|
|
2016-12-30 01:39:00 +01:00
|
|
|
// If we've still failed to prove we can sink the store, give up.
|
|
|
|
if (!SafeToInsertStore)
|
2016-12-30 01:43:22 +01:00
|
|
|
return false;
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2010-08-29 08:43:52 +02:00
|
|
|
// Otherwise, this is safe to promote, lets do it!
|
2018-05-14 14:53:11 +02:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM: Promoting value stored to in loop: " << *SomePtr
|
|
|
|
<< '\n');
|
2017-10-11 19:12:59 +02:00
|
|
|
ORE->emit([&]() {
|
|
|
|
return OptimizationRemark(DEBUG_TYPE, "PromoteLoopAccessesToScalar",
|
|
|
|
LoopUses[0])
|
|
|
|
<< "Moving accesses to memory location out of the loop";
|
|
|
|
});
|
2010-08-29 08:43:52 +02:00
|
|
|
++NumPromoted;
|
2007-10-01 20:12:58 +02:00
|
|
|
|
2011-05-27 22:31:51 +02:00
|
|
|
// Grab a debug location for the inserted loads/stores; given that the
|
|
|
|
// inserted loads/stores have little relation to the original loads/stores,
|
|
|
|
// this code just arbitrarily picks a location from one, since any debug
|
|
|
|
// location is better than none.
|
|
|
|
DebugLoc DL = LoopUses[0]->getDebugLoc();
|
|
|
|
|
2010-08-29 08:43:52 +02:00
|
|
|
// We use the SSAUpdater interface to insert phi nodes as required.
|
2016-05-05 02:54:54 +02:00
|
|
|
SmallVector<PHINode *, 16> NewPHIs;
|
2010-08-29 08:43:52 +02:00
|
|
|
SSAUpdater SSA(&NewPHIs);
|
2016-05-05 02:54:54 +02:00
|
|
|
LoopPromoter Promoter(SomePtr, LoopUses, SSA, PointerMustAliases, ExitBlocks,
|
2017-02-14 02:38:31 +01:00
|
|
|
InsertPts, PIC, *CurAST, *LI, DL, Alignment,
|
|
|
|
SawUnorderedAtomic, AATags);
|
2011-07-06 21:20:02 +02:00
|
|
|
|
2011-01-15 01:12:35 +01:00
|
|
|
// Set up the preheader to have a definition of the value. It is the live-out
|
|
|
|
// value from the preheader that uses in the loop will use.
|
2016-05-05 02:54:54 +02:00
|
|
|
LoadInst *PreheaderLoad = new LoadInst(
|
|
|
|
SomePtr, SomePtr->getName() + ".promoted", Preheader->getTerminator());
|
2017-02-14 02:38:31 +01:00
|
|
|
if (SawUnorderedAtomic)
|
|
|
|
PreheaderLoad->setOrdering(AtomicOrdering::Unordered);
|
2011-07-06 21:19:55 +02:00
|
|
|
PreheaderLoad->setAlignment(Alignment);
|
2011-05-27 22:31:51 +02:00
|
|
|
PreheaderLoad->setDebugLoc(DL);
|
2016-05-05 02:54:54 +02:00
|
|
|
if (AATags)
|
|
|
|
PreheaderLoad->setAAMetadata(AATags);
|
2010-08-29 08:43:52 +02:00
|
|
|
SSA.AddAvailableValue(Preheader, PreheaderLoad);
|
|
|
|
|
2011-01-15 01:12:35 +01:00
|
|
|
// Rewrite all the loads in the loop and remember all the definitions from
|
|
|
|
// stores in the loop.
|
|
|
|
Promoter.run(LoopUses);
|
2011-04-07 03:35:06 +02:00
|
|
|
|
|
|
|
// If the SSAUpdater didn't use the load in the preheader, just zap it now.
|
|
|
|
if (PreheaderLoad->use_empty())
|
|
|
|
PreheaderLoad->eraseFromParent();
|
2007-07-31 10:01:41 +02:00
|
|
|
|
2016-12-30 01:43:22 +01:00
|
|
|
return true;
|
2015-02-22 19:35:32 +01:00
|
|
|
}
|
2010-08-29 08:43:52 +02:00
|
|
|
|
2016-02-15 15:48:50 +01:00
|
|
|
/// Returns an owning pointer to an alias set which incorporates aliasing info
|
2016-02-27 05:34:07 +01:00
|
|
|
/// from L and all subloops of L.
|
2016-08-12 00:34:00 +02:00
|
|
|
/// FIXME: In new pass manager, there is no helper function to handle loop
|
|
|
|
/// analysis such as cloneBasicBlockAnalysis, so the AST needs to be recomputed
|
2016-07-13 00:37:48 +02:00
|
|
|
/// from scratch for every loop. Hook up with the helper functions when
|
|
|
|
/// available in the new pass manager to avoid redundant computation.
|
|
|
|
AliasSetTracker *
|
|
|
|
LoopInvariantCodeMotion::collectAliasInfoForLoop(Loop *L, LoopInfo *LI,
|
|
|
|
AliasAnalysis *AA) {
|
2016-02-15 15:48:50 +01:00
|
|
|
AliasSetTracker *CurAST = nullptr;
|
2016-02-27 05:34:07 +01:00
|
|
|
SmallVector<Loop *, 4> RecomputeLoops;
|
2016-02-15 15:48:50 +01:00
|
|
|
for (Loop *InnerL : L->getSubLoops()) {
|
2016-02-27 05:34:07 +01:00
|
|
|
auto MapI = LoopToAliasSetMap.find(InnerL);
|
|
|
|
// If the AST for this inner loop is missing it may have been merged into
|
|
|
|
// some other loop's AST and then that loop unrolled, and so we need to
|
|
|
|
// recompute it.
|
|
|
|
if (MapI == LoopToAliasSetMap.end()) {
|
|
|
|
RecomputeLoops.push_back(InnerL);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
AliasSetTracker *InnerAST = MapI->second;
|
2016-02-15 15:48:50 +01:00
|
|
|
|
|
|
|
if (CurAST != nullptr) {
|
|
|
|
// What if InnerLoop was modified by other passes ?
|
|
|
|
CurAST->add(*InnerAST);
|
|
|
|
|
|
|
|
// Once we've incorporated the inner loop's AST into ours, we don't need
|
|
|
|
// the subloop's anymore.
|
|
|
|
delete InnerAST;
|
|
|
|
} else {
|
|
|
|
CurAST = InnerAST;
|
|
|
|
}
|
2016-02-27 05:34:07 +01:00
|
|
|
LoopToAliasSetMap.erase(MapI);
|
2016-02-15 15:48:50 +01:00
|
|
|
}
|
|
|
|
if (CurAST == nullptr)
|
|
|
|
CurAST = new AliasSetTracker(*AA);
|
2016-02-27 05:34:07 +01:00
|
|
|
|
|
|
|
auto mergeLoop = [&](Loop *L) {
|
|
|
|
// Loop over the body of this loop, looking for calls, invokes, and stores.
|
|
|
|
for (BasicBlock *BB : L->blocks())
|
Make promoteLoopAccessesToScalars independent of AliasSet [NFC]
Summary:
The current promoteLoopAccessesToScalars method receives an AliasSet, but
the information used is in fact a list of Value*, known to must alias.
Create the list ahead of time to make this method independent of the AliasSet class.
While there is no functionality change, this adds overhead for creating
a set of Value*, when promotion would normally exit earlier.
This is meant to be as a first refactoring step in order to start replacing
AliasSetTracker with MemorySSA.
And while the end goal is to redesign LICM, the first few steps will focus on
adding MemorySSA as an alternative to the AliasSetTracker using most of the
existing functionality.
Reviewers: mkuper, danielcdh, dberlin
Subscribers: sanjoy, chandlerc, gberry, davide, llvm-commits
Differential Revision: https://reviews.llvm.org/D35439
llvm-svn: 313075
2017-09-12 23:18:44 +02:00
|
|
|
CurAST->add(*BB); // Incorporate the specified basic block
|
2016-02-27 05:34:07 +01:00
|
|
|
};
|
|
|
|
|
|
|
|
// Add everything from the sub loops that are no longer directly available.
|
|
|
|
for (Loop *InnerL : RecomputeLoops)
|
|
|
|
mergeLoop(InnerL);
|
|
|
|
|
|
|
|
// And merge in this loop.
|
|
|
|
mergeLoop(L);
|
|
|
|
|
2016-02-15 15:48:50 +01:00
|
|
|
return CurAST;
|
|
|
|
}
|
|
|
|
|
2015-08-13 13:18:35 +02:00
|
|
|
/// Simple analysis hook. Clone alias set info.
|
2015-02-22 19:35:32 +01:00
|
|
|
///
|
2016-07-13 00:37:48 +02:00
|
|
|
void LegacyLICMPass::cloneBasicBlockAnalysis(BasicBlock *From, BasicBlock *To,
|
|
|
|
Loop *L) {
|
|
|
|
AliasSetTracker *AST = LICM.getLoopToAliasSetMap().lookup(L);
|
2007-07-31 10:01:41 +02:00
|
|
|
if (!AST)
|
|
|
|
return;
|
|
|
|
|
|
|
|
AST->copyValue(From, To);
|
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// Simple Analysis hook. Delete value V from alias set
|
|
|
|
///
|
2016-07-13 00:37:48 +02:00
|
|
|
void LegacyLICMPass::deleteAnalysisValue(Value *V, Loop *L) {
|
|
|
|
AliasSetTracker *AST = LICM.getLoopToAliasSetMap().lookup(L);
|
2007-07-31 10:01:41 +02:00
|
|
|
if (!AST)
|
|
|
|
return;
|
|
|
|
|
|
|
|
AST->deleteValue(V);
|
|
|
|
}
|
2014-09-24 18:48:31 +02:00
|
|
|
|
|
|
|
/// Simple Analysis hook. Delete value L from alias set map.
|
2015-02-22 19:35:32 +01:00
|
|
|
///
|
2016-07-13 00:37:48 +02:00
|
|
|
void LegacyLICMPass::deleteAnalysisLoop(Loop *L) {
|
|
|
|
AliasSetTracker *AST = LICM.getLoopToAliasSetMap().lookup(L);
|
2014-09-24 18:48:31 +02:00
|
|
|
if (!AST)
|
|
|
|
return;
|
|
|
|
|
|
|
|
delete AST;
|
2016-07-13 00:37:48 +02:00
|
|
|
LICM.getLoopToAliasSetMap().erase(L);
|
2014-09-24 18:48:31 +02:00
|
|
|
}
|
2015-02-22 19:35:32 +01:00
|
|
|
|
|
|
|
/// Return true if the body of this loop may store into the memory
|
|
|
|
/// location pointed to by V.
|
|
|
|
///
|
|
|
|
static bool pointerInvalidatedByLoop(Value *V, uint64_t Size,
|
2016-05-05 02:54:54 +02:00
|
|
|
const AAMDNodes &AAInfo,
|
2015-02-22 19:35:32 +01:00
|
|
|
AliasSetTracker *CurAST) {
|
|
|
|
// Check to see if any of the basic blocks in CurLoop invalidate *V.
|
|
|
|
return CurAST->getAliasSetForPointer(V, Size, AAInfo).isMod();
|
|
|
|
}
|
|
|
|
|
|
|
|
/// Little predicate that returns true if the specified basic block is in
|
|
|
|
/// a subloop of the current one, not the current one itself.
|
|
|
|
///
|
|
|
|
static bool inSubLoop(BasicBlock *BB, Loop *CurLoop, LoopInfo *LI) {
|
|
|
|
assert(CurLoop->contains(BB) && "Only valid if BB is IN the loop");
|
|
|
|
return LI->getLoopFor(BB) != CurLoop;
|
|
|
|
}
|