Re: [PATCH] Improve DOM's optimization of control statements

2015-10-07 Thread Jeff Law
On 10/07/2015 04:03 PM, Andreas Schwab wrote: On powerpc: FAIL: gcc.c-torture/compile/pr52073.c -O2 (internal compiler error) /daten/gcc/gcc-20151006/gcc/testsuite/gcc.c-torture/compile/pr52073.c:6:1: internal compiler error: in duplicate_thread_path, at tree-ssa-threadupdate.c:2446. 0x108

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-07 Thread Andreas Schwab
On powerpc: FAIL: gcc.c-torture/compile/pr52073.c -O2 (internal compiler error) /daten/gcc/gcc-20151006/gcc/testsuite/gcc.c-torture/compile/pr52073.c:6:1: internal compiler error: in duplicate_thread_path, at tree-ssa-threadupdate.c:2446. 0x10874ab7 duplicate_thread_path. ../../gcc/t

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-07 Thread Jeff Law
the trunk. jeff commit 992d281b2d1ba53a49198db44fee92a505e16f5d Author: Jeff Law Date: Fri Oct 2 15:22:04 2015 -0400 Re: [PATCH] Improve DOM's optimization of control statements * tree-ssa-dom.c (optimize_stmt): Note when loop structures need fixups. diff --git a/gcc/ChangeLog b/gcc/Cha

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-06 Thread Jeff Law
On 10/05/2015 03:02 AM, Richard Biener wrote: + /* If BB is in a loop, then removing an outgoing edge from BB +may cause BB to move outside the loop, changes in the +loop exit edges, etc. So note that loops need fixing. */ + if (bb_loop_de

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-05 Thread Richard Biener
sion and did a bootstrap & > regression test on x86_64-linux-gnu. > > Installed on the trunk. > > jeff > > commit 992d281b2d1ba53a49198db44fee92a505e16f5d > Author: Jeff Law > Date: Fri Oct 2 15:22:04 2015 -0400 > > Re: [PATCH] Improve DOM's optimization of control state

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-02 Thread Jeff Law
15 -0400 Re: [PATCH] Improve DOM's optimization of control statements * tree-ssa-dom.c (optimize_stmt): Note when loop structures need fixups. diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 3f7561a..e541df3 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-02 Thread Jeff Law
On 10/02/2015 05:15 AM, Renlin Li wrote: Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling the test case with '-Os' flag. A quick check shows, the cfg has been changed, but the loop informat

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-02 Thread Jeff Law
On 10/02/2015 05:15 AM, Renlin Li wrote: Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling the test case with '-Os' flag. A quick check shows, the cfg has been changed, but the loop informat

Re: [PATCH] Improve DOM's optimization of control statements

2015-10-02 Thread Renlin Li
Hi Jeff, Your patch causes an ICE regression. The test case is " gcc.c-torture/compile/pr27087.c", I observed it on aarch64-none-elf target when compiling the test case with '-Os' flag. A quick check shows, the cfg has been changed, but the loop information is not updated. Thus the informatio

[PATCH] Improve DOM's optimization of control statements

2015-09-30 Thread Jeff Law
Until now DOM has had to be very conservative with handling control statements with known conditions. This as been an unfortunate side effect of the interaction between removing edges and recycling names via the SSA_NAME manager. Essentially DOM would have to leave control statements alone.