[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
Sat Jan 28 15:16:10 PST 2017


davide added a comment.

This is definitely a stop-gap solution to the problem, and although we seem to have a more general patch already available, you don't have time/don't want to work on it.
As this fixes a miscompile and a crash reported upstream, I'm inclined to see this patch or a modified form of it upstream. River can land his version once he starts working on LLVM again (which should, presumably, happen soon). The reason why I'd like to see pushing forward is that we can backport the "fix" to the 4.0 branch. Eli, WDYT?


https://reviews.llvm.org/D28922





More information about the llvm-commits mailing list