Re: [EXTERNAL] Re: [PATCH] Set discriminators for call stmts on the same line within the same basic block

2022-10-10 Thread Eugene Rozenfeld via Gcc-patches
I sent a patch that fixes a bug introduced by this patch: https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603203.html What you are seeing could have been caused by the same bug since it involves an uninitialized variable. Eugene On Oct 10, 2022, at 5:54 PM, David Edelsohn wrote:  Th

Re: [PATCH] Set discriminators for call stmts on the same line within the same basic block

2022-10-10 Thread David Edelsohn via Gcc-patches
This patch causes a bootstrap comparison failure on AIX. It apparently does not cause a failure on PPC64BE Linux with the same ABI, so I suspect that the failure may be related to the way that function aliases are implemented on AIX, which doesn't have ELF symbol alias semantics. "This change wil

Re: [PATCH] Set discriminators for call stmts on the same line within the same basic block

2022-10-06 Thread Jason Merrill via Gcc-patches
On 10/6/22 23:50, Eugene Rozenfeld wrote: Thank you for the review Jason. I fixed formatting and updated the commit description: OK. Call statements are possible split points of a basic block so they may end up in different basic blocks by the time pass_ipa_auto

Re: [PATCH] Set discriminators for call stmts on the same line within the same basic block

2022-10-06 Thread Eugene Rozenfeld via Gcc-patches
21 PM To: Eugene Rozenfeld ; gcc-patches@gcc.gnu.org Subject: [EXTERNAL] Re: [PATCH] Set discriminators for call stmts on the same line within the same basic block On 10/3/22 02:08, Eugene Rozenfeld wrote: > This change is based on commit > 1e6c4a7a8fb8e20545bb9f9032d3854f3f794c18 > by Deha

Re: [PATCH] Set discriminators for call stmts on the same line within the same basic block

2022-10-04 Thread Jason Merrill via Gcc-patches
On 10/3/22 02:08, Eugene Rozenfeld wrote: This change is based on commit 1e6c4a7a8fb8e20545bb9f9032d3854f3f794c18 by Dehao Chen in vendors/google/heads/gcc-4_8. Tested on x86_64-pc-linux-gnu. Brief rationale for the change? gcc/ChangeLog: * tree-cfg.cc (assign_discriminators): Set d

[PATCH] Set discriminators for call stmts on the same line within the same basic block

2022-10-02 Thread Eugene Rozenfeld via Gcc-patches
This change is based on commit 1e6c4a7a8fb8e20545bb9f9032d3854f3f794c18 by Dehao Chen in vendors/google/heads/gcc-4_8. Tested on x86_64-pc-linux-gnu. gcc/ChangeLog: * tree-cfg.cc (assign_discriminators): Set discriminators for call stmts on the same line within the same basic bloc