"H.J. Lu" <hjl.to...@gmail.com> writes:
> On Sat, Jul 12, 2025 at 6:58 AM Siddhesh Poyarekar <siddh...@gotplt.org>
> wrote:
>>
>> On 2025-07-11 15:28, Uros Bizjak wrote:
>> >> 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.
>>
>> Oh that's good then.
>>
>> Thanks,
>> Sid
>
> Here is the v2 patch. The differences are
>
> 1. Enable -mfentry by default for i?86-*-*gnu* | x86_64-*-*gnu*,
> not i?86-*-*linux* | x86_64-*-*linux*.
> 2. Adjust some testcases.
>
> OK for master?
I'll test it tonight but it looks good to me now. Thanks.
> [...]
sam
