Re: Fwd: [V2][PATCH] Fixing PR107411

2023-02-28 Thread Richard Biener via Gcc-patches
On Mon, 27 Feb 2023, Qing Zhao wrote: > Ping. OK. Thanks, Richard. > Qing > > Begin forwarded message: > > From: Qing Zhao mailto:qing.z...@oracle.com>> > Subject: [V2][PATCH] Fixing PR107411 > Date: February 21, 2023 at 9:46:04 AM EST > To: ja...@r

Fwd: [V2][PATCH] Fixing PR107411

2023-02-27 Thread Qing Zhao via Gcc-patches
Ping. Qing Begin forwarded message: From: Qing Zhao mailto:qing.z...@oracle.com>> Subject: [V2][PATCH] Fixing PR107411 Date: February 21, 2023 at 9:46:04 AM EST To: ja...@redhat.com<mailto:ja...@redhat.com>, rguent...@suse.de<mailto:rguent...@suse.de> Cc: gcc-patches@gcc.

[V2][PATCH] Fixing PR107411

2023-02-21 Thread Qing Zhao via Gcc-patches
This is the 2nd version of the patch. compared to the first version, the major change is: use sprintf to replace xasprintf per Jacub's suggestion. bootstrapped and regression tested on both x86 and aarch64. Okay for committing? thanks. Qing === This is a bug in

Re: [PATCH] Fixing PR107411

2023-02-20 Thread Qing Zhao via Gcc-patches
> On Feb 20, 2023, at 10:17 AM, Jakub Jelinek wrote: > > On Mon, Feb 20, 2023 at 03:04:51PM +, Qing Zhao via Gcc-patches wrote: >> >> >>> On Feb 17, 2023, at 5:35 PM, Jakub Jelinek wrote: >>> >>> On Fri, Feb 17, 2023 at 10:26:03PM +, Qing Zhao via Gcc-patches wrote: +

Re: [PATCH] Fixing PR107411

2023-02-20 Thread Jakub Jelinek via Gcc-patches
On Mon, Feb 20, 2023 at 03:04:51PM +, Qing Zhao via Gcc-patches wrote: > > > > On Feb 17, 2023, at 5:35 PM, Jakub Jelinek wrote: > > > > On Fri, Feb 17, 2023 at 10:26:03PM +, Qing Zhao via Gcc-patches wrote: > >> +else if (!DECL_NAME (lhs_var)) > >> + { > >> +

Re: [PATCH] Fixing PR107411

2023-02-20 Thread Qing Zhao via Gcc-patches
> On Feb 17, 2023, at 5:35 PM, Jakub Jelinek wrote: > > On Fri, Feb 17, 2023 at 10:26:03PM +, Qing Zhao via Gcc-patches wrote: >> + else if (!DECL_NAME (lhs_var)) >> +{ >> + char *lhs_var_name_str >> += xasprintf ("D.%u", DECL_UID

Re: [PATCH] Fixing PR107411

2023-02-17 Thread Jakub Jelinek via Gcc-patches
On Fri, Feb 17, 2023 at 10:26:03PM +, 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

[PATCH] Fixing PR107411

2023-02-17 Thread Qing Zhao via Gcc-patches
This is a bug in tree-ssa-uninit.cc. When doing the following: /* Ignore the call to .DEFERRED_INIT that define the original var itself as the following case: temp = .DEFERRED_INIT (4, 2, “alt_reloc"); alt_reloc = temp; In order to avoid generating warning for the fake