Re: [PATCH fortran/diagnostics] Move gfc_error (buffered) to common diagnostics

2014-12-07 Thread Tobias Burnus
Manuel López-Ibáñez wrote: OK, this is then the final patch. Bootstrapped and regression tested on x86_64-linux-gnu. OK to commit? void gfc_buffer_error (bool flag) { buffered_p = flag; - pp_warning_buffer.flush_p = !flag; + buffered_p = flag; } Now there are two "buffered_p" … Other

Re: [RFC] diagnostics.c: For terminals, restrict messages to terminal width?

2014-12-07 Thread FX
OK. But if we rename the function, why not simply terminal_width() ? FX > Le 6 déc. 2014 à 23:14, Tobias Burnus a écrit : > > This patch fixes a Fortran diagnostic "regression". > > With the current common diagnostic, the width shown with caret diagnostic is > determined by: > >case OP

Re: [PATCH, i386] Fix PR64003

2014-12-07 Thread Richard Sandiford
Ilya Enkovich writes: > diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md > index 88435d6..9019ed8 100644 > --- a/gcc/config/i386/i386.md > +++ b/gcc/config/i386/i386.md > @@ -10958,6 +10958,24 @@ > ;; Basic conditional jump instructions. > ;; We ignore the overflow flag for signed

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Andrew Pinski
On Sat, Dec 6, 2014 at 6:35 PM, Andrew Pinski wrote: > On Sat, Dec 6, 2014 at 5:54 PM, Andrew Pinski wrote: >> On Fri, Dec 5, 2014 at 9:08 AM, Marcus Shawcroft >> wrote: >>> On 18 November 2014 at 08:34, Bin Cheng wrote: >>> 2014-11-18 Bin Cheng * config/aarch64/aarch6

duplicated text in gcc.target/h8300

2014-12-07 Thread Bernd Edlinger
Hi Oleg, your checking from 2013-06-15 duplicated contents of the following files: ./gcc.target/h8300/pragma-isr2.c ./gcc.target/h8300/pragma-isr.c ./gcc.target/h8300/h8300.exp it is therefore unlikely that theses tests can work in the h8300 target. svn log -r200117   --

Re: duplicated text in gcc.target/h8300

2014-12-07 Thread Oleg Endo
Hi Bernd, On Sun, 2014-12-07 at 15:17 +0100, Bernd Edlinger wrote: > Hi Oleg, > > > your checking from 2013-06-15 duplicated contents of the following files: > > > ./gcc.target/h8300/pragma-isr2.c > ./gcc.target/h8300/pragma-isr.c > ./gcc.target/h8300/h8300.exp > > > it is therefore unlikely

[committed] Require c99_target for gcc.dg/ipa/ipa-icf-5.c and gcc.dg/ipa/ipa-icf-7.c

2014-12-07 Thread John David Anglin
The attached change fixes the fails of gcc.dg/ipa/ipa-icf-5.c and gcc.dg/ipa/ipa-icf-7.c on hppa*-*-hpux*. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2014-12-07 John David Anglin PR testsuite/64038 * gcc.dg/ipa/ipa-icf-5.c: Require c99_runtim

[PATCH] Fix failure of coarray tests on hpux

2014-12-07 Thread John David Anglin
The attached change fixes the fortran coarray tests on hpux. On hpux, it is necessary to link directly against libatomic. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Okay for trunk? Dave -- John David Anglin dave.ang...@bell.net 2014-12-07 John David Anglin P

Re: [PATCH] Fix failure of coarray tests on hpux

2014-12-07 Thread Tobias Burnus
John David Anglin wrote: The attached change fixes the fortran coarray tests on hpux. On hpux, it is necessary to link directly against libatomic. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Okay for trunk? From the Fortran side, it looks okay. Tobias

[PATCH] Correctly handle stack arguments of sibling calls in DSE pass

2014-12-07 Thread John David Anglin
On 1-Dec-14, at 11:57 AM, Jeff Law wrote: Prior to reload (ie, in DSE1) there's a bit of magic in that we do not set frame_read on call insns. That may in fact be wrong and possibly the source of the problem. /* This field is only used for the processing of const functions. These funct

Re: [patch, ARM] Add support for crtfastmath.o

2014-12-07 Thread Sandra Loosemore
On 11/08/2014 08:57 AM, Sandra Loosemore wrote: 2014-11-08 Sandra Loosemore Chris Jones Joshua Conner gcc/ * config/arm/unknown-elf.h (STARTFILE_SPEC): Add conditional linking of crtfastmath.o. * config/arm/linux-eabi.h (STARTFILE_SPEC): Likewise.

Re: [testsuite] Fix multiple definitions of _init

2014-12-07 Thread Oleg Endo
On Tue, 2014-12-02 at 10:23 -0800, Mike Stump wrote: > No. It is reasonable for the test suite to fail when the > implementation of gcc is wrong (unclean) or newlib startup code is > wrong (unclean). Since that is what happened, the fix is to fix the > cleanliness problem. > > I’ve read through

[SH][committed] Fix extendqihi2 expander on SH64

2014-12-07 Thread Oleg Endo
Hi, This fixes an SH64 problem which was introduced in r185534 and went unnoticed until recently. There, the extendqihi2 insn was converted to an expander and the TARGET_SH1 condition was wrongly omitted. Tested with 'make all-gcc' on sh-elf and sh64-elf. Committed on trunk as r218469, on 4.9 a

[SH][committed] PR 53513 - Fix test case failure

2014-12-07 Thread Oleg Endo
Hi, After the changes regarding the FPSCR handling, some SH test cases started to fail. This fixes one of them. Tested with make -k check-gcc RUNTESTFLAGS="sh.exp --target_board=sh-sim\{-m4/-ml}" to verify that the test case passes again. Committed as r218472. Cheers, Oleg gcc/testsuite/Chang

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

2014-12-07 Thread Ilya Verbin
Hi, On 28 Nov 09:36, Richard Biener wrote: > On Fri, 28 Nov 2014, Ilya Verbin wrote: > > I found a bug here, have_{lto,offload} must be set if at least one file > > contains > > lto/offload sections, but currently they are overwritten by the last file. > > Fix is bootstrapped and regtested on x86

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Bin.Cheng
On Sun, Dec 7, 2014 at 6:24 PM, Andrew Pinski wrote: > On Sat, Dec 6, 2014 at 6:35 PM, Andrew Pinski wrote: >> On Sat, Dec 6, 2014 at 5:54 PM, Andrew Pinski wrote: >>> On Fri, Dec 5, 2014 at 9:08 AM, Marcus Shawcroft >>> wrote: On 18 November 2014 at 08:34, Bin Cheng wrote: > 201

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Andrew Pinski
On Sun, Dec 7, 2014 at 5:47 PM, Bin.Cheng wrote: > On Sun, Dec 7, 2014 at 6:24 PM, Andrew Pinski wrote: >> On Sat, Dec 6, 2014 at 6:35 PM, Andrew Pinski wrote: >>> On Sat, Dec 6, 2014 at 5:54 PM, Andrew Pinski wrote: On Fri, Dec 5, 2014 at 9:08 AM, Marcus Shawcroft wrote: > On 18

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Bin.Cheng
On Mon, Dec 8, 2014 at 11:16 AM, Andrew Pinski wrote: > On Sun, Dec 7, 2014 at 5:47 PM, Bin.Cheng wrote: >> On Sun, Dec 7, 2014 at 6:24 PM, Andrew Pinski wrote: >>> On Sat, Dec 6, 2014 at 6:35 PM, Andrew Pinski wrote: On Sat, Dec 6, 2014 at 5:54 PM, Andrew Pinski wrote: > On Fri, Dec

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Bin.Cheng
On Mon, Dec 8, 2014 at 11:26 AM, Bin.Cheng wrote: > On Mon, Dec 8, 2014 at 11:16 AM, Andrew Pinski wrote: >> On Sun, Dec 7, 2014 at 5:47 PM, Bin.Cheng wrote: >>> On Sun, Dec 7, 2014 at 6:24 PM, Andrew Pinski wrote: On Sat, Dec 6, 2014 at 6:35 PM, Andrew Pinski wrote: > On Sat, Dec 6,

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Andrew Pinski
On Sun, Dec 7, 2014 at 7:35 PM, Bin.Cheng wrote: > On Mon, Dec 8, 2014 at 11:26 AM, Bin.Cheng wrote: >> On Mon, Dec 8, 2014 at 11:16 AM, Andrew Pinski wrote: >>> On Sun, Dec 7, 2014 at 5:47 PM, Bin.Cheng wrote: On Sun, Dec 7, 2014 at 6:24 PM, Andrew Pinski wrote: > On Sat, Dec 6, 2014

Re: [PATCH AARCH64]load store pair optimization using sched_fusion pass.

2014-12-07 Thread Bin.Cheng
On Mon, Dec 8, 2014 at 11:38 AM, Andrew Pinski wrote: > On Sun, Dec 7, 2014 at 7:35 PM, Bin.Cheng wrote: >> On Mon, Dec 8, 2014 at 11:26 AM, Bin.Cheng wrote: >>> On Mon, Dec 8, 2014 at 11:16 AM, Andrew Pinski wrote: On Sun, Dec 7, 2014 at 5:47 PM, Bin.Cheng wrote: > On Sun, Dec 7, 201

Re: [PATCH v4] gcc/c-family/c-cppbuiltin.c: Let buffer enough to print host wide integer value

2014-12-07 Thread Chen Gang
On 11/27/14 07:41, Chen Gang wrote: > OK, I shall send related test case within two weeks (2014-12-10). > > Joseph Myers wrote: > > On Thu, 27 Nov 2014, Chen Gang wrote: > >> The original length 18 is not enough for HOST_WIDE_INT printing, need >> use 20 instead of. >> >> Also need additional

Re: [testsuite] Fix multiple definitions of _init

2014-12-07 Thread Kaz Kojima
Oleg Endo wrote: > Kaz, could you please check if the patch doesn't break anything on > sh4-linux? If so, I'd like to commit this to trunk. Build and test ok on sh4-unknown-linux-gnu. Regards, kaz