[PATCH] D30703: [DSE] Merge stores when the later store only writes to memory locations the early store also wrote to.
Filipe Cabecinhas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 25 09:55:20 PDT 2017
filcab added a comment.
In https://reviews.llvm.org/D30703#880203, @spatel wrote:
> Patch updated:
>
> 1. Break from the inner loop after we successfully merge (kill) the earlier and later stores.
> 2. Add test reduced from PR34074 that was crashing.
Thanks for comandeering and fixing it, Sanjay!
Your fix looks good on my end.
https://reviews.llvm.org/D30703
More information about the llvm-commits
mailing list