LGTM
On Wed, May 21, 2025 at 9:49 PM Robin Dapp <rdapp....@gmail.com> wrote:
>
> Hi,
>
> this patch initializes saved_vxrm_mode to VXRM_MODE_NONE. This is a
> warning (but no error) when building the compiler so better fix it.
>
> Regtested on rv64gcv_zvl512b. Going to commit as obvious if the CI
> is happy.
>
> Regards
> Robin
>
> gcc/ChangeLog:
>
> * config/riscv/riscv.cc (singleton_vxrm_need): Init
> saved_vxrm_mode.
> ---
> gcc/config/riscv/riscv.cc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc
> index 0b10842d176..8849b2c05c1 100644
> --- a/gcc/config/riscv/riscv.cc
> +++ b/gcc/config/riscv/riscv.cc
> @@ -12298,7 +12298,7 @@ singleton_vxrm_need (void)
> /* Walk the IL noting if VXRM is needed and if there's more than one
> mode needed. */
> bool found = false;
> - int saved_vxrm_mode;
> + int saved_vxrm_mode = VXRM_MODE_NONE;
> for (rtx_insn *insn = get_insns (); insn; insn = NEXT_INSN (insn))
> {
> if (!INSN_P (insn) || DEBUG_INSN_P (insn))
> --
> 2.49.0
>
