[llvm] r213396 - MergedLoadStoreMotion pass

Gerolf Hoflehner ghoflehner at apple.com
Sun Jul 20 20:25:51 PDT 2014


Thanks for test case + analysis! Fixed with r213497.

Gerolf

On Jul 20, 2014, at 6:49 PM, Sanjay Patel <spatel at rotateright.com> wrote:

> I just posted a somewhat reduced test case in:
> http://llvm.org/bugs/show_bug.cgi?id=20369
> 
> 
> 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:
> http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/7492
> http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/7493
> 
> 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
> it..
> 
> cheers,
> --renato
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 
> 
> 
> -- 
> Sanjay Patel
> RotateRight, LLC
> http://www.rotateright.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140720/b8044d0d/attachment.html>


More information about the llvm-commits mailing list