<p dir="ltr">This fixes the bot, too. Thanks! </p>
<p dir="ltr">Cheers, <br>
Renato </p>
<div class="gmail_quote">On 21 Jul 2014 04:25, "Gerolf Hoflehner" <<a href="mailto:ghoflehner@apple.com">ghoflehner@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">Thanks for test case + analysis! Fixed with r213497.<div><br></div><div>Gerolf</div><div><br><div><div>On Jul 20, 2014, at 6:49 PM, Sanjay Patel <<a href="mailto:spatel@rotateright.com" target="_blank">spatel@rotateright.com</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr">I just posted a somewhat reduced test case in:<br><a href="http://llvm.org/bugs/show_bug.cgi?id=20369" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=20369</a><br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">
On Sun, Jul 20, 2014 at 7:36 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On 18 July 2014 20:13, Gerolf Hoflehner <<a href="mailto:ghoflehner@apple.com" target="_blank">ghoflehner@apple.com</a>> wrote:<br>
> Author: ghoflehner<br>
> Date: Fri Jul 18 14:13:09 2014<br>
> New Revision: 213396<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=213396&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=213396&view=rev</a><br>
> Log:<br>
> MergedLoadStoreMotion pass<br>
><br>
> Merges equivalent loads on both sides of a hammock/diamond<br>
> and hoists into into the header.<br>
> Merges equivalent stores on both sides of a hammock/diamond<br>
> and sinks it to the footer.<br>
> Can enable if conversion and tolerate better load misses<br>
> and store operand latencies.<br>
<br>
</div>Hi Gerolf,<br>
<br>
This commit broke the test-suite on ARM.<br>
<br>
I just bisected the commits between:<br>
<a href="http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/7492" target="_blank">http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/7492</a><br>
<a href="http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/7493" target="_blank">http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/7493</a><br>
<br>
And, although another commit broke the build, yours silently<br>
introduced a failure in MultiSource/Applications/obsequi.<br>
<br>
I'm trying to reduce the problem, but given that this is the release<br>
week, if we can't find it before 3.5 is branched, I think we better<br>
revert your commit until the release is gone or we can find a fix for<br>
it..<br>
<br>
cheers,<br>
--renato<br>
<div><div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Sanjay Patel<br>RotateRight, LLC<br><a href="http://www.rotateright.com/" target="_blank">http://www.rotateright.com</a>
</div>
</blockquote></div><br></div></div>
</blockquote></div>