[llvm] r213396 - MergedLoadStoreMotion pass
ghoflehner at apple.com
Sun Jul 20 20:25:51 PDT 2014
Thanks for test case + analysis! Fixed with r213497.
On Jul 20, 2014, at 6:49 PM, Sanjay Patel <spatel at rotateright.com> wrote:
> I just posted a somewhat reduced test case in:
> On Sun, Jul 20, 2014 at 7:36 PM, Renato Golin <renato.golin at linaro.org> wrote:
> On 18 July 2014 20:13, Gerolf Hoflehner <ghoflehner at apple.com> wrote:
> > Author: ghoflehner
> > Date: Fri Jul 18 14:13:09 2014
> > New Revision: 213396
> > URL: http://llvm.org/viewvc/llvm-project?rev=213396&view=rev
> > Log:
> > MergedLoadStoreMotion pass
> > Merges equivalent loads on both sides of a hammock/diamond
> > and hoists into into the header.
> > Merges equivalent stores on both sides of a hammock/diamond
> > and sinks it to the footer.
> > Can enable if conversion and tolerate better load misses
> > and store operand latencies.
> Hi Gerolf,
> This commit broke the test-suite on ARM.
> I just bisected the commits between:
> And, although another commit broke the build, yours silently
> introduced a failure in MultiSource/Applications/obsequi.
> I'm trying to reduce the problem, but given that this is the release
> week, if we can't find it before 3.5 is branched, I think we better
> revert your commit until the release is gone or we can find a fix for
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> Sanjay Patel
> RotateRight, LLC
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits