[Buildroot] [PATCH v2] xtensa: add config option to enable longcalls option

Chris Zankel chris at zankel.net
Tue Dec 4 20:40:51 UTC 2012


Hi,

On 12/04/2012 12:13 PM, Peter Korsgaard wrote:
>>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at free-electrons.com> writes:
> Hi,
>
>  >> It should actually be fine to always compile with the longcalls option, 
>  >> but I didn't just want to change it without having the option to keep 
>  >> the old behavior, hence the option.
>
>  Thomas> I'd prefer to have it always enabled, I'd say. We try to not add
>  Thomas> gazillions of very detailed architecture-specific options that are
>  Thomas> hard to understand. Hardcore users will know how/where to remove this
>  Thomas> option is they need, or they can even override it by passing a
>  Thomas> -mno-longcalls option in BR2_TARGET_OPTIMIZATION.
>
>  Thomas> What do you think?
>
> I agree. Lets use the safe option by default.
>
Not sure what you mean with the 'safe option'? I'm also not sure what
the preference for options is when you pass both, the -mlongcalls and
-mno-longcalls option at the same time?

Anyway, I'm fine with whatever solution you come up with, as long as it
makes it into the tree :-)

Thanks,
-Chris



More information about the buildroot mailing list