Re: [PATCH 4/N] Recover GOTO predictor.

2017-07-31 Thread David Edelsohn
This patch breaks bootstrap on AIX and probably is a hidden bug on all targets. With the patch, libbacktrace/xcoff.c fails to compile with the error: /nasfarm/edelsohn/src/src/libbacktrace/xcoff.c: In function 'xcoff_add': /nasfarm/edelsohn/src/src/libbacktrace/xcoff.c:822:13: error: 'incl' may

Re: [PATCH 4/N] Recover GOTO predictor.

2017-07-31 Thread Richard Biener
On Mon, Jul 31, 2017 at 9:46 AM, Martin Liška wrote: > Richi? 4 is fine. > Thanks > > On 06/30/2017 03:48 PM, Martin Liška wrote: >> On 06/22/2017 12:27 PM, Richard Biener wrote: >>> On Wed, Jun 21, 2017 at 3:06 PM, Martin Liška wrote: Hello.

Re: [PATCH 4/N] Recover GOTO predictor.

2017-07-31 Thread Martin Liška
Richi? Thanks On 06/30/2017 03:48 PM, Martin Liška wrote: > On 06/22/2017 12:27 PM, Richard Biener wrote: >> On Wed, Jun 21, 2017 at 3:06 PM, Martin Liška wrote: >>> Hello. >>> >>> There's one additional predictor enhancement that is GOTO predict that >>> used to working.

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-30 Thread Martin Liška
On 06/22/2017 12:27 PM, Richard Biener wrote: > On Wed, Jun 21, 2017 at 3:06 PM, Martin Liška wrote: >> Hello. >> >> There's one additional predictor enhancement that is GOTO predict that >> used to working. Following patch adds expect statement for C and C++ family >> languages.

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-30 Thread Jan Hubicka
> PING^1 > > Can you please Honza give a formal approval for the patch? OK, thanks! Honza > > Thanks, > Martin > > On 06/22/2017 01:47 PM, Richard Biener wrote: > > On Thu, Jun 22, 2017 at 12:57 PM, Martin Liška wrote: > >> On 06/22/2017 12:27 PM, Richard Biener wrote: > >>>

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-30 Thread Martin Liška
PING^1 Can you please Honza give a formal approval for the patch? Thanks, Martin On 06/22/2017 01:47 PM, Richard Biener wrote: > On Thu, Jun 22, 2017 at 12:57 PM, Martin Liška wrote: >> On 06/22/2017 12:27 PM, Richard Biener wrote: >>> On Wed, Jun 21, 2017 at 3:06 PM, Martin

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-22 Thread Richard Biener
On Thu, Jun 22, 2017 at 12:57 PM, Martin Liška wrote: > On 06/22/2017 12:27 PM, Richard Biener wrote: >> On Wed, Jun 21, 2017 at 3:06 PM, Martin Liška wrote: >>> Hello. >>> >>> There's one additional predictor enhancement that is GOTO predict that >>> used to

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-22 Thread Martin Liška
On 06/22/2017 12:27 PM, Richard Biener wrote: > On Wed, Jun 21, 2017 at 3:06 PM, Martin Liška wrote: >> Hello. >> >> There's one additional predictor enhancement that is GOTO predict that >> used to working. Following patch adds expect statement for C and C++ family >> languages.

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-22 Thread Richard Biener
On Wed, Jun 21, 2017 at 3:06 PM, Martin Liška wrote: > Hello. > > There's one additional predictor enhancement that is GOTO predict that > used to working. Following patch adds expect statement for C and C++ family > languages. > > There's one fallout which is vrp24.c test-case,

Re: [PATCH 4/N] Recover GOTO predictor.

2017-06-21 Thread Martin Liška
On 06/21/2017 03:06 PM, Martin Liška wrote: > Hello. > > There's one additional predictor enhancement that is GOTO predict that > used to working. Following patch adds expect statement for C and C++ family > languages. > > There's one fallout which is vrp24.c test-case, where only 'Simplified >

[PATCH 4/N] Recover GOTO predictor.

2017-06-21 Thread Martin Liška
Hello. There's one additional predictor enhancement that is GOTO predict that used to working. Following patch adds expect statement for C and C++ family languages. There's one fallout which is vrp24.c test-case, where only 'Simplified relational' appears just once. Adding Richi and Patrick who