<div dir="ltr">Hi Stepan,<div><br></div><div style>Your patch broke a lot of tests on the ARM test-suite:</div><div style><br></div><div style><a href="http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/442">http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/442</a><br>
</div><div style><br></div><div style>And the fix didn't fix all of them:</div><div style><br></div><div style><a href="http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/443">http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/443</a><br>
</div><div style><br></div><div style>CoyoteBench was passing before you committed:</div><div style><br></div><div style><a href="http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/441">http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/441</a><br>
</div><div style><br></div><div style>I'm surprised that you managed to run the test-suite both times and not see those failures... </div><div style><br></div><div style>If you can't fix the broken test, I suggest we revert your patch and make sure you can run the test-suite and see the same errors as the one in the build, and make sure all the tests pass before committing. I followed the instructions from the website and it behaves exactly like the buildbot:</div>
<div style><br></div><div style><a href="http://llvm.org/docs/lnt/quickstart.html">http://llvm.org/docs/lnt/quickstart.html</a><br></div><div style><br></div><div style>cheers,</div><div style>--renato</div><div style><br>
</div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 5 April 2013 07:47, Stepan Dyatkovskiy <span dir="ltr"><<a href="mailto:stpworld@narod.ru" target="_blank">stpworld@narod.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Got failure on clang-native-arm-cortex-a9. Fixing...<span class="HOEnZb"><font color="#888888"><br>
-Stepan</font></span><div class="HOEnZb"><div class="h5"><br>
Stepan Dyatkovskiy wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Committed as r178851. The only regression was added. Difficult to guess<br>
code that reproduces this bug.<br>
-Stepan<br>
<br>
Stepan Dyatkovskiy wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
OK. I'll do. Currently I got some little trouble with<br>
test/CodeGen/ARM/stm.ll. Hope fix it soon.<br>
<br>
-Stepan<br>
<br>
Renato Golin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 3 April 2013 06:29, Stepan Dyatkovskiy <<a href="mailto:stpworld@narod.ru" target="_blank">stpworld@narod.ru</a><br>
<mailto:<a href="mailto:stpworld@narod.ru" target="_blank">stpworld@narod.ru</a>>> wrote:<br>
<br>
    Hello Renato,<br>
    Thanks. I'll run test-suite today, if everything would fine, I<br>
    commit it then.<br>
<br>
<br>
Can you also add a few tests, just to make sure it won't regress in the<br>
future?<br>
<br>
Thanks!<br>
--renato<br>
<br>
</blockquote>
<br>
______________________________<u></u>_________________<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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>
<br>
______________________________<u></u>_________________<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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>
<br>
</div></div></blockquote></div><br></div>