On Fri, Feb 17, 2023 at 10:26:03PM +0000, Qing Zhao via Gcc-patches wrote:
> + else if (!DECL_NAME (lhs_var))
> + {
> + char *lhs_var_name_str
> + = xasprintf ("D.%u", DECL_UID (lhs_var));
Why xasprintf? D.%u can be sprintfed into a fixed size automatic buffer,
say 3 + (HOST_BITS_PER_INT + 2) / 3 would be a good upper bound for the size
of the buffer. Then you don't need to free it...
> + if (strcmp (lhs_var_name_str, var_name_str) == 0)
> + {
> + free (lhs_var_name_str);
> + return;
> + }
> + free (lhs_var_name_str);
> + }
> + }
> gcc_assert (var_name_str && var_def_stmt);
> }
> }
> --
> 2.31.1
Jakub
