diff options
| author | Florian Hahn <florian.hahn@arm.com> | 2018-08-24 11:40:04 +0000 |
|---|---|---|
| committer | Florian Hahn <florian.hahn@arm.com> | 2018-08-24 11:40:04 +0000 |
| commit | 406f1ff1cdfe3ef065a3ac3414ff1672dcfb9049 (patch) | |
| tree | 6ace68331402e73d75694236d0ced20b8cab2940 /llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp | |
| parent | a98961bc841fe930b80593af2342fa0c7d28d4f8 (diff) | |
| download | bcm5719-llvm-406f1ff1cdfe3ef065a3ac3414ff1672dcfb9049.tar.gz bcm5719-llvm-406f1ff1cdfe3ef065a3ac3414ff1672dcfb9049.zip | |
[Local] Make DoesKMove required for combineMetadata.
This patch makes the DoesKMove argument non-optional, to force people
to think about it. Most cases where it is false are either code hoisting
or code sinking, where we pick one instruction from a set of
equal instructions among different code paths.
Reviewers: dberlin, nlopes, efriedma, davide
Reviewed By: efriedma
Differential Revision: https://reviews.llvm.org/D47475
llvm-svn: 340606
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp b/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp index 5f0931ead49..1ff7037c8d2 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp @@ -1026,7 +1026,7 @@ Instruction *InstCombiner::visitLoadInst(LoadInst &LI) { if (Value *AvailableVal = FindAvailableLoadedValue( &LI, LI.getParent(), BBI, DefMaxInstsToScan, AA, &IsLoadCSE)) { if (IsLoadCSE) - combineMetadataForCSE(cast<LoadInst>(AvailableVal), &LI); + combineMetadataForCSE(cast<LoadInst>(AvailableVal), &LI, false); return replaceInstUsesWith( LI, Builder.CreateBitOrPointerCast(AvailableVal, LI.getType(), |
