Re: [PATCH] rs6000: Fix formatting of -mfuture warning [PR126738]

LGTM. Okay for trunk, and also backporting to GCC 16 after allowing it to bake in the trunk
for a while.

-Surya

On 10/08/26 1:14 PM, Kishan Parmar wrote:
> Hello,
> 
> The warning for the undocumented -mfuture option is missing the
> terminating '%' characters in the option markup, resulting in malformed
> diagnostics.
> 
> Add the missing '%' characters so that -mfuture and -mcpu=future are
> properly quoted in the warning message.
> 
> 2026-08-10  Kishan Parmar  <[email protected]>
> 	    Göran Uddeborg <[email protected]>
> 
> gcc/ChangeLog:
> 	* config/rs6000/rs6000.opt: Fix formatting of the -mfuture warning.
> 
> Thank You,
> Kishan
> 
> ---
>  gcc/config/rs6000/rs6000.opt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gcc/config/rs6000/rs6000.opt b/gcc/config/rs6000/rs6000.opt
> index d9e9de92b21..8632e106458 100644
> --- a/gcc/config/rs6000/rs6000.opt
> +++ b/gcc/config/rs6000/rs6000.opt
> @@ -597,7 +597,7 @@ Target Undocumented Mask(POWER11) Var(rs6000_isa_flags) WarnRemoved
>  
>  ;; Potential future machine
>  mfuture
> -Target Undocumented Mask(FUTURE) Var(rs6000_isa_flags) Warn(Do not use %<-mfuture>, use %<-mcpu=future>)
> +Target Undocumented Mask(FUTURE) Var(rs6000_isa_flags) Warn(Do not use %<-mfuture%>, use %<-mcpu=future%>)
>  
>  mdense-math
>  Target Mask(DMF) Var(rs6000_isa_flags)