[PATCH] D46600: [MergedLoadStoreMotion] Fix a debug invariant bug in mergeStores

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope created this revision.
bjope added reviewers: Gerolf, davide, majnemer.
Herald added subscribers: JDevlieghere, aprantl.

MergedLoadStoreMotion::mergeStores is using some heuristics
to limit the amount of stores that it tries to sink (see
MagicCompileTimeControl in MergedLoadStoreMotion.cpp). The
heuristic involves counting the number of instructions in
one of the basic blocks that is part of the transformation.

We now ignore dbg intrinsics when counting instruction for
the MagicCompileTimeControl heuristic. This to make sure that
the amount of stores that are sunk doesn't depend on the amount
of debug information (if -g is used or not).


Repository:
  rL LLVM

https://reviews.llvm.org/D46600

Files:
  lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
  test/Transforms/InstMerge/st_sink_debuginvariant.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46600.145772.patch
Type: text/x-patch
Size: 12062 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180508/ea26d476/attachment.bin>