[PATCH 2/4] PowerPC: Rename functions for min, max, cmove
Segher Boessenkool
segher@kernel.crashing.org
Thu Sep 10 19:18:49 GMT 2020
On Thu, Aug 27, 2020 at 03:47:15PM -0500, will schmidt wrote:
> On Wed, 2020-08-26 at 22:44 -0400, Michael Meissner via Gcc-patches wrote:
> > * config/rs6000/rs6000.c (rs6000_emit_p9_fp_minmax): Rename to
> > maybe_emit_fp_c_minmax.
> ok
>
> > (maybe_emit_fp_c_minmax): Rename rs6000_emit_p9_fp_minmax. Return
> > bool instead of int.
>
> function rename is redundant between the two entries?
Yes, there are cleaner ways to write this, while still keeping both the
old and new names on the lhs of the colon (where they should be!)
Without naming each thing twice, even.
Segher
More information about the Gcc-patches
mailing list