On 17 March 2013 19:10, Andrew Turner <[email protected]> wrote:

> > You're changing both EABI and GNUEABI but only testing one of them.
> I can add a test for EABI but only GNUEABI will be supported on FreeBSD.
>

It's weird, I agree. It should be safe to print version 5 on Clang's
binaries, since every new release should be backwards compatible, as is 5
with the previous.


How? When using an external assembler the the ABI version is sever
>
passed to it so will be 0.
>

-Wa?

Anyhow, I think I agree with Anton that the patch is harmless for most
cases and beneficial to your case. I can't think of a way where it would be
bad or break current code, maybe I was just being over cautious.

Feel free to commit.

cheers,
--renato
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to