[RFC] make arm-use-movt and arm-reserve-r9 options available for all arm

Renato Golin renato.golin at linaro.org
Mon Aug 12 13:13:47 PDT 2013


On 12 August 2013 20:30, Jeroen Hofstee <llvm at myspectrum.nl> wrote:

>  Don't worry nor fear, see
> http://lists.denx.de/pipermail/u-boot/2013-August/160604.html
>

Good!


Yes, that is why proposed to send both patches through llvm-commits, so it
> can be applied at the same time to keep the bots and users happy.
>

You can't commit *at the same time*, and it's possible that some bots will
fail if they get one and not the other. It'll also be harder to revert the
patch in case there is something wrong in the future. But it's not the end
of the world. Just remember to add comments on both commits pointing to
each other (with maybe the same header on both Clang and LLVM, to make it
easier to revert).


I did on IRC and is on cc. And besides the rename of the hidden flag,
> nothing changed for IOS.
>

I'm glad Jim checked it. I'm happy with the change, too.


make check-all output:
>
> Testing Time: 40.08s
>   Expected Passes    : 13181
>   Expected Failures  : 53
>   Unsupported Tests  : 1724
>

Excellent!


mmm, this is a bit more difficult since the target has no compiler and only
> 512Mb flash.
>

Ok, not to worry, that's why we've got buildbots for. Since this is a small
change, I think you can go ahead and commit, and let's see what happens
with the test-suite bot. I'm not expecting much, to be honest...

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130812/3ef8e4d8/attachment.html>


More information about the llvm-commits mailing list