On Tue, 15 Jul 2025, Uros Bizjak wrote: > LGTM for content, but let's ask Gerald to proofread the entry.
Happy to! + <li>The <code>--enable-x86-64-mfentry</code> configure option is + added to enable <code>-mfentry</code> for x86-64 by default to use + <code>__fentry__</code>, instead of <code>mcount</code> for + profiling. This option is enabled by default for glibc targets. + </li> This feels a bit complex to parse. How about something like + <li>The new <code>--enable-x86-64-mfentry</code> configure option + makes <code>-mfentry</code> use <code>__fentry__</code> instead + of <code>mcount</code> for profiling on x86-64. This option is + enabled by default for glibc targets. + </li> This replaces "option is added to enable" by "new option makes", fixes grammar/word order and drops what feels like an extra "by default". If the latter is wrong, add "by default" before "for profiling on x86-64". What do you think? (If you like it, go ahead and push this new version.) Gerald
