On 29/09/26 1:53 PM, Avinash Jayakar wrote:
> Hi,
>
> The following patch has been bootstrapped and regtested with no failures. I
> also built with the configure option used in PR127832 as mentioned in commit
> message. Ok for trunk?
>
> Thanks,
> Avinash
>
> The bootstrap build with
Please format correctly.
> configure --enable-checking=release --enable-languages=c,c++,fortran
> failed since the maybe-uninitialized pass reported the size variable to
s/size/'size'
> be unitialized before using it as false positive. This case is a false
Just 'be unitialized before using it'. Please remove 'as false positive'.
> positive because the size variable is accessed only from the second
> iteration in the loop at which time size would have been initialized.
s/size/'size'
>
> This patch initializes this variable to 0 to get rid of this warning.
>
> 2026-09-29 Avinash Jayakar <[email protected]>
>
> gcc/ChangeLog:
> PR target/127632
> * config/rs6000/rs6000-c.cc (altivec_resolve_overloaded_builtin):
> Initialize
line longer than 80 chars.
LGTM with the above changes.
-Surya
> size to 0.
> ---
> gcc/config/rs6000/rs6000-c.cc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/rs6000/rs6000-c.cc b/gcc/config/rs6000/rs6000-c.cc
> index ecc3ff4e24e..f51593e0455 100644
> --- a/gcc/config/rs6000/rs6000-c.cc
> +++ b/gcc/config/rs6000/rs6000-c.cc
> @@ -1734,7 +1734,7 @@ altivec_resolve_overloaded_builtin (location_t loc,
> tree fndecl,
> error_at (loc, "%qE requires 6 arguments", fndecl);
> return error_mark_node;
> }
> - unsigned HOST_WIDE_INT size;
> + unsigned HOST_WIDE_INT size = 0;
> for (int i=0; i<3; i++) {
> /* Get the first argument to determine the actual type. */
> tree arg0 = (*arglist)[i];
