[llvm] r213396 - MergedLoadStoreMotion pass

Renato Golin renato.golin at linaro.org
Sun Jul 20 18:36:43 PDT 2014


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



More information about the llvm-commits mailing list