[PATCH] D43177: Mark MergedLoadStoreMotion as not preserving MemDep results

[Daniel Berlin via llvm-commits]

FWIW: This same problem applies to memcpyopt as well, depending on
placement of the new instructions.



On Mon, Feb 12, 2018 at 3:04 AM, Björn Steinbrink via Phabricator via
llvm-commits <llvm-commits at lists.llvm.org> wrote:

> dotdash created this revision.
> dotdash added reviewers: davide, Gerolf.
>
> MemDep caches results that signify that a dependence is non-local, and
> there is currently no way to invalidate such cache entries.
> Unfortunately, when MLSM sinks a store that can result in a non-local
> dependence becoming a local one, and then MemDep gives wrong answers.
> The easiest way out here is to just say that MLSM does indeed not
> preserve MemDep results.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D43177
>
> Files:
>   lib/Transforms/Scalar/MergedLoadStoreMotion.cpp
>   test/Transforms/GVN/pr36063.ll
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180330/cb2df0a2/attachment.html>