[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