[cfe-dev] [PATCH] clang/MIPS additional command-line options

[Eric Christopher]

On Feb 17, 2010, at 12:25 AM, Oleksandr Tymoshenko wrote:

> 
> On 2010-02-16, at 8:09 PM, Eric Christopher wrote:
> 
>> 
>> On Feb 16, 2010, at 7:44 PM, Oleksandr Tymoshenko wrote:
>> 
>>> 
>>> On 2010-02-16, at 7:34 PM, Oleksandr Tymoshenko wrote:
>>> 
>>>> I was trying to test clang on FReeBSD/mips and came to conclusion that some additional
>>>> keys required to set proper target for compilation. This patch adds several knobs to clang/MIPS:
>>>> - floating point ABI (-msoft-float/-mhard-float)
>>>> - target CPU (-march=)
>>>> - target ABI (-mabi=)
>>>> 
>>>> Also byte order is set explicitly for toolchain assembler in based on target architecture 
>>>> (mips/mipsel) so we don't rely on assembler default.
>>> 
>>> 
>>> Shame on me. Forgot to attach the patch
>>> <clang-mips.diff>_________________
>> 
>> The semantics mostly look ok to me. I'm curious about choosing soft-float as a default?
> 
>    Most MIPS32 target I've been working with were FPU-less. My exposure to this type of 
> hardware is not that wide, after all, so default value for floating point ABI is open for discussion. 

Hunh. Whereas almost all of the ones I've dealt with have FPUs.  I don't care enough to hold it up on that basis though :)

I'll test the patch and commit if everything looks good.

Thanks!

-eric