Uros Bizjak <ubiz...@gmail.com> writes:
> On Fri, Jul 11, 2025 at 2:33 PM Siddhesh Poyarekar <siddh...@gotplt.org>
> wrote:
>>
>> On 2025-07-08 18:07, Sam James wrote:
>> >> OK in principle, but please allow some time for distro maintainers
>> >> (CC'd) to voice their opinion.
>> >
>> > It looks good to me and I plan on us using it. I'd like opinions from
>> > one other group first before it goes in if possible though, as our
>> > perspective is different from others (e.g. we don't have to worry about
>> > old enterprise deployments).
>>
>> Why not just switch over unconditionally? __fentry__ seems like a
>> better alternative to mcount overall and it has been around long enough
>> that even older deployments should be relatively unaffected.
>
> Actually, it is switched on by default for i?86-*-linux* |
> x86_64-*-linux*. The default for --enable-x86-64-mfentry is "auto",
> which triggers the mentioned condition. One still has a chance to use
> "yes" or "no" in addition to "auto" when configuring with
> --{enable|disable}-x86-64-mfentry.
I think we need to conditionalise it on gnu too unless I'm missing
something.
>
> Uros.
