[PATCH] D28922: [LoadCombine] Fix combining of loads which span an aliasing store.

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 22:43:11 PST 2017


uabelho added a comment.

In https://reviews.llvm.org/D28922#653261, @davide wrote:

> In https://reviews.llvm.org/D28922#653006, @uabelho wrote:
>
> > I don't know why but this patch seems to also make https://llvm.org/bugs/show_bug.cgi?id=31493 stop crashing.
>
>
> Can you please also try with the code in https://reviews.llvm.org/D21496 ? IIRC it was ready modulo tests.


Ok, PR31493 also stops crashing if applying only https://reviews.llvm.org/D21496.


https://reviews.llvm.org/D28922





More information about the llvm-commits mailing list