2002-05-11 00:44:58 +02:00
|
|
|
//===-- LICM.cpp - Loop Invariant Code Motion Pass ------------------------===//
|
2005-04-22 01:48:37 +02:00
|
|
|
//
|
2019-01-19 09:50:56 +01:00
|
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
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"
|
2018-11-29 18:10:00 +01:00
|
|
|
#include "llvm/ADT/SetOperations.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"
|
2018-08-30 05:39:16 +02:00
|
|
|
#include "llvm/Analysis/GuardUtils.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"
|
2018-11-29 18:10:00 +01:00
|
|
|
#include "llvm/Analysis/LoopIterator.h"
|
2010-09-01 01:00:16 +02:00
|
|
|
#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"
|
2019-01-10 20:29:04 +01:00
|
|
|
#include "llvm/Analysis/MemorySSAUpdater.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"
|
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"
|
2019-04-20 00:36:40 +02:00
|
|
|
#include "llvm/IR/DebugInfoMetadata.h"
|
2013-01-02 12:36:10 +01:00
|
|
|
#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"
|
2018-08-21 10:11:31 +02:00
|
|
|
#include "llvm/IR/PatternMatch.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"
|
2019-01-10 20:29:04 +01:00
|
|
|
#include "llvm/Transforms/Utils/Local.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"
|
|
|
|
|
2018-11-29 18:10:00 +01:00
|
|
|
STATISTIC(NumCreatedBlocks, "Number of blocks created");
|
|
|
|
STATISTIC(NumClonedBranches, "Number of branches cloned");
|
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
|
|
|
|
2018-11-29 18:10:00 +01:00
|
|
|
static cl::opt<bool> ControlFlowHoisting(
|
|
|
|
"licm-control-flow-hoisting", cl::Hidden, cl::init(false),
|
|
|
|
cl::desc("Enable control flow (and PHI) hoisting in LICM"));
|
|
|
|
|
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)"));
|
|
|
|
|
2018-08-17 15:44:00 +02:00
|
|
|
// Default value of zero implies we use the regular alias set tracker mechanism
|
|
|
|
// instead of the cross product using AA to identify aliasing of the memory
|
|
|
|
// location we are interested in.
|
|
|
|
static cl::opt<int>
|
|
|
|
LICMN2Theshold("licm-n2-threshold", cl::Hidden, cl::init(0),
|
|
|
|
cl::desc("How many instruction to cross product using AA"));
|
|
|
|
|
2019-02-13 00:05:40 +01:00
|
|
|
// Experimental option to allow imprecision in LICM in pathological cases, in
|
|
|
|
// exchange for faster compile. This is to be removed if MemorySSA starts to
|
|
|
|
// address the same issue. This flag applies only when LICM uses MemorySSA
|
|
|
|
// instead on AliasSetTracker. LICM calls MemorySSAWalker's
|
|
|
|
// getClobberingMemoryAccess, up to the value of the Cap, getting perfect
|
|
|
|
// accuracy. Afterwards, LICM will call into MemorySSA's getDefiningAccess,
|
|
|
|
// which may not be precise, since optimizeUses is capped. The result is
|
|
|
|
// correct, but we may not get as "far up" as possible to get which access is
|
|
|
|
// clobbering the one queried.
|
2019-04-19 19:46:50 +02:00
|
|
|
cl::opt<unsigned> llvm::SetLicmMssaOptCap(
|
2019-02-13 00:05:40 +01:00
|
|
|
"licm-mssa-optimization-cap", cl::init(100), cl::Hidden,
|
|
|
|
cl::desc("Enable imprecision in LICM in pathological cases, in exchange "
|
|
|
|
"for faster compile. Caps the MemorySSA clobbering calls."));
|
2019-01-10 20:29:04 +01:00
|
|
|
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
// Experimentally, memory promotion carries less importance than sinking and
|
|
|
|
// hoisting. Limit when we do promotion when using MemorySSA, in order to save
|
|
|
|
// compile time.
|
2019-04-19 19:46:50 +02:00
|
|
|
cl::opt<unsigned> llvm::SetLicmMssaNoAccForPromotionCap(
|
|
|
|
"licm-mssa-max-acc-promotion", cl::init(250), cl::Hidden,
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
cl::desc("[LICM & MemorySSA] When MSSA in LICM is disabled, this has no "
|
|
|
|
"effect. When MSSA in LICM is enabled, then this is the maximum "
|
|
|
|
"number of accesses allowed to be present in a loop in order to "
|
|
|
|
"enable memory promotion."));
|
|
|
|
|
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);
|
2018-08-15 04:49:12 +02:00
|
|
|
static void hoist(Instruction &I, const DominatorTree *DT, const Loop *CurLoop,
|
2018-11-29 18:10:00 +01:00
|
|
|
BasicBlock *Dest, ICFLoopSafetyInfo *SafetyInfo,
|
2019-01-10 20:29:04 +01:00
|
|
|
MemorySSAUpdater *MSSAU, OptimizationRemarkEmitter *ORE);
|
2017-11-03 17:24:53 +01:00
|
|
|
static bool sink(Instruction &I, LoopInfo *LI, DominatorTree *DT,
|
2019-08-14 06:50:33 +02:00
|
|
|
const Loop *CurLoop, ICFLoopSafetyInfo *SafetyInfo,
|
|
|
|
MemorySSAUpdater *MSSAU, OptimizationRemarkEmitter *ORE);
|
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);
|
2018-08-17 15:44:00 +02:00
|
|
|
static bool pointerInvalidatedByLoop(MemoryLocation MemLoc,
|
|
|
|
AliasSetTracker *CurAST, Loop *CurLoop,
|
|
|
|
AliasAnalysis *AA);
|
2019-01-10 20:29:04 +01:00
|
|
|
static bool pointerInvalidatedByLoopWithMSSA(MemorySSA *MSSA, MemoryUse *MU,
|
2019-02-13 00:05:40 +01:00
|
|
|
Loop *CurLoop,
|
2019-04-19 19:46:50 +02:00
|
|
|
SinkAndHoistLICMFlags &Flags);
|
2019-01-10 20:29:04 +01:00
|
|
|
static Instruction *CloneInstructionInExitBlock(
|
|
|
|
Instruction &I, BasicBlock &ExitBlock, PHINode &PN, const LoopInfo *LI,
|
|
|
|
const LoopSafetyInfo *SafetyInfo, MemorySSAUpdater *MSSAU);
|
2015-02-22 19:35:32 +01:00
|
|
|
|
2018-11-06 03:44:49 +01:00
|
|
|
static void eraseInstruction(Instruction &I, ICFLoopSafetyInfo &SafetyInfo,
|
2019-01-10 20:29:04 +01:00
|
|
|
AliasSetTracker *AST, MemorySSAUpdater *MSSAU);
|
2018-11-02 01:21:45 +01:00
|
|
|
|
2018-11-09 06:39:04 +01:00
|
|
|
static void moveInstructionBefore(Instruction &I, Instruction &Dest,
|
2019-01-24 20:57:30 +01:00
|
|
|
ICFLoopSafetyInfo &SafetyInfo,
|
|
|
|
MemorySSAUpdater *MSSAU);
|
2018-11-09 06:39:04 +01:00
|
|
|
|
2008-05-13 02:00:25 +02:00
|
|
|
namespace {
|
2016-07-13 00:37:48 +02:00
|
|
|
struct LoopInvariantCodeMotion {
|
2018-08-21 22:30:14 +02:00
|
|
|
using ASTrackerMapTy = DenseMap<Loop *, std::unique_ptr<AliasSetTracker>>;
|
2016-07-13 00:37:48 +02:00
|
|
|
bool runOnLoop(Loop *L, AliasAnalysis *AA, LoopInfo *LI, DominatorTree *DT,
|
2019-08-14 06:50:33 +02: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
|
|
|
|
2018-08-21 22:30:14 +02:00
|
|
|
ASTrackerMapTy &getLoopToAliasSetMap() { return LoopToAliasSetMap; }
|
2019-04-19 19:46:50 +02:00
|
|
|
LoopInvariantCodeMotion(unsigned LicmMssaOptCap,
|
|
|
|
unsigned LicmMssaNoAccForPromotionCap)
|
|
|
|
: LicmMssaOptCap(LicmMssaOptCap),
|
|
|
|
LicmMssaNoAccForPromotionCap(LicmMssaNoAccForPromotionCap) {}
|
2016-07-13 00:37:48 +02:00
|
|
|
|
|
|
|
private:
|
2018-08-21 22:30:14 +02:00
|
|
|
ASTrackerMapTy LoopToAliasSetMap;
|
2019-04-19 19:46:50 +02:00
|
|
|
unsigned LicmMssaOptCap;
|
|
|
|
unsigned LicmMssaNoAccForPromotionCap;
|
2016-07-13 00:37:48 +02:00
|
|
|
|
2018-08-21 22:30:14 +02:00
|
|
|
std::unique_ptr<AliasSetTracker>
|
|
|
|
collectAliasInfoForLoop(Loop *L, LoopInfo *LI, AliasAnalysis *AA);
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
std::unique_ptr<AliasSetTracker>
|
|
|
|
collectAliasInfoForLoopWithMSSA(Loop *L, AliasAnalysis *AA,
|
|
|
|
MemorySSAUpdater *MSSAU);
|
2016-07-13 00:37:48 +02:00
|
|
|
};
|
|
|
|
|
|
|
|
struct LegacyLICMPass : public LoopPass {
|
2016-05-05 02:54:54 +02:00
|
|
|
static char ID; // Pass identification, replacement for typeid
|
2019-04-19 19:46:50 +02:00
|
|
|
LegacyLICMPass(
|
|
|
|
unsigned LicmMssaOptCap = SetLicmMssaOptCap,
|
|
|
|
unsigned LicmMssaNoAccForPromotionCap = SetLicmMssaNoAccForPromotionCap)
|
|
|
|
: LoopPass(ID), LICM(LicmMssaOptCap, LicmMssaNoAccForPromotionCap) {
|
2016-07-13 00:37:48 +02:00
|
|
|
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.
|
|
|
|
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 {
|
[LICM] Preserve DT and LoopInfo specifically
Summary:
In LICM, CFG could be changed in splitPredecessorsOfLoopExit(), which update
only DT and LoopInfo. Therefore, we should preserve only DT and LoopInfo specifically,
instead of all analyses that depend on the CFG (setPreservesCFG()).
This change should fix PR37323.
Reviewers: uabelho, davide, dberlin, Ka-Ka
Reviewed By: dberlin
Subscribers: mzolotukhin, bjope, mcrosier, llvm-commits
Differential Revision: https://reviews.llvm.org/D46775
llvm-svn: 333198
2018-05-24 17:58:34 +02:00
|
|
|
AU.addPreserved<DominatorTreeWrapperPass>();
|
|
|
|
AU.addPreserved<LoopInfoWrapperPass>();
|
2016-05-05 02:54:54 +02:00
|
|
|
AU.addRequired<TargetLibraryInfoWrapperPass>();
|
2019-01-10 20:29:04 +01:00
|
|
|
if (EnableMSSALoopDependency) {
|
2017-11-21 16:45:46 +01:00
|
|
|
AU.addRequired<MemorySSAWrapperPass>();
|
2019-01-10 20:29:04 +01:00
|
|
|
AU.addPreserved<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 {
|
2019-05-14 21:41:36 +02:00
|
|
|
auto &AliasSetMap = LICM.getLoopToAliasSetMap();
|
|
|
|
// All loops in the AliasSetMap should be cleaned up already. The only case
|
|
|
|
// where we fail to do so is if an outer loop gets deleted before LICM
|
|
|
|
// visits it.
|
|
|
|
assert(all_of(AliasSetMap,
|
|
|
|
[](LoopInvariantCodeMotion::ASTrackerMapTy::value_type &KV) {
|
|
|
|
return !KV.first->getParentLoop();
|
|
|
|
}) &&
|
2016-07-13 00:37:48 +02:00
|
|
|
"Didn't free loop alias sets");
|
2019-05-14 21:41:36 +02:00
|
|
|
AliasSetMap.clear();
|
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
|
|
|
|
2019-04-19 19:46:50 +02:00
|
|
|
LoopInvariantCodeMotion LICM(LicmMssaOptCap, LicmMssaNoAccForPromotionCap);
|
2019-08-14 06:50:33 +02:00
|
|
|
if (!LICM.runOnLoop(&L, &AR.AA, &AR.LI, &AR.DT, &AR.TLI, &AR.TTI, &AR.SE,
|
2017-12-15 21:33:24 +01:00
|
|
|
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();
|
[LICM] Preserve DT and LoopInfo specifically
Summary:
In LICM, CFG could be changed in splitPredecessorsOfLoopExit(), which update
only DT and LoopInfo. Therefore, we should preserve only DT and LoopInfo specifically,
instead of all analyses that depend on the CFG (setPreservesCFG()).
This change should fix PR37323.
Reviewers: uabelho, davide, dberlin, Ka-Ka
Reviewed By: dberlin
Subscribers: mzolotukhin, bjope, mcrosier, llvm-commits
Differential Revision: https://reviews.llvm.org/D46775
llvm-svn: 333198
2018-05-24 17:58:34 +02:00
|
|
|
|
|
|
|
PA.preserve<DominatorTreeAnalysis>();
|
|
|
|
PA.preserve<LoopAnalysis>();
|
2019-08-17 03:02:12 +02:00
|
|
|
if (AR.MSSA)
|
2019-06-11 20:27:49 +02:00
|
|
|
PA.preserve<MemorySSAAnalysis>();
|
[LICM] Preserve DT and LoopInfo specifically
Summary:
In LICM, CFG could be changed in splitPredecessorsOfLoopExit(), which update
only DT and LoopInfo. Therefore, we should preserve only DT and LoopInfo specifically,
instead of all analyses that depend on the CFG (setPreservesCFG()).
This change should fix PR37323.
Reviewers: uabelho, davide, dberlin, Ka-Ka
Reviewed By: dberlin
Subscribers: mzolotukhin, bjope, mcrosier, llvm-commits
Differential Revision: https://reviews.llvm.org/D46775
llvm-svn: 333198
2018-05-24 17:58:34 +02:00
|
|
|
|
2017-01-15 07:32:49 +01:00
|
|
|
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(); }
|
2019-04-19 19:46:50 +02:00
|
|
|
Pass *llvm::createLICMPass(unsigned LicmMssaOptCap,
|
|
|
|
unsigned LicmMssaNoAccForPromotionCap) {
|
|
|
|
return new LegacyLICMPass(LicmMssaOptCap, LicmMssaNoAccForPromotionCap);
|
|
|
|
}
|
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,
|
2019-08-14 06:50:33 +02:00
|
|
|
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.");
|
|
|
|
|
2019-08-08 15:46:17 +02:00
|
|
|
// If this loop has metadata indicating that LICM is not to be performed then
|
|
|
|
// just exit.
|
|
|
|
if (hasDisableLICMTransformsHint(L)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
std::unique_ptr<AliasSetTracker> CurAST;
|
|
|
|
std::unique_ptr<MemorySSAUpdater> MSSAU;
|
2019-02-11 20:07:15 +01:00
|
|
|
bool NoOfMemAccTooLarge = false;
|
2019-04-19 19:46:50 +02:00
|
|
|
unsigned LicmMssaOptCounter = 0;
|
2019-02-11 20:07:15 +01:00
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
if (!MSSA) {
|
|
|
|
LLVM_DEBUG(dbgs() << "LICM: Using Alias Set Tracker.\n");
|
|
|
|
CurAST = collectAliasInfoForLoop(L, LI, AA);
|
|
|
|
} else {
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM: Using MemorySSA.\n");
|
2019-08-15 17:54:37 +02:00
|
|
|
MSSAU = std::make_unique<MemorySSAUpdater>(MSSA);
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
|
|
|
|
unsigned AccessCapCount = 0;
|
|
|
|
for (auto *BB : L->getBlocks()) {
|
|
|
|
if (auto *Accesses = MSSA->getBlockAccesses(BB)) {
|
|
|
|
for (const auto &MA : *Accesses) {
|
|
|
|
(void)MA;
|
|
|
|
AccessCapCount++;
|
2019-04-19 19:46:50 +02:00
|
|
|
if (AccessCapCount > LicmMssaNoAccForPromotionCap) {
|
2019-02-11 20:07:15 +01:00
|
|
|
NoOfMemAccTooLarge = true;
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2019-02-11 20:07:15 +01:00
|
|
|
if (NoOfMemAccTooLarge)
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
break;
|
|
|
|
}
|
2019-01-10 20:29:04 +01:00
|
|
|
}
|
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.
|
2018-11-06 03:44:49 +01:00
|
|
|
ICFLoopSafetyInfo SafetyInfo(DT);
|
2018-08-15 07:55:43 +02:00
|
|
|
SafetyInfo.computeLoopSafetyInfo(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.
|
2019-04-19 19:46:50 +02:00
|
|
|
SinkAndHoistLICMFlags Flags = {NoOfMemAccTooLarge, LicmMssaOptCounter,
|
2019-06-20 23:09:09 +02:00
|
|
|
LicmMssaOptCap, LicmMssaNoAccForPromotionCap,
|
|
|
|
/*IsSink=*/true};
|
2009-11-05 22:11:53 +01:00
|
|
|
if (L->hasDedicatedExits())
|
2019-08-14 06:50:33 +02:00
|
|
|
Changed |= sinkRegion(DT->getNode(L->getHeader()), AA, LI, DT, TLI, TTI, L,
|
2019-04-19 19:46:50 +02:00
|
|
|
CurAST.get(), MSSAU.get(), &SafetyInfo, Flags, ORE);
|
2019-06-20 23:09:09 +02:00
|
|
|
Flags.IsSink = false;
|
2009-11-05 22:11:53 +01:00
|
|
|
if (Preheader)
|
2019-08-14 06:50:33 +02:00
|
|
|
Changed |= hoistRegion(DT->getNode(L->getHeader()), AA, LI, DT, TLI, L,
|
2019-04-19 19:46:50 +02:00
|
|
|
CurAST.get(), MSSAU.get(), &SafetyInfo, Flags, 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.
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
if (!DisablePromotion && Preheader && L->hasDedicatedExits() &&
|
2019-02-11 20:07:15 +01:00
|
|
|
!NoOfMemAccTooLarge) {
|
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;
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
SmallVector<MemoryAccess *, 8> MSSAInsertPts;
|
2016-12-30 00:11:19 +01:00
|
|
|
InsertPts.reserve(ExitBlocks.size());
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
if (MSSAU)
|
|
|
|
MSSAInsertPts.reserve(ExitBlocks.size());
|
|
|
|
for (BasicBlock *ExitBlock : ExitBlocks) {
|
2016-12-30 00:11:19 +01:00
|
|
|
InsertPts.push_back(&*ExitBlock->getFirstInsertionPt());
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
if (MSSAU)
|
|
|
|
MSSAInsertPts.push_back(nullptr);
|
|
|
|
}
|
2016-12-30 00:11:19 +01:00
|
|
|
|
|
|
|
PredIteratorCache PIC;
|
|
|
|
|
|
|
|
bool Promoted = false;
|
|
|
|
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
// Build an AST using MSSA.
|
|
|
|
if (!CurAST.get())
|
|
|
|
CurAST = collectAliasInfoForLoopWithMSSA(L, AA, MSSAU.get());
|
|
|
|
|
|
|
|
// Loop over all of the alias sets in the tracker object.
|
|
|
|
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() ||
|
|
|
|
!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, MSSAInsertPts, PIC, LI,
|
|
|
|
DT, TLI, L, CurAST.get(), MSSAU.get(), &SafetyInfo, ORE);
|
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
|
|
|
}
|
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.
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
if (!MSSAU.get() && CurAST.get() && L->getParentLoop() && !DeleteAST)
|
2018-08-21 22:30:14 +02:00
|
|
|
LoopToAliasSetMap[L] = std::move(CurAST);
|
2016-05-03 19:50:11 +02:00
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
if (MSSAU.get() && VerifyMemorySSA)
|
|
|
|
MSSAU->getMemorySSA()->verifyMemorySSA();
|
|
|
|
|
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,
|
2019-08-14 06:50:33 +02:00
|
|
|
DominatorTree *DT, TargetLibraryInfo *TLI,
|
|
|
|
TargetTransformInfo *TTI, Loop *CurLoop,
|
|
|
|
AliasSetTracker *CurAST, MemorySSAUpdater *MSSAU,
|
2019-04-19 19:46:50 +02:00
|
|
|
ICFLoopSafetyInfo *SafetyInfo,
|
|
|
|
SinkAndHoistLICMFlags &Flags,
|
|
|
|
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 &&
|
2019-01-10 20:29:04 +01:00
|
|
|
CurLoop != nullptr && SafetyInfo != nullptr &&
|
|
|
|
"Unexpected input to sinkRegion.");
|
|
|
|
assert(((CurAST != nullptr) ^ (MSSAU != nullptr)) &&
|
|
|
|
"Either AliasSetTracker or MemorySSA should be initialized.");
|
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;
|
2019-01-10 20:29:04 +01:00
|
|
|
eraseInstruction(I, *SafetyInfo, CurAST, MSSAU);
|
2017-07-20 05:27:02 +02:00
|
|
|
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) &&
|
2019-04-19 19:46:50 +02:00
|
|
|
canSinkOrHoistInst(I, AA, DT, CurLoop, CurAST, MSSAU, true, &Flags,
|
|
|
|
ORE) &&
|
2018-08-29 23:49:30 +02:00
|
|
|
!I.mayHaveSideEffects()) {
|
2019-08-14 06:50:33 +02:00
|
|
|
if (sink(I, LI, DT, CurLoop, SafetyInfo, MSSAU, ORE)) {
|
2017-12-15 21:33:24 +01:00
|
|
|
if (!FreeInLoop) {
|
|
|
|
++II;
|
2019-01-10 20:29:04 +01:00
|
|
|
eraseInstruction(I, *SafetyInfo, CurAST, MSSAU);
|
2017-12-15 21:33:24 +01:00
|
|
|
}
|
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
|
|
|
}
|
2019-01-10 20:29:04 +01:00
|
|
|
if (MSSAU && VerifyMemorySSA)
|
|
|
|
MSSAU->getMemorySSA()->verifyMemorySSA();
|
2015-02-22 19:35:32 +01:00
|
|
|
return Changed;
|
2003-12-19 08:22:45 +01:00
|
|
|
}
|
|
|
|
|
2019-01-12 19:36:22 +01:00
|
|
|
namespace {
|
2018-11-29 18:10:00 +01:00
|
|
|
// This is a helper class for hoistRegion to make it able to hoist control flow
|
|
|
|
// in order to be able to hoist phis. The way this works is that we initially
|
|
|
|
// start hoisting to the loop preheader, and when we see a loop invariant branch
|
|
|
|
// we make note of this. When we then come to hoist an instruction that's
|
|
|
|
// conditional on such a branch we duplicate the branch and the relevant control
|
|
|
|
// flow, then hoist the instruction into the block corresponding to its original
|
|
|
|
// block in the duplicated control flow.
|
|
|
|
class ControlFlowHoister {
|
|
|
|
private:
|
|
|
|
// Information about the loop we are hoisting from
|
|
|
|
LoopInfo *LI;
|
|
|
|
DominatorTree *DT;
|
|
|
|
Loop *CurLoop;
|
2019-01-10 20:29:04 +01:00
|
|
|
MemorySSAUpdater *MSSAU;
|
2018-11-29 18:10:00 +01:00
|
|
|
|
|
|
|
// A map of blocks in the loop to the block their instructions will be hoisted
|
|
|
|
// to.
|
|
|
|
DenseMap<BasicBlock *, BasicBlock *> HoistDestinationMap;
|
|
|
|
|
|
|
|
// The branches that we can hoist, mapped to the block that marks a
|
|
|
|
// convergence point of their control flow.
|
|
|
|
DenseMap<BranchInst *, BasicBlock *> HoistableBranches;
|
|
|
|
|
|
|
|
public:
|
2019-01-10 20:29:04 +01:00
|
|
|
ControlFlowHoister(LoopInfo *LI, DominatorTree *DT, Loop *CurLoop,
|
|
|
|
MemorySSAUpdater *MSSAU)
|
|
|
|
: LI(LI), DT(DT), CurLoop(CurLoop), MSSAU(MSSAU) {}
|
2018-11-29 18:10:00 +01:00
|
|
|
|
|
|
|
void registerPossiblyHoistableBranch(BranchInst *BI) {
|
|
|
|
// We can only hoist conditional branches with loop invariant operands.
|
|
|
|
if (!ControlFlowHoisting || !BI->isConditional() ||
|
|
|
|
!CurLoop->hasLoopInvariantOperands(BI))
|
|
|
|
return;
|
|
|
|
|
|
|
|
// The branch destinations need to be in the loop, and we don't gain
|
|
|
|
// anything by duplicating conditional branches with duplicate successors,
|
|
|
|
// as it's essentially the same as an unconditional branch.
|
|
|
|
BasicBlock *TrueDest = BI->getSuccessor(0);
|
|
|
|
BasicBlock *FalseDest = BI->getSuccessor(1);
|
|
|
|
if (!CurLoop->contains(TrueDest) || !CurLoop->contains(FalseDest) ||
|
|
|
|
TrueDest == FalseDest)
|
|
|
|
return;
|
|
|
|
|
|
|
|
// We can hoist BI if one branch destination is the successor of the other,
|
|
|
|
// or both have common successor which we check by seeing if the
|
|
|
|
// intersection of their successors is non-empty.
|
|
|
|
// TODO: This could be expanded to allowing branches where both ends
|
|
|
|
// eventually converge to a single block.
|
|
|
|
SmallPtrSet<BasicBlock *, 4> TrueDestSucc, FalseDestSucc;
|
|
|
|
TrueDestSucc.insert(succ_begin(TrueDest), succ_end(TrueDest));
|
|
|
|
FalseDestSucc.insert(succ_begin(FalseDest), succ_end(FalseDest));
|
|
|
|
BasicBlock *CommonSucc = nullptr;
|
|
|
|
if (TrueDestSucc.count(FalseDest)) {
|
|
|
|
CommonSucc = FalseDest;
|
|
|
|
} else if (FalseDestSucc.count(TrueDest)) {
|
|
|
|
CommonSucc = TrueDest;
|
|
|
|
} else {
|
|
|
|
set_intersect(TrueDestSucc, FalseDestSucc);
|
|
|
|
// If there's one common successor use that.
|
|
|
|
if (TrueDestSucc.size() == 1)
|
|
|
|
CommonSucc = *TrueDestSucc.begin();
|
|
|
|
// If there's more than one pick whichever appears first in the block list
|
|
|
|
// (we can't use the value returned by TrueDestSucc.begin() as it's
|
|
|
|
// unpredicatable which element gets returned).
|
|
|
|
else if (!TrueDestSucc.empty()) {
|
|
|
|
Function *F = TrueDest->getParent();
|
|
|
|
auto IsSucc = [&](BasicBlock &BB) { return TrueDestSucc.count(&BB); };
|
|
|
|
auto It = std::find_if(F->begin(), F->end(), IsSucc);
|
|
|
|
assert(It != F->end() && "Could not find successor in function");
|
|
|
|
CommonSucc = &*It;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// The common successor has to be dominated by the branch, as otherwise
|
|
|
|
// there will be some other path to the successor that will not be
|
|
|
|
// controlled by this branch so any phi we hoist would be controlled by the
|
|
|
|
// wrong condition. This also takes care of avoiding hoisting of loop back
|
|
|
|
// edges.
|
|
|
|
// TODO: In some cases this could be relaxed if the successor is dominated
|
|
|
|
// by another block that's been hoisted and we can guarantee that the
|
|
|
|
// control flow has been replicated exactly.
|
|
|
|
if (CommonSucc && DT->dominates(BI, CommonSucc))
|
|
|
|
HoistableBranches[BI] = CommonSucc;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool canHoistPHI(PHINode *PN) {
|
|
|
|
// The phi must have loop invariant operands.
|
|
|
|
if (!ControlFlowHoisting || !CurLoop->hasLoopInvariantOperands(PN))
|
|
|
|
return false;
|
|
|
|
// We can hoist phis if the block they are in is the target of hoistable
|
|
|
|
// branches which cover all of the predecessors of the block.
|
|
|
|
SmallPtrSet<BasicBlock *, 8> PredecessorBlocks;
|
|
|
|
BasicBlock *BB = PN->getParent();
|
|
|
|
for (BasicBlock *PredBB : predecessors(BB))
|
|
|
|
PredecessorBlocks.insert(PredBB);
|
|
|
|
// If we have less predecessor blocks than predecessors then the phi will
|
|
|
|
// have more than one incoming value for the same block which we can't
|
|
|
|
// handle.
|
|
|
|
// TODO: This could be handled be erasing some of the duplicate incoming
|
|
|
|
// values.
|
|
|
|
if (PredecessorBlocks.size() != pred_size(BB))
|
|
|
|
return false;
|
|
|
|
for (auto &Pair : HoistableBranches) {
|
|
|
|
if (Pair.second == BB) {
|
|
|
|
// Which blocks are predecessors via this branch depends on if the
|
|
|
|
// branch is triangle-like or diamond-like.
|
|
|
|
if (Pair.first->getSuccessor(0) == BB) {
|
|
|
|
PredecessorBlocks.erase(Pair.first->getParent());
|
|
|
|
PredecessorBlocks.erase(Pair.first->getSuccessor(1));
|
|
|
|
} else if (Pair.first->getSuccessor(1) == BB) {
|
|
|
|
PredecessorBlocks.erase(Pair.first->getParent());
|
|
|
|
PredecessorBlocks.erase(Pair.first->getSuccessor(0));
|
|
|
|
} else {
|
|
|
|
PredecessorBlocks.erase(Pair.first->getSuccessor(0));
|
|
|
|
PredecessorBlocks.erase(Pair.first->getSuccessor(1));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// PredecessorBlocks will now be empty if for every predecessor of BB we
|
|
|
|
// found a hoistable branch source.
|
|
|
|
return PredecessorBlocks.empty();
|
|
|
|
}
|
|
|
|
|
|
|
|
BasicBlock *getOrCreateHoistedBlock(BasicBlock *BB) {
|
2018-12-05 11:16:21 +01:00
|
|
|
if (!ControlFlowHoisting)
|
|
|
|
return CurLoop->getLoopPreheader();
|
2018-11-29 18:10:00 +01:00
|
|
|
// If BB has already been hoisted, return that
|
|
|
|
if (HoistDestinationMap.count(BB))
|
|
|
|
return HoistDestinationMap[BB];
|
|
|
|
|
|
|
|
// Check if this block is conditional based on a pending branch
|
|
|
|
auto HasBBAsSuccessor =
|
|
|
|
[&](DenseMap<BranchInst *, BasicBlock *>::value_type &Pair) {
|
|
|
|
return BB != Pair.second && (Pair.first->getSuccessor(0) == BB ||
|
|
|
|
Pair.first->getSuccessor(1) == BB);
|
|
|
|
};
|
|
|
|
auto It = std::find_if(HoistableBranches.begin(), HoistableBranches.end(),
|
|
|
|
HasBBAsSuccessor);
|
|
|
|
|
|
|
|
// If not involved in a pending branch, hoist to preheader
|
|
|
|
BasicBlock *InitialPreheader = CurLoop->getLoopPreheader();
|
|
|
|
if (It == HoistableBranches.end()) {
|
|
|
|
LLVM_DEBUG(dbgs() << "LICM using " << InitialPreheader->getName()
|
|
|
|
<< " as hoist destination for " << BB->getName()
|
|
|
|
<< "\n");
|
|
|
|
HoistDestinationMap[BB] = InitialPreheader;
|
|
|
|
return InitialPreheader;
|
|
|
|
}
|
|
|
|
BranchInst *BI = It->first;
|
|
|
|
assert(std::find_if(++It, HoistableBranches.end(), HasBBAsSuccessor) ==
|
|
|
|
HoistableBranches.end() &&
|
|
|
|
"BB is expected to be the target of at most one branch");
|
|
|
|
|
|
|
|
LLVMContext &C = BB->getContext();
|
|
|
|
BasicBlock *TrueDest = BI->getSuccessor(0);
|
|
|
|
BasicBlock *FalseDest = BI->getSuccessor(1);
|
|
|
|
BasicBlock *CommonSucc = HoistableBranches[BI];
|
|
|
|
BasicBlock *HoistTarget = getOrCreateHoistedBlock(BI->getParent());
|
|
|
|
|
|
|
|
// Create hoisted versions of blocks that currently don't have them
|
|
|
|
auto CreateHoistedBlock = [&](BasicBlock *Orig) {
|
|
|
|
if (HoistDestinationMap.count(Orig))
|
|
|
|
return HoistDestinationMap[Orig];
|
|
|
|
BasicBlock *New =
|
|
|
|
BasicBlock::Create(C, Orig->getName() + ".licm", Orig->getParent());
|
|
|
|
HoistDestinationMap[Orig] = New;
|
|
|
|
DT->addNewBlock(New, HoistTarget);
|
|
|
|
if (CurLoop->getParentLoop())
|
|
|
|
CurLoop->getParentLoop()->addBasicBlockToLoop(New, *LI);
|
|
|
|
++NumCreatedBlocks;
|
|
|
|
LLVM_DEBUG(dbgs() << "LICM created " << New->getName()
|
|
|
|
<< " as hoist destination for " << Orig->getName()
|
|
|
|
<< "\n");
|
|
|
|
return New;
|
|
|
|
};
|
|
|
|
BasicBlock *HoistTrueDest = CreateHoistedBlock(TrueDest);
|
|
|
|
BasicBlock *HoistFalseDest = CreateHoistedBlock(FalseDest);
|
|
|
|
BasicBlock *HoistCommonSucc = CreateHoistedBlock(CommonSucc);
|
|
|
|
|
|
|
|
// Link up these blocks with branches.
|
|
|
|
if (!HoistCommonSucc->getTerminator()) {
|
|
|
|
// The new common successor we've generated will branch to whatever that
|
|
|
|
// hoist target branched to.
|
|
|
|
BasicBlock *TargetSucc = HoistTarget->getSingleSuccessor();
|
|
|
|
assert(TargetSucc && "Expected hoist target to have a single successor");
|
|
|
|
HoistCommonSucc->moveBefore(TargetSucc);
|
|
|
|
BranchInst::Create(TargetSucc, HoistCommonSucc);
|
|
|
|
}
|
|
|
|
if (!HoistTrueDest->getTerminator()) {
|
|
|
|
HoistTrueDest->moveBefore(HoistCommonSucc);
|
|
|
|
BranchInst::Create(HoistCommonSucc, HoistTrueDest);
|
|
|
|
}
|
|
|
|
if (!HoistFalseDest->getTerminator()) {
|
|
|
|
HoistFalseDest->moveBefore(HoistCommonSucc);
|
|
|
|
BranchInst::Create(HoistCommonSucc, HoistFalseDest);
|
|
|
|
}
|
|
|
|
|
|
|
|
// If BI is being cloned to what was originally the preheader then
|
|
|
|
// HoistCommonSucc will now be the new preheader.
|
|
|
|
if (HoistTarget == InitialPreheader) {
|
|
|
|
// Phis in the loop header now need to use the new preheader.
|
|
|
|
InitialPreheader->replaceSuccessorsPhiUsesWith(HoistCommonSucc);
|
2019-01-10 20:29:04 +01:00
|
|
|
if (MSSAU)
|
|
|
|
MSSAU->wireOldPredecessorsToNewImmediatePredecessor(
|
|
|
|
HoistTarget->getSingleSuccessor(), HoistCommonSucc, {HoistTarget});
|
2018-11-29 18:10:00 +01:00
|
|
|
// The new preheader dominates the loop header.
|
|
|
|
DomTreeNode *PreheaderNode = DT->getNode(HoistCommonSucc);
|
|
|
|
DomTreeNode *HeaderNode = DT->getNode(CurLoop->getHeader());
|
|
|
|
DT->changeImmediateDominator(HeaderNode, PreheaderNode);
|
|
|
|
// The preheader hoist destination is now the new preheader, with the
|
|
|
|
// exception of the hoist destination of this branch.
|
|
|
|
for (auto &Pair : HoistDestinationMap)
|
|
|
|
if (Pair.second == InitialPreheader && Pair.first != BI->getParent())
|
|
|
|
Pair.second = HoistCommonSucc;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Now finally clone BI.
|
|
|
|
ReplaceInstWithInst(
|
|
|
|
HoistTarget->getTerminator(),
|
|
|
|
BranchInst::Create(HoistTrueDest, HoistFalseDest, BI->getCondition()));
|
|
|
|
++NumClonedBranches;
|
|
|
|
|
|
|
|
assert(CurLoop->getLoopPreheader() &&
|
|
|
|
"Hoisting blocks should not have destroyed preheader");
|
|
|
|
return HoistDestinationMap[BB];
|
|
|
|
}
|
|
|
|
};
|
2019-01-12 19:36:22 +01:00
|
|
|
} // namespace
|
2018-11-29 18:10:00 +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,
|
2019-08-14 06:50:33 +02:00
|
|
|
DominatorTree *DT, TargetLibraryInfo *TLI, Loop *CurLoop,
|
2019-01-10 20:29:04 +01:00
|
|
|
AliasSetTracker *CurAST, MemorySSAUpdater *MSSAU,
|
2019-04-19 19:46:50 +02:00
|
|
|
ICFLoopSafetyInfo *SafetyInfo,
|
|
|
|
SinkAndHoistLICMFlags &Flags,
|
2017-01-11 05:39:35 +01:00
|
|
|
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 &&
|
2019-01-10 20:29:04 +01:00
|
|
|
CurLoop != nullptr && SafetyInfo != nullptr &&
|
|
|
|
"Unexpected input to hoistRegion.");
|
|
|
|
assert(((CurAST != nullptr) ^ (MSSAU != nullptr)) &&
|
|
|
|
"Either AliasSetTracker or MemorySSA should be initialized.");
|
2016-01-14 00:01:57 +01:00
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
ControlFlowHoister CFH(LI, DT, CurLoop, MSSAU);
|
2018-11-28 18:21:49 +01:00
|
|
|
|
2018-11-29 18:10:00 +01:00
|
|
|
// Keep track of instructions that have been hoisted, as they may need to be
|
|
|
|
// re-hoisted if they end up not dominating all of their uses.
|
|
|
|
SmallVector<Instruction *, 16> HoistedInstructions;
|
|
|
|
|
|
|
|
// For PHI hoisting to work we need to hoist blocks before their successors.
|
|
|
|
// We can do this by iterating through the blocks in the loop in reverse
|
|
|
|
// post-order.
|
|
|
|
LoopBlocksRPO Worklist(CurLoop);
|
|
|
|
Worklist.perform(LI);
|
2016-01-14 00:01:57 +01:00
|
|
|
bool Changed = false;
|
2018-11-29 18:10:00 +01:00
|
|
|
for (BasicBlock *BB : Worklist) {
|
2017-07-20 05:27:02 +02:00
|
|
|
// 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-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');
|
2019-01-10 20:29:04 +01:00
|
|
|
if (CurAST)
|
|
|
|
CurAST->copyValue(&I, C);
|
|
|
|
// FIXME MSSA: Such replacements may make accesses unoptimized (D51960).
|
2018-04-27 22:58:30 +02:00
|
|
|
I.replaceAllUsesWith(C);
|
2018-11-02 01:21:45 +01:00
|
|
|
if (isInstructionTriviallyDead(&I, TLI))
|
2019-01-10 20:29:04 +01:00
|
|
|
eraseInstruction(I, *SafetyInfo, CurAST, MSSAU);
|
2018-04-27 22:58:30 +02:00
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2018-06-23 06:01:28 +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
|
2019-08-14 06:50:33 +02:00
|
|
|
// if it is safe to hoist the instruction.
|
2018-11-29 18:10:00 +01:00
|
|
|
// TODO: It may be safe to hoist if we are hoisting to a conditional block
|
|
|
|
// and we have accurately duplicated the control flow from the loop header
|
|
|
|
// to that block.
|
2018-06-23 06:01:28 +02:00
|
|
|
if (CurLoop->hasLoopInvariantOperands(&I) &&
|
2019-04-19 19:46:50 +02:00
|
|
|
canSinkOrHoistInst(I, AA, DT, CurLoop, CurAST, MSSAU, true, &Flags,
|
|
|
|
ORE) &&
|
2018-11-06 05:17:40 +01:00
|
|
|
isSafeToExecuteUnconditionally(
|
|
|
|
I, DT, CurLoop, SafetyInfo, ORE,
|
|
|
|
CurLoop->getLoopPreheader()->getTerminator())) {
|
2019-01-10 20:29:04 +01:00
|
|
|
hoist(I, DT, CurLoop, CFH.getOrCreateHoistedBlock(BB), SafetyInfo,
|
|
|
|
MSSAU, ORE);
|
2018-11-29 18:10:00 +01:00
|
|
|
HoistedInstructions.push_back(&I);
|
2018-08-15 04:49:12 +02:00
|
|
|
Changed = true;
|
2018-06-23 06:01:28 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2018-04-27 22:58:30 +02:00
|
|
|
// 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());
|
2019-01-09 08:28:13 +01:00
|
|
|
SafetyInfo->insertInstructionTo(ReciprocalDivisor, I.getParent());
|
2018-04-27 22:58:30 +02:00
|
|
|
ReciprocalDivisor->insertBefore(&I);
|
|
|
|
|
|
|
|
auto Product =
|
|
|
|
BinaryOperator::CreateFMul(I.getOperand(0), ReciprocalDivisor);
|
|
|
|
Product->setFastMathFlags(I.getFastMathFlags());
|
2019-01-09 08:28:13 +01:00
|
|
|
SafetyInfo->insertInstructionTo(Product, I.getParent());
|
2018-04-27 22:58:30 +02:00
|
|
|
Product->insertAfter(&I);
|
|
|
|
I.replaceAllUsesWith(Product);
|
2019-01-10 20:29:04 +01:00
|
|
|
eraseInstruction(I, *SafetyInfo, CurAST, MSSAU);
|
2017-04-11 04:22:54 +02:00
|
|
|
|
2018-11-29 18:10:00 +01:00
|
|
|
hoist(*ReciprocalDivisor, DT, CurLoop, CFH.getOrCreateHoistedBlock(BB),
|
2019-01-10 20:29:04 +01:00
|
|
|
SafetyInfo, MSSAU, ORE);
|
2018-11-29 18:10:00 +01:00
|
|
|
HoistedInstructions.push_back(ReciprocalDivisor);
|
2018-04-27 22:58:30 +02:00
|
|
|
Changed = true;
|
|
|
|
continue;
|
2017-07-20 05:27:02 +02:00
|
|
|
}
|
2018-04-27 22:58:30 +02:00
|
|
|
|
2019-02-14 10:04:12 +01:00
|
|
|
auto IsInvariantStart = [&](Instruction &I) {
|
|
|
|
using namespace PatternMatch;
|
|
|
|
return I.use_empty() &&
|
|
|
|
match(&I, m_Intrinsic<Intrinsic::invariant_start>());
|
|
|
|
};
|
|
|
|
auto MustExecuteWithoutWritesBefore = [&](Instruction &I) {
|
|
|
|
return SafetyInfo->isGuaranteedToExecute(I, DT, CurLoop) &&
|
|
|
|
SafetyInfo->doesNotWriteMemoryBefore(I, CurLoop);
|
|
|
|
};
|
|
|
|
if ((IsInvariantStart(I) || isGuard(&I)) &&
|
2018-11-12 10:29:58 +01:00
|
|
|
CurLoop->hasLoopInvariantOperands(&I) &&
|
2019-02-14 10:04:12 +01:00
|
|
|
MustExecuteWithoutWritesBefore(I)) {
|
2019-01-10 20:29:04 +01:00
|
|
|
hoist(I, DT, CurLoop, CFH.getOrCreateHoistedBlock(BB), SafetyInfo,
|
|
|
|
MSSAU, ORE);
|
2018-11-29 18:10:00 +01:00
|
|
|
HoistedInstructions.push_back(&I);
|
2018-08-21 10:11:31 +02:00
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
2018-11-29 18:10:00 +01:00
|
|
|
|
|
|
|
if (PHINode *PN = dyn_cast<PHINode>(&I)) {
|
|
|
|
if (CFH.canHoistPHI(PN)) {
|
|
|
|
// Redirect incoming blocks first to ensure that we create hoisted
|
|
|
|
// versions of those blocks before we hoist the phi.
|
|
|
|
for (unsigned int i = 0; i < PN->getNumIncomingValues(); ++i)
|
|
|
|
PN->setIncomingBlock(
|
|
|
|
i, CFH.getOrCreateHoistedBlock(PN->getIncomingBlock(i)));
|
|
|
|
hoist(*PN, DT, CurLoop, CFH.getOrCreateHoistedBlock(BB), SafetyInfo,
|
2019-01-10 20:29:04 +01:00
|
|
|
MSSAU, ORE);
|
2018-11-29 18:10:00 +01:00
|
|
|
assert(DT->dominates(PN, BB) && "Conditional PHIs not expected");
|
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// Remember possibly hoistable branches so we can actually hoist them
|
|
|
|
// later if needed.
|
|
|
|
if (BranchInst *BI = dyn_cast<BranchInst>(&I))
|
|
|
|
CFH.registerPossiblyHoistableBranch(BI);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// If we hoisted instructions to a conditional block they may not dominate
|
|
|
|
// their uses that weren't hoisted (such as phis where some operands are not
|
|
|
|
// loop invariant). If so make them unconditional by moving them to their
|
|
|
|
// immediate dominator. We iterate through the instructions in reverse order
|
|
|
|
// which ensures that when we rehoist an instruction we rehoist its operands,
|
|
|
|
// and also keep track of where in the block we are rehoisting to to make sure
|
|
|
|
// that we rehoist instructions before the instructions that use them.
|
|
|
|
Instruction *HoistPoint = nullptr;
|
2018-12-05 11:16:21 +01:00
|
|
|
if (ControlFlowHoisting) {
|
|
|
|
for (Instruction *I : reverse(HoistedInstructions)) {
|
|
|
|
if (!llvm::all_of(I->uses(),
|
|
|
|
[&](Use &U) { return DT->dominates(I, U); })) {
|
|
|
|
BasicBlock *Dominator =
|
|
|
|
DT->getNode(I->getParent())->getIDom()->getBlock();
|
2019-01-04 18:12:09 +01:00
|
|
|
if (!HoistPoint || !DT->dominates(HoistPoint->getParent(), Dominator)) {
|
2018-12-05 11:16:21 +01:00
|
|
|
if (HoistPoint)
|
|
|
|
assert(DT->dominates(Dominator, HoistPoint->getParent()) &&
|
|
|
|
"New hoist point expected to dominate old hoist point");
|
|
|
|
HoistPoint = Dominator->getTerminator();
|
|
|
|
}
|
2019-01-04 18:12:09 +01:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM rehoisting to "
|
|
|
|
<< HoistPoint->getParent()->getName()
|
|
|
|
<< ": " << *I << "\n");
|
2019-01-24 20:57:30 +01:00
|
|
|
moveInstructionBefore(*I, *HoistPoint, *SafetyInfo, MSSAU);
|
2018-12-05 11:16:21 +01:00
|
|
|
HoistPoint = I;
|
|
|
|
Changed = true;
|
2018-11-29 18:10:00 +01:00
|
|
|
}
|
2018-04-27 22:58:30 +02:00
|
|
|
}
|
2017-07-20 05:27:02 +02:00
|
|
|
}
|
2019-01-10 20:29:04 +01:00
|
|
|
if (MSSAU && VerifyMemorySSA)
|
|
|
|
MSSAU->getMemorySSA()->verifyMemorySSA();
|
2002-09-29 23:46:09 +02:00
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
// Now that we've finished hoisting make sure that LI and DT are still
|
|
|
|
// valid.
|
2018-11-29 18:10:00 +01:00
|
|
|
#ifndef NDEBUG
|
|
|
|
if (Changed) {
|
|
|
|
assert(DT->verify(DominatorTree::VerificationLevel::Fast) &&
|
|
|
|
"Dominator tree verification failed");
|
|
|
|
LI->verify(*DT);
|
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
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();
|
2019-07-11 15:12:08 +02:00
|
|
|
const uint32_t LocSizeInBits = DL.getTypeSizeInBits(LI->getType());
|
2017-02-02 14:22:03 +01:00
|
|
|
|
|
|
|
// 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;
|
|
|
|
}
|
|
|
|
|
2018-08-02 02:54:14 +02:00
|
|
|
namespace {
|
|
|
|
/// Return true if-and-only-if we know how to (mechanically) both hoist and
|
|
|
|
/// sink a given instruction out of a loop. Does not address legality
|
2019-04-01 22:37:56 +02:00
|
|
|
/// concerns such as aliasing or speculation safety.
|
2018-08-02 02:54:14 +02:00
|
|
|
bool isHoistableAndSinkableInst(Instruction &I) {
|
|
|
|
// Only these instructions are hoistable/sinkable.
|
2019-04-01 22:37:56 +02:00
|
|
|
return (isa<LoadInst>(I) || isa<StoreInst>(I) || isa<CallInst>(I) ||
|
2019-08-08 23:38:31 +02:00
|
|
|
isa<FenceInst>(I) || isa<CastInst>(I) ||
|
|
|
|
isa<UnaryOperator>(I) || isa<BinaryOperator>(I) ||
|
2019-04-01 22:37:56 +02:00
|
|
|
isa<SelectInst>(I) || isa<GetElementPtrInst>(I) || isa<CmpInst>(I) ||
|
|
|
|
isa<InsertElementInst>(I) || isa<ExtractElementInst>(I) ||
|
|
|
|
isa<ShuffleVectorInst>(I) || isa<ExtractValueInst>(I) ||
|
|
|
|
isa<InsertValueInst>(I));
|
2018-08-02 02:54:14 +02:00
|
|
|
}
|
2018-08-07 00:07:37 +02:00
|
|
|
/// Return true if all of the alias sets within this AST are known not to
|
2019-01-10 20:29:04 +01:00
|
|
|
/// contain a Mod, or if MSSA knows thare are no MemoryDefs in the loop.
|
|
|
|
bool isReadOnly(AliasSetTracker *CurAST, const MemorySSAUpdater *MSSAU,
|
|
|
|
const Loop *L) {
|
|
|
|
if (CurAST) {
|
|
|
|
for (AliasSet &AS : *CurAST) {
|
|
|
|
if (!AS.isForwardingAliasSet() && AS.isMod()) {
|
|
|
|
return false;
|
|
|
|
}
|
2018-08-07 00:07:37 +02:00
|
|
|
}
|
2019-01-10 20:29:04 +01:00
|
|
|
return true;
|
|
|
|
} else { /*MSSAU*/
|
|
|
|
for (auto *BB : L->getBlocks())
|
|
|
|
if (MSSAU->getMemorySSA()->getBlockDefs(BB))
|
|
|
|
return false;
|
|
|
|
return true;
|
2018-08-07 00:07:37 +02:00
|
|
|
}
|
2019-01-10 20:29:04 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
/// Return true if I is the only Instruction with a MemoryAccess in L.
|
|
|
|
bool isOnlyMemoryAccess(const Instruction *I, const Loop *L,
|
|
|
|
const MemorySSAUpdater *MSSAU) {
|
|
|
|
for (auto *BB : L->getBlocks())
|
|
|
|
if (auto *Accs = MSSAU->getMemorySSA()->getBlockAccesses(BB)) {
|
|
|
|
int NotAPhi = 0;
|
|
|
|
for (const auto &Acc : *Accs) {
|
|
|
|
if (isa<MemoryPhi>(&Acc))
|
|
|
|
continue;
|
|
|
|
const auto *MUD = cast<MemoryUseOrDef>(&Acc);
|
|
|
|
if (MUD->getMemoryInst() != I || NotAPhi++ == 1)
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
2018-08-07 00:07:37 +02:00
|
|
|
return true;
|
|
|
|
}
|
2018-08-02 02:54:14 +02:00
|
|
|
}
|
|
|
|
|
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,
|
2019-01-10 20:29:04 +01:00
|
|
|
MemorySSAUpdater *MSSAU,
|
2018-08-02 06:08:04 +02:00
|
|
|
bool TargetExecutesOncePerLoop,
|
2019-04-19 19:46:50 +02:00
|
|
|
SinkAndHoistLICMFlags *Flags,
|
2017-01-11 05:39:45 +01:00
|
|
|
OptimizationRemarkEmitter *ORE) {
|
2018-08-02 02:54:14 +02:00
|
|
|
// If we don't understand the instruction, bail early.
|
|
|
|
if (!isHoistableAndSinkableInst(I))
|
|
|
|
return false;
|
2019-01-10 20:29:04 +01:00
|
|
|
|
|
|
|
MemorySSA *MSSA = MSSAU ? MSSAU->getMemorySSA() : nullptr;
|
2019-02-13 00:05:40 +01:00
|
|
|
if (MSSA)
|
2019-04-19 19:46:50 +02:00
|
|
|
assert(Flags != nullptr && "Flags cannot be null.");
|
2019-01-10 20:29:04 +01:00
|
|
|
|
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
|
|
|
|
2018-08-02 06:08:04 +02:00
|
|
|
if (LI->isAtomic() && !TargetExecutesOncePerLoop)
|
|
|
|
return false; // Don't risk duplicating unordered loads
|
2017-10-11 09:26:45 +02:00
|
|
|
|
2017-02-02 14:22:03 +01:00
|
|
|
// This checks for an invariant.start dominating the load.
|
|
|
|
if (isLoadInvariantInLoop(LI, DT, CurLoop))
|
|
|
|
return true;
|
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
bool Invalidated;
|
|
|
|
if (CurAST)
|
|
|
|
Invalidated = pointerInvalidatedByLoop(MemoryLocation::get(LI), CurAST,
|
|
|
|
CurLoop, AA);
|
|
|
|
else
|
|
|
|
Invalidated = pointerInvalidatedByLoopWithMSSA(
|
2019-04-19 19:46:50 +02:00
|
|
|
MSSA, cast<MemoryUse>(MSSA->getMemoryAccess(LI)), CurLoop, *Flags);
|
2017-01-11 05:39:45 +01:00
|
|
|
// 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;
|
|
|
|
|
2018-08-24 21:13:39 +02:00
|
|
|
using namespace PatternMatch;
|
|
|
|
if (match(CI, m_Intrinsic<Intrinsic::assume>()))
|
|
|
|
// Assumes don't actually alias anything or throw
|
|
|
|
return true;
|
2019-01-10 20:29:04 +01:00
|
|
|
|
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)) {
|
2018-08-29 23:49:30 +02:00
|
|
|
// TODO: expand to writeable arguments
|
2015-09-22 00:27:59 +02:00
|
|
|
for (Value *Op : CI->arg_operands())
|
2019-01-10 20:29:04 +01:00
|
|
|
if (Op->getType()->isPointerTy()) {
|
|
|
|
bool Invalidated;
|
|
|
|
if (CurAST)
|
|
|
|
Invalidated = pointerInvalidatedByLoop(
|
2018-10-10 23:28:44 +02:00
|
|
|
MemoryLocation(Op, LocationSize::unknown(), AAMDNodes()),
|
2019-01-10 20:29:04 +01:00
|
|
|
CurAST, CurLoop, AA);
|
|
|
|
else
|
|
|
|
Invalidated = pointerInvalidatedByLoopWithMSSA(
|
2019-02-13 00:05:40 +01:00
|
|
|
MSSA, cast<MemoryUse>(MSSA->getMemoryAccess(CI)), CurLoop,
|
2019-04-19 19:46:50 +02:00
|
|
|
*Flags);
|
2019-01-10 20:29:04 +01:00
|
|
|
if (Invalidated)
|
|
|
|
return false;
|
|
|
|
}
|
2015-09-22 00:27:59 +02:00
|
|
|
return true;
|
|
|
|
}
|
2018-08-07 00:07:37 +02:00
|
|
|
|
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.
|
2019-01-10 20:29:04 +01:00
|
|
|
if (isReadOnly(CurAST, MSSAU, CurLoop))
|
2016-05-05 02:54:54 +02:00
|
|
|
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;
|
2018-08-09 22:18:42 +02:00
|
|
|
} else if (auto *FI = dyn_cast<FenceInst>(&I)) {
|
|
|
|
// Fences alias (most) everything to provide ordering. For the moment,
|
|
|
|
// just give up if there are any other memory operations in the loop.
|
2019-01-10 20:29:04 +01:00
|
|
|
if (CurAST) {
|
|
|
|
auto Begin = CurAST->begin();
|
|
|
|
assert(Begin != CurAST->end() && "must contain FI");
|
|
|
|
if (std::next(Begin) != CurAST->end())
|
|
|
|
// constant memory for instance, TODO: handle better
|
|
|
|
return false;
|
|
|
|
auto *UniqueI = Begin->getUniqueInstruction();
|
|
|
|
if (!UniqueI)
|
|
|
|
// other memory op, give up
|
|
|
|
return false;
|
|
|
|
(void)FI; // suppress unused variable warning
|
|
|
|
assert(UniqueI == FI && "AS must contain FI");
|
|
|
|
return true;
|
|
|
|
} else // MSSAU
|
|
|
|
return isOnlyMemoryAccess(FI, CurLoop, MSSAU);
|
2018-08-29 23:49:30 +02:00
|
|
|
} else if (auto *SI = dyn_cast<StoreInst>(&I)) {
|
|
|
|
if (!SI->isUnordered())
|
|
|
|
return false; // Don't sink/hoist volatile or ordered atomic store!
|
|
|
|
|
|
|
|
// We can only hoist a store that we can prove writes a value which is not
|
|
|
|
// read or overwritten within the loop. For those cases, we fallback to
|
2018-08-30 00:09:21 +02:00
|
|
|
// load store promotion instead. TODO: We can extend this to cases where
|
|
|
|
// there is exactly one write to the location and that write dominates an
|
|
|
|
// arbitrary number of reads in the loop.
|
2019-01-10 20:29:04 +01:00
|
|
|
if (CurAST) {
|
|
|
|
auto &AS = CurAST->getAliasSetFor(MemoryLocation::get(SI));
|
2018-08-29 23:49:30 +02:00
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
if (AS.isRef() || !AS.isMustAlias())
|
|
|
|
// Quick exit test, handled by the full path below as well.
|
|
|
|
return false;
|
|
|
|
auto *UniqueI = AS.getUniqueInstruction();
|
|
|
|
if (!UniqueI)
|
|
|
|
// other memory op, give up
|
|
|
|
return false;
|
|
|
|
assert(UniqueI == SI && "AS must contain SI");
|
|
|
|
return true;
|
|
|
|
} else { // MSSAU
|
|
|
|
if (isOnlyMemoryAccess(SI, CurLoop, MSSAU))
|
|
|
|
return true;
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
// If there are more accesses than the Promotion cap, give up, we're not
|
|
|
|
// walking a list that long.
|
2019-04-19 19:46:50 +02:00
|
|
|
if (Flags->NoOfMemAccTooLarge)
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
return false;
|
|
|
|
// Check store only if there's still "quota" to check clobber.
|
2019-04-19 19:46:50 +02:00
|
|
|
if (Flags->LicmMssaOptCounter >= Flags->LicmMssaOptCap)
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
return false;
|
|
|
|
// If there are interfering Uses (i.e. their defining access is in the
|
|
|
|
// loop), or ordered loads (stored as Defs!), don't move this store.
|
|
|
|
// Could do better here, but this is conservatively correct.
|
|
|
|
// TODO: Cache set of Uses on the first walk in runOnLoop, update when
|
|
|
|
// moving accesses. Can also extend to dominating uses.
|
2019-06-20 23:09:09 +02:00
|
|
|
auto *SIMD = MSSA->getMemoryAccess(SI);
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
for (auto *BB : CurLoop->getBlocks())
|
|
|
|
if (auto *Accesses = MSSA->getBlockAccesses(BB)) {
|
|
|
|
for (const auto &MA : *Accesses)
|
|
|
|
if (const auto *MU = dyn_cast<MemoryUse>(&MA)) {
|
|
|
|
auto *MD = MU->getDefiningAccess();
|
|
|
|
if (!MSSA->isLiveOnEntryDef(MD) &&
|
|
|
|
CurLoop->contains(MD->getBlock()))
|
|
|
|
return false;
|
2019-06-20 23:09:09 +02:00
|
|
|
// Disable hoisting past potentially interfering loads. Optimized
|
|
|
|
// Uses may point to an access outside the loop, as getClobbering
|
|
|
|
// checks the previous iteration when walking the backedge.
|
|
|
|
// FIXME: More precise: no Uses that alias SI.
|
|
|
|
if (!Flags->IsSink && !MSSA->dominates(SIMD, MU))
|
|
|
|
return false;
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
} else if (const auto *MD = dyn_cast<MemoryDef>(&MA))
|
|
|
|
if (auto *LI = dyn_cast<LoadInst>(MD->getMemoryInst())) {
|
2019-03-21 08:54:44 +01:00
|
|
|
(void)LI; // Silence warning.
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
assert(!LI->isUnordered() && "Expected unordered load");
|
|
|
|
return false;
|
|
|
|
}
|
2019-03-21 12:58:22 +01:00
|
|
|
}
|
[LICM & MemorySSA] Don't sink/hoist stores in the presence of ordered loads.
Summary:
Before this patch, if any Use existed in the loop, with a defining
access in the loop, we conservatively decide to not move the store.
What this approach was missing, is that ordered loads are not Uses, they're Defs
in MemorySSA. So, even when the clobbering walker does not find that
volatile load to interfere, we still cannot hoist a store past a
volatile load.
Resolves PR41140.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59564
llvm-svn: 356588
2019-03-20 19:33:37 +01:00
|
|
|
|
2019-03-21 12:58:22 +01:00
|
|
|
auto *Source = MSSA->getSkipSelfWalker()->getClobberingMemoryAccess(SI);
|
2019-04-19 19:46:50 +02:00
|
|
|
Flags->LicmMssaOptCounter++;
|
2019-03-21 12:58:22 +01:00
|
|
|
// If there are no clobbering Defs in the loop, store is safe to hoist.
|
|
|
|
return MSSA->isLiveOnEntryDef(Source) ||
|
|
|
|
!CurLoop->contains(Source->getBlock());
|
2019-01-10 20:29:04 +01:00
|
|
|
}
|
2003-12-09 20:32:44 +01:00
|
|
|
}
|
|
|
|
|
2018-08-09 05:44:28 +02:00
|
|
|
assert(!I.mayReadOrWriteMemory() && "unhandled aliasing");
|
|
|
|
|
2018-08-02 06:08:04 +02:00
|
|
|
// We've established mechanical ability and aliasing, it's up to the caller
|
|
|
|
// to check fault safety
|
|
|
|
return true;
|
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
|
|
|
///
|
2018-07-02 20:53:40 +02:00
|
|
|
static bool isTriviallyReplaceablePHI(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) {
|
2018-10-16 10:07:14 +02:00
|
|
|
const auto &BlockColors = SafetyInfo->getBlockColors();
|
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;
|
|
|
|
}
|
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
static Instruction *CloneInstructionInExitBlock(
|
|
|
|
Instruction &I, BasicBlock &ExitBlock, PHINode &PN, const LoopInfo *LI,
|
|
|
|
const LoopSafetyInfo *SafetyInfo, MemorySSAUpdater *MSSAU) {
|
2016-01-04 04:37:39 +01:00
|
|
|
Instruction *New;
|
|
|
|
if (auto *CI = dyn_cast<CallInst>(&I)) {
|
2018-10-16 10:07:14 +02:00
|
|
|
const auto &BlockColors = SafetyInfo->getBlockColors();
|
2016-01-04 04:37:39 +01:00
|
|
|
|
|
|
|
// Sinking call-sites need to be handled differently from other
|
|
|
|
// instructions. The cloned call-site needs a funclet bundle operand
|
2019-04-01 22:37:56 +02:00
|
|
|
// appropriate for its location in the CFG.
|
2016-01-04 04:37:39 +01:00
|
|
|
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
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
MemoryAccess *OldMemAcc;
|
|
|
|
if (MSSAU && (OldMemAcc = MSSAU->getMemorySSA()->getMemoryAccess(&I))) {
|
|
|
|
// Create a new MemoryAccess and let MemorySSA set its defining access.
|
|
|
|
MemoryAccess *NewMemAcc = MSSAU->createMemoryAccessInBB(
|
|
|
|
New, nullptr, New->getParent(), MemorySSA::Beginning);
|
|
|
|
if (NewMemAcc) {
|
|
|
|
if (auto *MemDef = dyn_cast<MemoryDef>(NewMemAcc))
|
|
|
|
MSSAU->insertDef(MemDef, /*RenameUses=*/true);
|
|
|
|
else {
|
|
|
|
auto *MemUse = cast<MemoryUse>(NewMemAcc);
|
2019-08-19 20:57:40 +02:00
|
|
|
MSSAU->insertUse(MemUse, /*RenameUses=*/true);
|
2019-01-10 20:29:04 +01:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
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;
|
|
|
|
}
|
|
|
|
|
2018-11-06 03:44:49 +01:00
|
|
|
static void eraseInstruction(Instruction &I, ICFLoopSafetyInfo &SafetyInfo,
|
2019-01-10 20:29:04 +01:00
|
|
|
AliasSetTracker *AST, MemorySSAUpdater *MSSAU) {
|
2018-11-02 01:21:45 +01:00
|
|
|
if (AST)
|
|
|
|
AST->deleteValue(&I);
|
2019-01-10 20:29:04 +01:00
|
|
|
if (MSSAU)
|
|
|
|
MSSAU->removeMemoryAccess(&I);
|
2018-11-06 03:44:49 +01:00
|
|
|
SafetyInfo.removeInstruction(&I);
|
2018-11-02 01:21:45 +01:00
|
|
|
I.eraseFromParent();
|
|
|
|
}
|
|
|
|
|
2018-11-09 06:39:04 +01:00
|
|
|
static void moveInstructionBefore(Instruction &I, Instruction &Dest,
|
2019-01-24 20:57:30 +01:00
|
|
|
ICFLoopSafetyInfo &SafetyInfo,
|
|
|
|
MemorySSAUpdater *MSSAU) {
|
2018-11-09 06:39:04 +01:00
|
|
|
SafetyInfo.removeInstruction(&I);
|
2019-01-09 08:28:13 +01:00
|
|
|
SafetyInfo.insertInstructionTo(&I, Dest.getParent());
|
2018-11-09 06:39:04 +01:00
|
|
|
I.moveBefore(&Dest);
|
2019-01-24 20:57:30 +01:00
|
|
|
if (MSSAU)
|
|
|
|
if (MemoryUseOrDef *OldMemAcc = cast_or_null<MemoryUseOrDef>(
|
|
|
|
MSSAU->getMemorySSA()->getMemoryAccess(&I)))
|
2019-01-24 21:30:48 +01:00
|
|
|
MSSAU->moveToPlace(OldMemAcc, Dest.getParent(), MemorySSA::End);
|
2018-11-09 06:39:04 +01:00
|
|
|
}
|
|
|
|
|
2018-07-02 20:53:40 +02:00
|
|
|
static Instruction *sinkThroughTriviallyReplaceablePHI(
|
2017-11-03 17:24:53 +01:00
|
|
|
PHINode *TPN, Instruction *I, LoopInfo *LI,
|
|
|
|
SmallDenseMap<BasicBlock *, Instruction *, 32> &SunkCopies,
|
2019-01-10 20:29:04 +01:00
|
|
|
const LoopSafetyInfo *SafetyInfo, const Loop *CurLoop,
|
|
|
|
MemorySSAUpdater *MSSAU) {
|
2018-07-02 20:53:40 +02:00
|
|
|
assert(isTriviallyReplaceablePHI(*TPN, *I) &&
|
|
|
|
"Expect only trivially replaceable PHI");
|
2017-11-03 17:24:53 +01:00
|
|
|
BasicBlock *ExitBlock = TPN->getParent();
|
|
|
|
Instruction *New;
|
|
|
|
auto It = SunkCopies.find(ExitBlock);
|
|
|
|
if (It != SunkCopies.end())
|
|
|
|
New = It->second;
|
|
|
|
else
|
2019-01-10 20:29:04 +01:00
|
|
|
New = SunkCopies[ExitBlock] = CloneInstructionInExitBlock(
|
|
|
|
*I, *ExitBlock, *TPN, LI, SafetyInfo, MSSAU);
|
2017-11-03 17:24:53 +01:00
|
|
|
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.
|
2018-10-16 10:07:14 +02:00
|
|
|
if (!SafetyInfo->getBlockColors().empty() && BB->getFirstNonPHI()->isEHPad())
|
2018-02-12 18:56:55 +01:00
|
|
|
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,
|
2019-01-10 20:29:04 +01:00
|
|
|
LoopSafetyInfo *SafetyInfo,
|
|
|
|
MemorySSAUpdater *MSSAU) {
|
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
|
2018-07-02 20:53:40 +02:00
|
|
|
// exposed to exit blocks through trivially replaceable PHIs while keeping the
|
2017-11-03 17:24:53 +01:00
|
|
|
// 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:
|
2018-07-02 20:53:40 +02:00
|
|
|
// %p = phi [%v1, %LB1], [%v2, %LB2] <-- non-trivially replaceable
|
2017-11-03 17:24:53 +01:00
|
|
|
//
|
|
|
|
// to
|
|
|
|
//
|
|
|
|
// LB1:
|
|
|
|
// %v1 =
|
|
|
|
// br %LE.split, %LB2
|
|
|
|
// LB2:
|
|
|
|
// %v2 =
|
|
|
|
// br %LE.split2, %LB1
|
|
|
|
// LE.split:
|
2018-07-02 20:53:40 +02:00
|
|
|
// %p1 = phi [%v1, %LB1] <-- trivially replaceable
|
2017-11-03 17:24:53 +01:00
|
|
|
// br %LE
|
|
|
|
// LE.split2:
|
2018-07-02 20:53:40 +02:00
|
|
|
// %p2 = phi [%v2, %LB2] <-- trivially replaceable
|
2017-11-03 17:24:53 +01:00
|
|
|
// br %LE
|
|
|
|
// LE:
|
|
|
|
// %p = phi [%p1, %LE.split], [%p2, %LE.split2]
|
|
|
|
//
|
2018-10-16 10:07:14 +02:00
|
|
|
const auto &BlockColors = SafetyInfo->getBlockColors();
|
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(
|
2019-01-10 20:29:04 +01:00
|
|
|
ExitBB, PredBB, ".split.loop.exit", DT, LI, MSSAU, true);
|
2018-02-12 18:56:55 +01:00
|
|
|
// Since we do not allow splitting EH-block with BlockColors in
|
|
|
|
// canSplitPredecessors(), we can simply assign predecessor's color to
|
|
|
|
// the new block.
|
2018-10-16 10:07:14 +02:00
|
|
|
if (!BlockColors.empty())
|
2018-03-23 18:36:18 +01:00
|
|
|
// 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.
|
2018-10-16 10:07:14 +02:00
|
|
|
SafetyInfo->copyColors(NewPred, PredBB);
|
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,
|
2019-08-14 06:50:33 +02:00
|
|
|
const Loop *CurLoop, ICFLoopSafetyInfo *SafetyInfo,
|
|
|
|
MemorySSAUpdater *MSSAU, OptimizationRemarkEmitter *ORE) {
|
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
|
2019-04-01 22:37:56 +02:00
|
|
|
// unreachable blocks with undef and make all user PHIs trivially replaceable.
|
2017-11-03 17:24:53 +01:00
|
|
|
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);
|
2018-07-02 20:53:40 +02:00
|
|
|
if (isTriviallyReplaceablePHI(*PN, I))
|
2017-11-03 17:24:53 +01:00
|
|
|
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
|
2018-07-02 20:53:40 +02:00
|
|
|
// replaceable.
|
2019-01-10 20:29:04 +01:00
|
|
|
splitPredecessorsOfLoopExit(PN, DT, LI, CurLoop, SafetyInfo, MSSAU);
|
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!");
|
2018-07-02 20:53:40 +02:00
|
|
|
// The PHI must be trivially replaceable.
|
2019-01-10 20:29:04 +01:00
|
|
|
Instruction *New = sinkThroughTriviallyReplaceablePHI(
|
|
|
|
PN, &I, LI, SunkCopies, SafetyInfo, CurLoop, MSSAU);
|
2014-02-11 13:52:27 +01:00
|
|
|
PN->replaceAllUsesWith(New);
|
2019-01-10 20:29:04 +01:00
|
|
|
eraseInstruction(*PN, *SafetyInfo, nullptr, nullptr);
|
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
|
|
|
///
|
2018-08-15 04:49:12 +02:00
|
|
|
static void hoist(Instruction &I, const DominatorTree *DT, const Loop *CurLoop,
|
2018-11-29 18:10:00 +01:00
|
|
|
BasicBlock *Dest, ICFLoopSafetyInfo *SafetyInfo,
|
2019-01-10 20:29:04 +01:00
|
|
|
MemorySSAUpdater *MSSAU, OptimizationRemarkEmitter *ORE) {
|
2018-11-29 18:10:00 +01:00
|
|
|
LLVM_DEBUG(dbgs() << "LICM hoisting to " << Dest->getName() << ": " << I
|
2018-05-14 14:53:11 +02:00
|
|
|
<< "\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.
|
2018-10-16 08:34:53 +02:00
|
|
|
!SafetyInfo->isGuaranteedToExecute(I, DT, CurLoop))
|
2016-01-28 16:51:58 +01:00
|
|
|
I.dropUnknownNonDebugMetadata();
|
|
|
|
|
2018-11-29 18:10:00 +01:00
|
|
|
if (isa<PHINode>(I))
|
|
|
|
// Move the new node to the end of the phi list in the destination block.
|
2019-01-24 20:57:30 +01:00
|
|
|
moveInstructionBefore(I, *Dest->getFirstNonPHI(), *SafetyInfo, MSSAU);
|
2018-11-29 18:10:00 +01:00
|
|
|
else
|
|
|
|
// Move the new node to the destination block, before its terminator.
|
2019-01-24 20:57:30 +01:00
|
|
|
moveInstructionBefore(I, *Dest->getTerminator(), *SafetyInfo, MSSAU);
|
2005-04-22 01:48:37 +02:00
|
|
|
|
2019-04-20 00:36:40 +02:00
|
|
|
// Apply line 0 debug locations when we are moving instructions to different
|
|
|
|
// basic blocks because we want to avoid jumpy line tables.
|
|
|
|
if (const DebugLoc &DL = I.getDebugLoc())
|
|
|
|
I.setDebugLoc(DebugLoc::get(0, 0, DL.getScope(), DL.getInlinedAt()));
|
2017-01-06 19:38:57 +01:00
|
|
|
|
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;
|
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 =
|
2018-10-16 08:34:53 +02:00
|
|
|
SafetyInfo->isGuaranteedToExecute(Inst, DT, CurLoop);
|
2017-01-11 05:39:49 +01:00
|
|
|
|
|
|
|
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;
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
SmallVectorImpl<MemoryAccess *> &MSSAInsertPts;
|
2016-05-05 02:54:54 +02:00
|
|
|
PredIteratorCache &PredCache;
|
|
|
|
AliasSetTracker &AST;
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
MemorySSAUpdater *MSSAU;
|
2016-05-05 02:54:54 +02:00
|
|
|
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;
|
2018-11-06 03:44:49 +01:00
|
|
|
ICFLoopSafetyInfo &SafetyInfo;
|
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,
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
SmallVectorImpl<Instruction *> &LIP,
|
|
|
|
SmallVectorImpl<MemoryAccess *> &MSSAIP, PredIteratorCache &PIC,
|
|
|
|
AliasSetTracker &ast, MemorySSAUpdater *MSSAU, LoopInfo &li,
|
|
|
|
DebugLoc dl, int alignment, bool UnorderedAtomic,
|
|
|
|
const AAMDNodes &AATags, ICFLoopSafetyInfo &SafetyInfo)
|
2016-05-05 02:54:54 +02:00
|
|
|
: LoadAndStorePromoter(Insts, S), SomePtr(SP), PointerMustAliases(PMA),
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
LoopExitBlocks(LEB), LoopInsertPts(LIP), MSSAInsertPts(MSSAIP),
|
|
|
|
PredCache(PIC), AST(ast), MSSAU(MSSAU), LI(li), DL(std::move(dl)),
|
|
|
|
Alignment(alignment), UnorderedAtomic(UnorderedAtomic), AATags(AATags),
|
|
|
|
SafetyInfo(SafetyInfo) {}
|
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
|
|
|
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
void doExtraRewritesBeforeFinalDeletion() override {
|
2016-05-05 02:54:54 +02:00
|
|
|
// 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);
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
|
|
|
|
if (MSSAU) {
|
|
|
|
MemoryAccess *MSSAInsertPoint = MSSAInsertPts[i];
|
|
|
|
MemoryAccess *NewMemAcc;
|
|
|
|
if (!MSSAInsertPoint) {
|
|
|
|
NewMemAcc = MSSAU->createMemoryAccessInBB(
|
|
|
|
NewSI, nullptr, NewSI->getParent(), MemorySSA::Beginning);
|
|
|
|
} else {
|
|
|
|
NewMemAcc =
|
|
|
|
MSSAU->createMemoryAccessAfter(NewSI, nullptr, MSSAInsertPoint);
|
|
|
|
}
|
|
|
|
MSSAInsertPts[i] = NewMemAcc;
|
|
|
|
MSSAU->insertDef(cast<MemoryDef>(NewMemAcc), true);
|
|
|
|
// FIXME: true for safety, false may still be correct.
|
|
|
|
}
|
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);
|
|
|
|
}
|
2018-11-06 03:44:49 +01:00
|
|
|
void instructionDeleted(Instruction *I) const override {
|
|
|
|
SafetyInfo.removeInstruction(I);
|
|
|
|
AST.deleteValue(I);
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
if (MSSAU)
|
|
|
|
MSSAU->removeMemoryAccess(I);
|
2018-11-06 03:44:49 +01:00
|
|
|
}
|
2016-05-05 02:54:54 +02:00
|
|
|
};
|
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
|
2018-07-30 21:41:25 +02:00
|
|
|
// capture.
|
2017-10-26 23:00:15 +02:00
|
|
|
return true;
|
2018-07-30 21:41:25 +02:00
|
|
|
|
2017-10-26 23:00:15 +02:00
|
|
|
// 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,
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
SmallVectorImpl<Instruction *> &InsertPts,
|
|
|
|
SmallVectorImpl<MemoryAccess *> &MSSAInsertPts, PredIteratorCache &PIC,
|
2016-05-05 02:54:54 +02:00
|
|
|
LoopInfo *LI, DominatorTree *DT, const TargetLibraryInfo *TLI,
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
Loop *CurLoop, AliasSetTracker *CurAST, MemorySSAUpdater *MSSAU,
|
|
|
|
ICFLoopSafetyInfo *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;
|
2018-08-15 07:55:43 +02:00
|
|
|
if (SafetyInfo->anyBlockMayThrow()) {
|
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
|
2018-07-30 21:41:25 +02:00
|
|
|
// 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)) {
|
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
|
|
|
|
[LICM] Infer proper alignment from loads during scalar promotion
This patch fixes an issue where we would compute an unnecessarily small alignment during scalar promotion when no store is not to be guaranteed to execute, but we've proven load speculation safety. Since speculating a load requires proving the existing alignment is valid at the new location (see Loads.cpp), we can use the alignment fact from the load.
For non-atomics, this is a performance problem. For atomics, this is a correctness issue, though an *incredibly* rare one to see in practice. For atomics, we might not be able to lower an improperly aligned load or store (i.e. i32 align 1). If such an instruction makes it all the way to codegen, we *may* fail to codegen the operation, or we may simply generate a slow call to a library function. The part that makes this super hard to see in practice is that the memory location actually *is* well aligned, and instcombine knows that. So, to see a failure, you have to have a) hit the bug in LICM, b) somehow hit a depth limit in InstCombine/ValueTracking to avoid fixing the alignment, and c) then have generated an instruction which fails codegen rather than simply emitting a slow libcall. All around, pretty hard to hit.
Differential Revision: https://reviews.llvm.org/D58809
llvm-svn: 355217
2019-03-01 19:45:05 +01:00
|
|
|
unsigned InstAlignment = Load->getAlignment();
|
|
|
|
if (!InstAlignment)
|
|
|
|
InstAlignment =
|
|
|
|
MDL.getABITypeAlignment(Load->getType());
|
|
|
|
|
|
|
|
// Note that proving a load safe to speculate requires proving
|
|
|
|
// sufficient alignment at the target location. Proving it guaranteed
|
|
|
|
// to execute does as well. Thus we can increase our guaranteed
|
2019-08-14 06:50:33 +02:00
|
|
|
// alignment as well.
|
[LICM] Infer proper alignment from loads during scalar promotion
This patch fixes an issue where we would compute an unnecessarily small alignment during scalar promotion when no store is not to be guaranteed to execute, but we've proven load speculation safety. Since speculating a load requires proving the existing alignment is valid at the new location (see Loads.cpp), we can use the alignment fact from the load.
For non-atomics, this is a performance problem. For atomics, this is a correctness issue, though an *incredibly* rare one to see in practice. For atomics, we might not be able to lower an improperly aligned load or store (i.e. i32 align 1). If such an instruction makes it all the way to codegen, we *may* fail to codegen the operation, or we may simply generate a slow call to a library function. The part that makes this super hard to see in practice is that the memory location actually *is* well aligned, and instcombine knows that. So, to see a failure, you have to have a) hit the bug in LICM, b) somehow hit a depth limit in InstCombine/ValueTracking to avoid fixing the alignment, and c) then have generated an instruction which fails codegen rather than simply emitting a slow libcall. All around, pretty hard to hit.
Differential Revision: https://reviews.llvm.org/D58809
llvm-svn: 355217
2019-03-01 19:45:05 +01:00
|
|
|
if (!DereferenceableInPH || (InstAlignment > Alignment))
|
|
|
|
if (isSafeToExecuteUnconditionally(*Load, DT, CurLoop, SafetyInfo,
|
|
|
|
ORE, Preheader->getTerminator())) {
|
|
|
|
DereferenceableInPH = true;
|
|
|
|
Alignment = std::max(Alignment, InstAlignment);
|
|
|
|
}
|
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;
|
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)) {
|
2018-10-16 08:34:53 +02:00
|
|
|
if (SafetyInfo->isGuaranteedToExecute(*UI, DT, CurLoop)) {
|
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(
|
2019-07-09 13:35:35 +02:00
|
|
|
Store->getPointerOperand(), Store->getValueOperand()->getType(),
|
|
|
|
Store->getAlignment(), MDL, 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
|
|
|
|
[LICM] Infer proper alignment from loads during scalar promotion
This patch fixes an issue where we would compute an unnecessarily small alignment during scalar promotion when no store is not to be guaranteed to execute, but we've proven load speculation safety. Since speculating a load requires proving the existing alignment is valid at the new location (see Loads.cpp), we can use the alignment fact from the load.
For non-atomics, this is a performance problem. For atomics, this is a correctness issue, though an *incredibly* rare one to see in practice. For atomics, we might not be able to lower an improperly aligned load or store (i.e. i32 align 1). If such an instruction makes it all the way to codegen, we *may* fail to codegen the operation, or we may simply generate a slow call to a library function. The part that makes this super hard to see in practice is that the memory location actually *is* well aligned, and instcombine knows that. So, to see a failure, you have to have a) hit the bug in LICM, b) somehow hit a depth limit in InstCombine/ValueTracking to avoid fixing the alignment, and c) then have generated an instruction which fails codegen rather than simply emitting a slow libcall. All around, pretty hard to hit.
Differential Revision: https://reviews.llvm.org/D58809
llvm-svn: 355217
2019-03-01 19:45:05 +01:00
|
|
|
// If we're inserting an atomic load in the preheader, we must be able to
|
|
|
|
// lower it. We're only guaranteed to be able to lower naturally aligned
|
|
|
|
// atomics.
|
|
|
|
auto *SomePtrElemType = SomePtr->getType()->getPointerElementType();
|
|
|
|
if (SawUnorderedAtomic &&
|
|
|
|
Alignment < MDL.getTypeStoreSize(SomePtrElemType))
|
|
|
|
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,
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
InsertPts, MSSAInsertPts, PIC, *CurAST, MSSAU, *LI, DL,
|
|
|
|
Alignment, SawUnorderedAtomic, AATags, *SafetyInfo);
|
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(
|
2019-02-01 21:44:24 +01:00
|
|
|
SomePtr->getType()->getPointerElementType(), 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);
|
|
|
|
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
MemoryAccess *PreheaderLoadMemoryAccess;
|
|
|
|
if (MSSAU) {
|
|
|
|
PreheaderLoadMemoryAccess = MSSAU->createMemoryAccessInBB(
|
|
|
|
PreheaderLoad, nullptr, PreheaderLoad->getParent(), MemorySSA::End);
|
|
|
|
MemoryUse *NewMemUse = cast<MemoryUse>(PreheaderLoadMemoryAccess);
|
2019-08-19 20:57:40 +02:00
|
|
|
MSSAU->insertUse(NewMemUse, /*RenameUses=*/true);
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
}
|
|
|
|
|
2019-08-19 20:57:40 +02:00
|
|
|
if (MSSAU && VerifyMemorySSA)
|
|
|
|
MSSAU->getMemorySSA()->verifyMemorySSA();
|
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
|
|
|
|
[MemorySSA] Make insertDef insert corresponding phi nodes.
Summary:
The original assumption for the insertDef method was that it would not
materialize Defs out of no-where, hence it will not insert phis needed
after inserting a Def.
However, when cloning an instruction (use case used in LICM), we do
materialize Defs "out of no-where". If the block receiving a Def has at
least one other Def, then no processing is needed. If the block just
received its first Def, we must check where Phi placement is needed.
The only new usage of insertDef is in LICM, hence the trigger for the bug.
But the original goal of the method also fails to apply for the move()
method. If we move a Def from the entry point of a diamond to either the
left or right blocks, then the merge block must add a phi.
While this usecase does not currently occur, or may be viewed as an
incorrect transformation, MSSA must behave corectly given the scenario.
Resolves PR40749 and PR40754.
Reviewers: george.burgess.iv
Subscribers: sanjoy, jlebar, Prazek, jdoerfert, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D58652
llvm-svn: 355040
2019-02-27 23:20:22 +01:00
|
|
|
if (MSSAU && VerifyMemorySSA)
|
|
|
|
MSSAU->getMemorySSA()->verifyMemorySSA();
|
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())
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
eraseInstruction(*PreheaderLoad, *SafetyInfo, CurAST, MSSAU);
|
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.
|
2018-08-21 22:30:14 +02:00
|
|
|
std::unique_ptr<AliasSetTracker>
|
2016-07-13 00:37:48 +02:00
|
|
|
LoopInvariantCodeMotion::collectAliasInfoForLoop(Loop *L, LoopInfo *LI,
|
|
|
|
AliasAnalysis *AA) {
|
2018-08-21 22:30:14 +02:00
|
|
|
std::unique_ptr<AliasSetTracker> CurAST;
|
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;
|
|
|
|
}
|
2018-08-21 22:30:14 +02:00
|
|
|
std::unique_ptr<AliasSetTracker> InnerAST = std::move(MapI->second);
|
2016-02-15 15:48:50 +01:00
|
|
|
|
2018-08-21 22:30:14 +02:00
|
|
|
if (CurAST) {
|
2016-02-15 15:48:50 +01:00
|
|
|
// What if InnerLoop was modified by other passes ?
|
|
|
|
// Once we've incorporated the inner loop's AST into ours, we don't need
|
|
|
|
// the subloop's anymore.
|
2018-08-21 22:30:14 +02:00
|
|
|
CurAST->add(*InnerAST);
|
2016-02-15 15:48:50 +01:00
|
|
|
} else {
|
2018-08-21 22:30:14 +02:00
|
|
|
CurAST = std::move(InnerAST);
|
2016-02-15 15:48:50 +01:00
|
|
|
}
|
2016-02-27 05:34:07 +01:00
|
|
|
LoopToAliasSetMap.erase(MapI);
|
2016-02-15 15:48:50 +01:00
|
|
|
}
|
2018-08-21 22:30:14 +02:00
|
|
|
if (!CurAST)
|
2019-08-15 17:54:37 +02:00
|
|
|
CurAST = std::make_unique<AliasSetTracker>(*AA);
|
2016-02-27 05:34:07 +01:00
|
|
|
|
|
|
|
// Add everything from the sub loops that are no longer directly available.
|
|
|
|
for (Loop *InnerL : RecomputeLoops)
|
2018-09-11 06:07:36 +02:00
|
|
|
for (BasicBlock *BB : InnerL->blocks())
|
|
|
|
CurAST->add(*BB);
|
2016-02-27 05:34:07 +01:00
|
|
|
|
2018-09-11 06:07:36 +02:00
|
|
|
// And merge in this loop (without anything from inner loops).
|
|
|
|
for (BasicBlock *BB : L->blocks())
|
|
|
|
if (LI->getLoopFor(BB) == L)
|
|
|
|
CurAST->add(*BB);
|
2016-02-27 05:34:07 +01:00
|
|
|
|
2016-02-15 15:48:50 +01:00
|
|
|
return CurAST;
|
|
|
|
}
|
|
|
|
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
std::unique_ptr<AliasSetTracker>
|
|
|
|
LoopInvariantCodeMotion::collectAliasInfoForLoopWithMSSA(
|
|
|
|
Loop *L, AliasAnalysis *AA, MemorySSAUpdater *MSSAU) {
|
|
|
|
auto *MSSA = MSSAU->getMemorySSA();
|
2019-08-15 17:54:37 +02:00
|
|
|
auto CurAST = std::make_unique<AliasSetTracker>(*AA, MSSA, L);
|
[LICM/MSSA] Add promotion to scalars by building an AliasSetTracker with MemorySSA.
Summary:
Experimentally we found that promotion to scalars carries less benefits
than sinking and hoisting in LICM. When using MemorySSA, we build an
AliasSetTracker on demand in order to reuse the current infrastructure.
We only build it if less than AccessCapForMSSAPromotion exist in the
loop, a cap that is by default set to 250. This value ensures there are
no runtime regressions, and there are small compile time gains for
pathological cases. A much lower value (20) was found to yield a single
regression in the llvm-test-suite and much higher benefits for compile
times. Conservatively we set the current cap to a high value, but we will
explore lowering it when MemorySSA is enabled by default.
Reviewers: sanjoy, chandlerc
Subscribers: nemanjai, jlebar, Prazek, george.burgess.iv, jfb, jsji, llvm-commits
Differential Revision: https://reviews.llvm.org/D56625
llvm-svn: 353339
2019-02-06 21:25:17 +01:00
|
|
|
CurAST->addAllInstructionsInLoopUsingMSSA();
|
|
|
|
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) {
|
2018-08-21 22:30:14 +02:00
|
|
|
auto ASTIt = LICM.getLoopToAliasSetMap().find(L);
|
|
|
|
if (ASTIt == LICM.getLoopToAliasSetMap().end())
|
2007-07-31 10:01:41 +02:00
|
|
|
return;
|
|
|
|
|
2018-08-21 22:30:14 +02:00
|
|
|
ASTIt->second->copyValue(From, To);
|
2007-07-31 10:01:41 +02:00
|
|
|
}
|
|
|
|
|
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) {
|
2018-08-21 22:30:14 +02:00
|
|
|
auto ASTIt = LICM.getLoopToAliasSetMap().find(L);
|
|
|
|
if (ASTIt == LICM.getLoopToAliasSetMap().end())
|
2007-07-31 10:01:41 +02:00
|
|
|
return;
|
|
|
|
|
2018-08-21 22:30:14 +02:00
|
|
|
ASTIt->second->deleteValue(V);
|
2007-07-31 10:01:41 +02:00
|
|
|
}
|
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) {
|
2018-08-21 22:30:14 +02:00
|
|
|
if (!LICM.getLoopToAliasSetMap().count(L))
|
2014-09-24 18:48:31 +02:00
|
|
|
return;
|
|
|
|
|
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
|
|
|
|
2018-08-17 15:44:00 +02:00
|
|
|
static bool pointerInvalidatedByLoop(MemoryLocation MemLoc,
|
|
|
|
AliasSetTracker *CurAST, Loop *CurLoop,
|
|
|
|
AliasAnalysis *AA) {
|
|
|
|
// First check to see if any of the basic blocks in CurLoop invalidate *V.
|
|
|
|
bool isInvalidatedAccordingToAST = CurAST->getAliasSetFor(MemLoc).isMod();
|
|
|
|
|
|
|
|
if (!isInvalidatedAccordingToAST || !LICMN2Theshold)
|
|
|
|
return isInvalidatedAccordingToAST;
|
|
|
|
|
|
|
|
// Check with a diagnostic analysis if we can refine the information above.
|
|
|
|
// This is to identify the limitations of using the AST.
|
|
|
|
// The alias set mechanism used by LICM has a major weakness in that it
|
|
|
|
// combines all things which may alias into a single set *before* asking
|
|
|
|
// modref questions. As a result, a single readonly call within a loop will
|
|
|
|
// collapse all loads and stores into a single alias set and report
|
|
|
|
// invalidation if the loop contains any store. For example, readonly calls
|
|
|
|
// with deopt states have this form and create a general alias set with all
|
|
|
|
// loads and stores. In order to get any LICM in loops containing possible
|
|
|
|
// deopt states we need a more precise invalidation of checking the mod ref
|
|
|
|
// info of each instruction within the loop and LI. This has a complexity of
|
|
|
|
// O(N^2), so currently, it is used only as a diagnostic tool since the
|
|
|
|
// default value of LICMN2Threshold is zero.
|
|
|
|
|
|
|
|
// Don't look at nested loops.
|
|
|
|
if (CurLoop->begin() != CurLoop->end())
|
|
|
|
return true;
|
|
|
|
|
|
|
|
int N = 0;
|
|
|
|
for (BasicBlock *BB : CurLoop->getBlocks())
|
|
|
|
for (Instruction &I : *BB) {
|
|
|
|
if (N >= LICMN2Theshold) {
|
|
|
|
LLVM_DEBUG(dbgs() << "Alasing N2 threshold exhausted for "
|
|
|
|
<< *(MemLoc.Ptr) << "\n");
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
N++;
|
|
|
|
auto Res = AA->getModRefInfo(&I, MemLoc);
|
|
|
|
if (isModSet(Res)) {
|
|
|
|
LLVM_DEBUG(dbgs() << "Aliasing failed on " << I << " for "
|
|
|
|
<< *(MemLoc.Ptr) << "\n");
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
LLVM_DEBUG(dbgs() << "Aliasing okay for " << *(MemLoc.Ptr) << "\n");
|
|
|
|
return false;
|
2015-02-22 19:35:32 +01:00
|
|
|
}
|
|
|
|
|
2019-01-10 20:29:04 +01:00
|
|
|
static bool pointerInvalidatedByLoopWithMSSA(MemorySSA *MSSA, MemoryUse *MU,
|
2019-02-13 00:05:40 +01:00
|
|
|
Loop *CurLoop,
|
2019-04-19 19:46:50 +02:00
|
|
|
SinkAndHoistLICMFlags &Flags) {
|
2019-06-20 23:09:09 +02:00
|
|
|
// For hoisting, use the walker to determine safety
|
|
|
|
if (!Flags.IsSink) {
|
|
|
|
MemoryAccess *Source;
|
|
|
|
// See declaration of SetLicmMssaOptCap for usage details.
|
|
|
|
if (Flags.LicmMssaOptCounter >= Flags.LicmMssaOptCap)
|
|
|
|
Source = MU->getDefiningAccess();
|
|
|
|
else {
|
|
|
|
Source = MSSA->getSkipSelfWalker()->getClobberingMemoryAccess(MU);
|
|
|
|
Flags.LicmMssaOptCounter++;
|
|
|
|
}
|
|
|
|
return !MSSA->isLiveOnEntryDef(Source) &&
|
|
|
|
CurLoop->contains(Source->getBlock());
|
2019-02-13 00:05:40 +01:00
|
|
|
}
|
2019-06-20 23:09:09 +02:00
|
|
|
|
|
|
|
// For sinking, we'd need to check all Defs below this use. The getClobbering
|
|
|
|
// call will look on the backedge of the loop, but will check aliasing with
|
|
|
|
// the instructions on the previous iteration.
|
|
|
|
// For example:
|
|
|
|
// for (i ... )
|
|
|
|
// load a[i] ( Use (LoE)
|
|
|
|
// store a[i] ( 1 = Def (2), with 2 = Phi for the loop.
|
|
|
|
// i++;
|
|
|
|
// The load sees no clobbering inside the loop, as the backedge alias check
|
|
|
|
// does phi translation, and will check aliasing against store a[i-1].
|
|
|
|
// However sinking the load outside the loop, below the store is incorrect.
|
|
|
|
|
|
|
|
// For now, only sink if there are no Defs in the loop, and the existing ones
|
|
|
|
// precede the use and are in the same block.
|
|
|
|
// FIXME: Increase precision: Safe to sink if Use post dominates the Def;
|
|
|
|
// needs PostDominatorTreeAnalysis.
|
|
|
|
// FIXME: More precise: no Defs that alias this Use.
|
|
|
|
if (Flags.NoOfMemAccTooLarge)
|
|
|
|
return true;
|
|
|
|
for (auto *BB : CurLoop->getBlocks())
|
|
|
|
if (auto *Accesses = MSSA->getBlockDefs(BB))
|
|
|
|
for (const auto &MA : *Accesses)
|
|
|
|
if (const auto *MD = dyn_cast<MemoryDef>(&MA))
|
|
|
|
if (MU->getBlock() != MD->getBlock() ||
|
|
|
|
!MSSA->locallyDominates(MD, MU))
|
|
|
|
return true;
|
|
|
|
return false;
|
2019-01-10 20:29:04 +01:00
|
|
|
}
|
|
|
|
|
2015-02-22 19:35:32 +01:00
|
|
|
/// 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;
|
|
|
|
}
|