On Jun 13, 2024, Hongyu Wang <wwwhhhyyy...@gmail.com> wrote: > I think the function name can be like ix86_unroll_flag_adjust instead > of ix86_override_options_after_change_1, like the previous 2 functions > which declares the usage more clearly.
I'd be happy to rename it, but we have a long-established convention of using _<number> suffixes for functions that are wrapped or otherwise formerly part of another, and that would likely be immediately recognizable by GCC developers, whereas there seems to be a bit of an organically evolutionary mess of recompute/override/adjust/tweak functions in this general area, so I figured we'd be better off sticking to a name that more clearly refers to the target hook. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer More tolerance and less prejudice are key for inclusion and diversity Excluding neuro-others for not behaving ""normal"" is *not* inclusive
