Re: [build] Have s-macro_list depend on cc1

2014-03-22 Thread Paolo Bonzini
Il 21/03/2014 18:05, Rainer Orth ha scritto: The following patch adds it and indeed macro_list now is non-empty, as expected. I'm just not sure if cc1 is the correct one in gcc/Makefile.in, or if it should rather be $(COMPILERS) instead. Anyway, with that patch a i386-pc-solaris2.10 bootstrap c

Re: [PATCH] Minor ipa-utils dumping fix

2014-03-22 Thread Richard Biener
On March 22, 2014 2:40:37 AM CET, Teresa Johnson wrote: >Minor dumping fix. >Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for stage 1? It's OK now. Thanks, Richard. >Thanks, >Teresa > >2014-03-21 Teresa Johnson > >* ipa-utils.c (ipa_print_order): Use specified dump file. >

Re: [PATCH] Fix two undefined behaviors in gcc sources

2014-03-22 Thread Richard Biener
On March 21, 2014 9:30:25 PM CET, Jakub Jelinek wrote: >Hi! > >--with-build-config=bootstrap-ubsan on i686-linux showed >most often these two issues. > >The first one is that 32-bit signed time_t is multiplied by 1000, which >overflows the int type. The result is then cast to unsigned, so even >o

Re: [PATCH] Fix ubsan expansion (PR sanitizer/60613)

2014-03-22 Thread Richard Biener
On March 21, 2014 9:32:54 PM CET, Jakub Jelinek wrote: >Hi! > >As MINUS_EXPR is not commutative, we really can't swap op0 with op1 >for testing whether subtraction overflowed, that is only possible for >PLUS_EXPR, for MINUS_EXPR we really have to know if op1 is constant >or negative or non-negativ

Re: [patch, libgfortran] Committed as obvious

2014-03-22 Thread Tobias Burnus
Can you add some documentation to gfortran.texi. I think we need to make clear that for library extension, the flag used for compiling the main program is relevant. Additionally, I believe some of the extensions are always on while others are only enabled with a flag - if so, I think we should

LTO / invoke.texi: Add some words about static libraries and LTO

2014-03-22 Thread Tobias Burnus
Mention gcc-ar, gcc-ranlib, gcc-nm in the documentation. OK? Tobias 2014-03-22 Tobias Burnus * doc/invoke.texi (-flto): Expand section about using static libraries with LTO. diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 4c183a3..78ddde0 100644 --- a/gcc/doc/invoke.texi +++

Re: [wwwdocs] Update list of 4.9 secondary platforms

2014-03-22 Thread Dave Anglin
On Thu, Mar 13, 2014 at 09:06:43AM -0600, Jeff Law wrote: > On 03/13/14 09:02, Jakub Jelinek wrote: > >Hi! > > > >I've been told that the SC has approved the secondary platform list changes, > >so I went ahead and committed the changes to our web pages. > Thanks. I should have taken care of this a

Re: [PING^6][PATCH] Add a couple of dialect and warning options regarding Objective-C instance variable scope

2014-03-22 Thread Dimitris Papavasiliou
Ping! On 03/13/2014 11:54 AM, Dimitris Papavasiliou wrote: Ping! On 03/06/2014 07:44 PM, Dimitris Papavasiliou wrote: Ping! On 02/27/2014 11:44 AM, Dimitris Papavasiliou wrote: Ping! On 02/20/2014 12:11 PM, Dimitris Papavasiliou wrote: Hello all, Pinging this patch review request again. S

Re: [PATCH] Update the overall summary after edge_summary is updated

2014-03-22 Thread Jan Hubicka
> Hi, > > This patch updates node's inline summary after edge_summary is > updated. Otherwise it could lead to incorrect inline summary. > > Bootstrapped and gcc regression test on-going. > > OK for trunk? > > Thanks, > Dehao > > gcc/ChangeLog: > > 2014-03-21 Dehao Chen > > *ipa-inline.c

Re: [PATCH] __builtin_expect with alternate predictors for Fortran (PR ipa/58721)

2014-03-22 Thread Jan Hubicka
> Hi! > > Here is an updated patch for what Tobias has posted earlier: > http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00043.html > While that version bootstrapped/regtested fine, most of the Fortran > tests ICEd, primarily because the 3 operand __builtin_expect wasn't being > removed from the IL a