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

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 12:59:28 PST 2017


davide added a comment.

In https://reviews.llvm.org/D28922#660788, @efriedma wrote:

> Why do you want to backport this to 4.0?  LoadCombine isn't part of the default pass pipeline.


While the pass is not part of the default pipeline, some people use it (and apparently breaks).
That said, on a second thought, I decide to withdraw my request to backport this as it doesn't seem there's a reason strong enough to do that.
Also, the crash has been there since forever as far as I can tell, in fact I can reproduce it pre 3.9.


https://reviews.llvm.org/D28922





More information about the llvm-commits mailing list