[llvm] r213396 - MergedLoadStoreMotion pass
renato.golin at linaro.org
Sun Jul 20 23:31:13 PDT 2014
This fixes the bot, too. Thanks!
On 21 Jul 2014 04:25, "Gerolf Hoflehner" <ghoflehner at apple.com> wrote:
> 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>
>> 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