Re: [PATCH, i386, Darwin RFT]: Remove reload_in_progress checks

2015-04-20 Thread Uros Bizjak
On Mon, Apr 20, 2015 at 12:00 PM, Iain Sandoe wrote: > > On 20 Apr 2015, at 10:47, Dominique d'Humières wrote: > >> After having fixed the typo, regtesting went without regression. > > I have done a bootstrap on i686-darwin10 with the amended patch - slow > machine, so testing still in progress (

Re: [PATCH][AArch64] Implement -m{cpu,tune,arch}=native using only /proc/cpuinfo

2015-04-20 Thread Andrew Pinski
On Mon, Apr 20, 2015 at 8:47 AM, Kyrill Tkachov wrote: > Hi all, > > This is an attempt to add native CPU detection to AArch64 GNU/Linux targets. > Similar to other ports we use SPEC rewriting to rewrite > -m{cpu,tune,arch}=native > options into the appropriate CPU/architecture and the architectur

Re: Handle redirection blocks with clobbers

2015-04-20 Thread Jeff Law
On 04/20/2015 11:19 AM, Jakub Jelinek wrote: On Mon, Apr 20, 2015 at 11:14:03AM -0600, Jeff Law wrote: while (!gsi_end_p (gsi) && (gimple_code (gsi_stmt (gsi)) == GIMPLE_LABEL || is_gimple_debug (gsi_stmt (gsi)) -|| gimple_nop_p (gsi_stmt (gsi +

Re: [PATCH 1/2] ?PR c++/61636

2015-04-20 Thread Adam Butcher
On 2015-04-20 11:25, Marek Polacek wrote: On Sat, Apr 18, 2015 at 06:53:28PM +0100, Adam Butcher wrote: Test like this? /* { dg-do run { target c++14 } } */ /* { dg-final { scan-assembler-not "..." } } */ What is this dg-final supposed to do here? It was a placeholder for making sure that

Re: [patch, fortran] PR 37131

2015-04-20 Thread Dominique d'Humières
I have played a little bit with the patched gfortran. (1) gfortran.dg/coarray_lib_this_image_2.f90 is still failing FAIL: gfortran.dg/coarray_lib_this_image_2.f90 -O scan-tree-dump-times original "mylbound = parm...dim\\[0\\].stride >= 0 && parm...dim\\[0\\].ubound >= parm...dim\\[0\\].lbou

[PATCH diagnostics/fortran] Handle two locations for the same diagnostic. Convert all gfc_warning_1 and gfc_notify_std_1 calls

2015-04-20 Thread Manuel López-Ibáñez
The Fortran FE allows diagnostics with two different locations. Depending on whether these locations are on the same line or not, this may produce one or two caret lines. This is the last remaining issue left to make Fortran diagnostics use the common code. In the common diagnostics: I added supp

Re: [PATCH 0/13] Add musl support to GCC

2015-04-20 Thread Gregor Richards
To confirm: Yes, I've signed the FSF copyright assignment document, and yes we've discussed the changes relative to my own (which were mostly just cleanups and sanity anyway). I'm Gregor Richards and I approve this message. With valediction, - Gregor Richards On 04/20/2015 02:49 PM, Szabolc

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-04-20 Thread Gerald Pfeifer
On Mon, 20 Apr 2015, Thomas Schwinge wrote: > Gerald, is it OK to commit the following to update GCC 5 changes' »New > Languages and Language specific improvements« section? Sure thing. Gerald

Re: [AArch64][PR65375] Fix RTX cost for vector SET

2015-04-20 Thread James Greenhalgh
On Fri, Apr 17, 2015 at 12:19:14PM +0100, Kugan wrote: > >> My point is that adding your patch while keeping the logic at the top > >> which claims to catch ALL vector operations makes for less readable > >> code. > >> > >> At the very least you'll need to update this comment: > >> > >> /* TODO:

Re: [PATCH 8/13] sh musl support

2015-04-20 Thread Oleg Endo
On Mon, 2015-04-20 at 19:56 +0100, Szabolcs Nagy wrote: > Set up dynamic linker name for sh. > > gcc/Changelog: > > 2015-04-16 Gregor Richards > > * config/sh/linux.h (MUSL_DYNAMIC_LINKER): Define. The SH parts are OK for trunk. Cheers, Oleg

Re: [PATCH][AArch64] Increase static buffer size in aarch64_rewrite_selected_cpu

2015-04-20 Thread James Greenhalgh
On Mon, Apr 20, 2015 at 05:24:39PM +0100, Kyrill Tkachov wrote: > Hi all, > > When trying to compile a testcase with -mcpu=cortex-a57+crypto+nocrc I got > the weird assembler error: > Assembler messages: > Error: missing architectural extension > Error: unrecognized option -mcpu=cortex-a57+crypto+

Re: [PATCH 10/13] fixincludes

2015-04-20 Thread Jeff Law
On 04/20/2015 12:58 PM, Szabolcs Nagy wrote: No fixincludes are needed for musl. fixincludes/Changelog: 2015-04-16 Gregor Richards * mkfixinc.sh: Add *-musl* with no fixes. OK. jeff

Re: [PATCH 1/13] libitm fixes for musl support

2015-04-20 Thread Jeff Law
On 04/20/2015 12:51 PM, Szabolcs Nagy wrote: This are minor correctness fixes required for musl. (fcntl.h is the standard header and always available on Linux, sys/fcntl.h is just a legacy alias, so use the standard one.) libitm/Changelog: 2015-04-16 Gregor Richards * config/arm/hw

RE: [PATCH][MIPS] Enable load-load/store-store bonding

2015-04-20 Thread Moore, Catherine
> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Monday, April 20, 2015 3:10 PM > To: Sameera Deshpande; Moore, Catherine > Cc: Richard Sandiford; gcc-patches@gcc.gnu.org; echri...@gmail.com > Subject: RE: [PATCH][MIPS] Enable load-load/store-store

Re: Announcing Segher Boessenkool as combine.c maintainer

2015-04-20 Thread Segher Boessenkool
On Mon, Apr 20, 2015 at 01:29:03PM -0600, Jeff Law wrote: > Segher, can you please add yourself to the MAINTAINERS file for the > additional role. Done as this. Cheers, Segher 2015-04-20 Segher Boessenkool * MAINTAINERS (Various Maintainers): Add myself as instruction com

Re: [wwwdocs] Add porting_to.html for GCC 5 (again)

2015-04-20 Thread Gerald Pfeifer
On Wed, 14 Jan 2015, Marek Polacek wrote: > A few months ago I posted the "porting to" document for GCC 5. > But I never got around to commit it, so here it is again, this > time with feewing. Thanks again for doing this. Below is my last set of changes: Break some paragraphs and longer sentence

Re: [RS6000] pr65810, powerpc64 alignment of r2

2015-04-20 Thread Alan Modra
On Mon, Apr 20, 2015 at 01:11:41PM -0400, David Edelsohn wrote: > > @@ -6510,6 +6518,8 @@ offsettable_ok_by_alignment (rtx op, HOST_WIDE_INT > > return false; > > > >dsize = GET_MODE_SIZE (mode); > > + if (dsize > POWERPC64_TOC_POINTER_ALIGNMENT) > > +return false; > > Why do you imm

Re: [PATCH] 65479 - sanitizer stack trace missing frames past #0 on powerpc64

2015-04-20 Thread Martin Sebor
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index b4052ef..18eede3 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,12 @@ +2015-04-19 Martin Sebor + +PR sanitizer/65479 +* gcc/testsuite/c-c++-common/asan/misalign-1.c [powerpc*-*-*-*]: +U

[wwwdocs] PATCH for Re: [ANN] gcc-lua: Lua plugin for the GNU Compiler Collection

2015-04-20 Thread Gerald Pfeifer
Hi Peter, On Wed, 31 Oct 2012, Peter Colberg wrote: > gcc‑lua extends the GNU Compiler Collection with the ability to run Lua > scripts. The plugin provides an interface to register callback functions > for plugin events, and inspect the abstract syntax tree of a translation > unit. The plugin

Re: [Patch] pr65779 - [5/6 Regression] undefined local symbol on powerpc

2015-04-20 Thread Alan Modra
On Mon, Apr 20, 2015 at 12:17:08PM -0600, Jeff Law wrote: > So just one question. Why handle the split case differently? In the split > case you effectively move the debug insn to the new block. In the !split > case, you just delete the debug insn. The idea was that when split we have a new blo

[PATCH] use gold/configure.tgt to calc supported targets

2015-04-20 Thread Mike Frysinger
Rather than maintain two lists, re-use gold's target file to determine whether the current target is supported. 2015-04-20 Mike Frysinger * configure.ac: Replace $target checks with gold/configure.tgt. * configure: Regenerate. --- configure| 20 ++-- config

Ping^3 : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-04-20 Thread Terry Guo
Hi there, Is this one ok to trunk? BR, Terry On Wed, Apr 15, 2015 at 6:45 PM, Hale Wang wrote: > Ping for trunk? > > Hale > >> -Original Message- >> From: Richard Sandiford [mailto:rdsandif...@googlemail.com] >> Sent: Friday, February 27, 2015 4:04 AM >> To: Terry Guo >> Cc: Segher Boes

[PATCH] Emit -Waddress warnings for comparing address of reference against NULL

2015-04-20 Thread Patrick Palka
Implementation is pretty straightforward. The only catch is that the middle-end doesn't actually assume that REFERENCE_TYPEs are non-NULL so code like int &a = *(int *)0; if (&a != 0) will warn that &a will never be NULL yet the middle-end will fold the conditional to false instead of tr

Re: Ping^3 : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-04-20 Thread Segher Boessenkool
On Tue, Apr 21, 2015 at 09:39:16AM +0800, Terry Guo wrote: > Is this one ok to trunk? Probably, if you send the patch + changelog entry :-) Did you fix the comment? REG_USERVAR_P and HARD_REGISTER_P can be set for more than just register asm. Segher

RE: [PATCH][AArch64] Implement -m{cpu,tune,arch}=native using only /proc/cpuinfo

2015-04-20 Thread Kumar, Venkataramanan
Hi Kyrill, In AMD Seattle board, I see that CPU implementer is 0x41 and CPU part is 0xd07.CPU variant is 1 but you don’t do anything with that. It matches with cortex-a57 and its features. I will try a bootstrap test as well. Regards, Venkat. -Original Message- From: gcc-pat

Re: ira.c update_equiv_regs patch causes gcc/testsuite/gcc.target/arm/pr43920-2.c regression

2015-04-20 Thread Jeff Law
On 04/20/2015 01:09 AM, Shiva Chen wrote: Hi, Jeff Thanks for your advice. can_replace_by.patch is the new patch to handle both cases. pr43920-2.c.244r.jump2.ori is the original jump2 rtl dump pr43920-2.c.244r.jump2.patch_can_replace_by is the jump2 rtl dump after patch can_replace_by.patch

Re: [PING^2] [PATCH][5a/5] Postpone expanding va_arg until pass_stdarg

2015-04-20 Thread Bin.Cheng
On Thu, Apr 16, 2015 at 4:55 PM, Richard Biener wrote: > On Thu, 16 Apr 2015, Tom de Vries wrote: > >> [stage1 ping^2] >> On 10-03-15 16:30, Tom de Vries wrote: >> > [stage1 ping] >> > On 22-02-15 14:13, Tom de Vries wrote: >> > > On 19-02-15 14:03, Richard Biener wrote: >> > > > On Thu, 19 Feb 20

Re: [PATCH] Add new target h8300-*-linux

2015-04-20 Thread Yoshinori Sato
At Mon, 20 Apr 2015 09:26:21 -0600, Jeff Law wrote: > > On 04/19/2015 10:51 PM, Yoshinori Sato wrote: > >>> + if (TARGET_H8300H && (TARGET_H8300S || TARGET_H8300SX)) > >>> +{ > >>> + target_flags ^= MASK_H8300H; > >>> +} > >> I'm a bit concerned by this. Why did you need to make thi

Re: [PATCH] 65479 - sanitizer stack trace missing frames past #0 on powerpc64

2015-04-20 Thread Jakub Jelinek
Hi! Note, the changes aren't acceptable for 5.1 at this point (release is tomorrow), and for 5.2 backport they should be tested for a while on the trunk, so there is no rush now. > --- a/libsanitizer/ChangeLog > +++ b/libsanitizer/ChangeLog > @@ -1,3 +1,15 @@ > +2015-04-19 Martin Sebor > + > +

[PATCH, alpha]: Revert PR 41089 workaround

2015-04-20 Thread Uros Bizjak
Hello! Now that PR 64950 is fixed, we can safely remove PR 41089 workaround. 2015-04-21 Uros Bizjak PR tree-optimization/64950 Revert: 2010-08-02 Uros Bizjak PR target/41089 * config/alpha/alpha.c (alpha_build_builtin_va_list): Mark __offset

Re: [PATCH, PR target/65103, 2/3] Propagate address constants into loops for i386

2015-04-20 Thread Jeff Law
On 04/17/2015 02:34 AM, Ilya Enkovich wrote: On 15 Apr 14:07, Ilya Enkovich wrote: 2015-04-14 8:22 GMT+03:00 Jeff Law : On 03/15/2015 02:30 PM, Richard Sandiford wrote: Ilya Enkovich writes: This patch allows propagation of loop invariants for i386 if propagated value is a constant to be u

<    1   2