Re-enable shadd insns on the PA

2015-05-20 Thread Jeff Law
This is the first in a series of patches to fix the fallout from recent combiner changes in how shift-add style insns are canonicalized. This patch effectively just adds a new shift-add insn to the PA port. The old shift-add insn stays for now, but will be removed in a follow-up once I'm con

C++ PATCH to change -Wc++14-compat operator delete warning

2015-05-20 Thread Jason Merrill
I noticed that -Wc++14-compat was warning about headers that had been updated to include a declaration of a global sized operator delete. This was intended to catch problematic placement deletes, but now I think that C++14 headers that just don't bother to guard the declaration with a C++14 #if

C++ PATCH for variable template partial specialization bug

2015-05-20 Thread Jason Merrill
register_specialization complains about a specialization in a different namespace from the main template; we shouldn't give that error about an instantiation of a partial specialization. And we were about to SET_DECL_IMPLICIT_INSTANTIATION anyway, so let's just do that before we call register_

Re: [patch 3/10] debug-early merge: C++ front-end

2015-05-20 Thread Aldy Hernandez
On 05/20/2015 03:18 PM, Jason Merrill wrote: On 05/08/2015 09:14 PM, Aldy Hernandez wrote: + if (!flag_syntax_only) +c_parse_final_cleanups (); The condition is a significant change of behavior for the C++ front end; doing final instantiation and such even with -fsyntax-only was a deliber

Re: [patch 2/10] debug-early merge: C front-end (include c-family/)

2015-05-20 Thread Aldy Hernandez
Update for the c-family bits removing the flag_syntax_only part Jason requested. And BTW, ping for you C front-end maintainers (unless Jason is reviewing the C bits, in which case the rest of you can sit back and look pretty). Aldy gcc/c-family/ * c-common.h (c_parse_final_cleanups): New pr

Re: [patch 10/10] debug-early merge: compiler proper

2015-05-20 Thread Aldy Hernandez
On 05/20/2015 05:01 PM, Jan Hubicka wrote: commit 8824b5ecba26cef065e47b34609c72677c3c36fc Author: Aldy Hernandez Date: Wed May 20 16:31:14 2015 -0400 Set DECL_IGNORED_P on temporary arrays created in the switch conversion pass. diff --git a/gcc/tree-switch-conversion.c b/gcc/tree

[PATCH v2] Handle OS X deployment targets correctly

2015-05-20 Thread Lawrence Velázquez
As described in PR target/63810, this addresses several problems with the validation and encoding of deployment target version strings for the __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ macro. There are currently four testcases exercising inputs to -mmacosx-version-min (gcc/testsuite/gcc.dg/da

Re: RFA: PATCH to use -std=c++98 in stage 1 of bootstrap

2015-05-20 Thread Alexandre Oliva
On May 20, 2015, Jason Merrill wrote: > I want to explicitly pass -std=c++98 to the compiler used in building > stage 1. Does this seem like the right way to do that? It certainly works, but I'm a bit concerned about its overriding the earlier choices of STAGE1_CXXFLAGS, that use CXXFLAGS or ST

Add statistics to alias.c

2015-05-20 Thread Jan Hubicka
Hi, this patch extends statistics from tree-ssa-alias to also cover TBAA oracle. This is useful to keep track of aliasing effectivity. For example the hack in alias.c putting globbing all pointers to one costs about 20% of all answers on firefox. I.e. from 15500978 disambiguations/23744267 querrie

Re: [patch 2/10] debug-early merge: C front-end (include c-family/)

2015-05-20 Thread Joseph Myers
On Wed, 20 May 2015, Aldy Hernandez wrote: > Update for the c-family bits removing the flag_syntax_only part Jason > requested. > > And BTW, ping for you C front-end maintainers (unless Jason is reviewing the C > bits, in which case the rest of you can sit back and look pretty). The C front-end

Re: [patch 2/10] debug-early merge: C front-end (include c-family/)

2015-05-20 Thread Aldy Hernandez
On 05/20/2015 06:16 PM, Joseph Myers wrote: On Wed, 20 May 2015, Aldy Hernandez wrote: Update for the c-family bits removing the flag_syntax_only part Jason requested. And BTW, ping for you C front-end maintainers (unless Jason is reviewing the C bits, in which case the rest of you can sit bac

PING: Re: [patch 6/10] debug-early merge: Java front-end

2015-05-20 Thread Aldy Hernandez
Perhaps I should've sent this to the java-patches list. PING. gcc/java/ * class.c: Remove pending_static_fields. (add_field): Remove use of pending_static_fields. (java_write_globals): Remove. * decl.c (java_mark_class_local): Remove use of pending_static_fields. * java-tree.h: Remove pen

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-20 Thread Hans-Peter Nilsson
> From: "H.J. Lu" > Date: Tue, 19 May 2015 21:20:46 +0200 > This patch affects your target. Can you verify if it is OK on > your target? If you mean philosophically, it's ok when the general bits and direction are ok, but for the practical part of checking e.g. for typos through compilation, pl

Re: [PATCH/libiberty] fix build of gdb/binutils with clang.

2015-05-20 Thread Yunlian Jiang
GCC bootstraps with this patch. On Wed, May 20, 2015 at 12:54 PM, Ian Lance Taylor wrote: > This is OK if GCC bootstraps. > > Thanks. > > Ian > > On Wed, May 20, 2015 at 11:25 AM, Yunlian Jiang wrote: >> I have the following change to make libiberty compile with _GNU_SOURCE >> defined >> and re

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

2015-05-20 Thread Martin Sebor
Now that I know a bit more from Jakub & Yuri's comments, I don't think we should be turning that flag on for all the targets in the testsuite. I was primarily trying to avoid someone else having to go through the same analysis and reach the same conclusion for another port. But I'm less concern

Re: [patch,gomp4] error on invalid acc loop clauses

2015-05-20 Thread Cesar Philippidis
On 05/20/2015 07:32 AM, Jakub Jelinek wrote: > For OpenMP/OpenACC, there is still lots of diagnostics emitted during > gimplification and at the start of the omp lowering phases, so for > diagnostics purposes you can consider them as part of a common layer for all > the 3 FEs. Thanks, that makes

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-20 Thread H.J. Lu
On Wed, May 20, 2015 at 3:53 PM, Hans-Peter Nilsson wrote: >> From: "H.J. Lu" >> Date: Tue, 19 May 2015 21:20:46 +0200 > >> This patch affects your target. Can you verify if it is OK on >> your target? > > If you mean philosophically, it's ok when the general bits and > direction are ok, but for

Re: [patch 3/10] debug-early merge: C++ front-end

2015-05-20 Thread Jason Merrill
OK. Jason

Re: [PATCH 3/4] split-stack for powerpc64

2015-05-20 Thread Alan Modra
Older assemblers don't understand .abiversion, so I'm committing the following as obvious to fix a problem Michael Meissner found when building gcc for powerpc64-linux. PR libgcc/66225 * config/rs6000/morestack.S: Remove ".abiversion 1". Index: libgcc/config/rs6000/morestack.S ===

RE: [PATCH] Fix PR66168: ICE due to incorrect invariant register info

2015-05-20 Thread Thomas Preud'homme
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme > > From: Steven Bosscher [mailto:stevenb@gmail.com] > > Sent: Tuesday, May 19, 2015 7:21 PM > > > > Not OK. > > This will break in move_invariants() when it looks at REGNO (inv->reg)

[PATCH 0/7] Some ifdef removal

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders Hi, Another week and more patches to remove conditional compilation ;-) each patch individually bootstrapped + regtested on x86_64-unknown-linux-gnu, and run through config-list.mk with a couple more patches I'll commit when I finish writing ChangeLogs. Committing to trun

[PATCH 2/7] remove most ifdef STACK_GROWS_DOWNWARD

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/c-family/ChangeLog: 2015-05-20 Trevor Saunders * c-cppbuiltin.c (c_cpp_builtins): Use if instead of #if with STACK_GROWS_DOWNWARD. gcc/ChangeLog: 2015-05-20 Trevor Saunders * *.c: Use if instead of preprocessor checks with STACK

[PATCH 3/7] move default for STACK_PUSH_CODE to defaults.h

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-05-20 Trevor Saunders * defaults.h: Add default for STACK_PUSH_CODE. * expr.c: Don't redefine STACK_PUSH_CODE. * recog.c: Likewise. --- gcc/ChangeLog | 6 ++ gcc/defaults.h | 8 gcc/expr.c | 8 gcc/

[PATCH 1/7] always define STACK_GROWS_DOWNWARD

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/c-family/ChangeLog: 2015-05-20 Trevor Saunders * c-cppbuiltin.c (c_cpp_builtins): Check the value of STACK_GROWS_DOWNWARD rather than if it is defined. gcc/ChangeLog: 2015-05-20 Trevor Saunders * *.c: Check the value of STACK_GROWS_DOWN

[PATCH 4/7] don't compare ARG_FRAME_POINTER_REGNUM and FRAME_POINTER_REGNUM with the preprocessor

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-05-20 Trevor Saunders * *.c: Remove comparison of ARG_FRAME_POINTER_REGNUM and FRAME_POINTER_REGNUM with the preprocessor. --- gcc/ChangeLog | 5 + gcc/combine.c | 18 +++--- gcc/df-problems.c | 5 ++--- gcc

[PATCH 7/7] always define HAVE_peephole

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-05-20 Trevor Saunders * final.c (final_scan_insn): Don't check HAVE_peephole with the preprocessor. * output.h: Likewise. * genconfig.c (main): Alwways define HAVE_peephole. * genpeep.c: Don't emit checks of HA

[PATCH 5/7] always define HAVE_conditional_move

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-05-20 Trevor Saunders * genconfig.c (main): Always define HAVE_conditional_move. * *.c: Don't check if HAVE_conditional_move is defined. --- gcc/ChangeLog | 4 gcc/combine.c | 2 +- gcc/expmed.c | 4 ++-- gcc

[PATCH 6/7] remove #if HAVE_conditional_move

2015-05-20 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-05-20 Trevor Saunders * *.c, *.h: DOn't check HAVE_conditional_move with the preprocessor. --- gcc/ChangeLog | 4 gcc/combine.c | 5 ++--- gcc/expmed.c | 13 + gcc/expr.c| 10 ++ gcc/optabs.c | 30 +++

[patch, testsuite] don't specify "dg-do run" explicitly for vect test cases

2015-05-20 Thread Sandra Loosemore
On targets such as ARM, some arches are compatible with options needed to enable compilation with vectorization, but the specific hardware (or simulator or BSP) available for execution tests may not implement or enable those features. The vect.exp test harness already includes some magic to de

[patch, testsuite, ARM] don't try to execute advsimd-intrinsics tests on hardware without NEON

2015-05-20 Thread Sandra Loosemore
ARM testing shares the AArch64 advsimd-intrinsics execution tests. On ARM, though, the NEON support being tested is optional -- some arches are compatible with the NEON compilation options but hardware available for testing might or might not be able to execute those instructions. In arm-none-

[RFA] Restore combine.c split point for multiply-accumulate instructions

2015-05-20 Thread Jeff Law
find_split_point will tend to favor splitting complex insns in such a way as to encourage multiply-add insns. It does this by splitting an unrecognizable insn at the (plus (mult)). Now that many MULTs are canonicalized as ASHIFT, that code to prefer the multiply-add is no longer triggering

Re: [PATCH 7/7] always define HAVE_peephole

2015-05-20 Thread Jeff Law
On 05/20/2015 08:09 PM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders gcc/ChangeLog: 2015-05-20 Trevor Saunders * final.c (final_scan_insn): Don't check HAVE_peephole with the preprocessor. * output.h: Likewise. * genconfig.c (main): Alwways define

[patch, testsuite, ARM] don't try to execute simd.exp tests on targets without NEON

2015-05-20 Thread Sandra Loosemore
This is another patch aimed at fixing bugs relating to trying to execute NEON code on a target that doesn't support it revealed by my arm-none-eabi testing on a gazillion different multilibs. Inspired by what vect.exp does and my other patch in this group to fix advsimd-intrinsics.exp, I've ha

Re: Add statistics to alias.c

2015-05-20 Thread Bernhard Reutner-Fischer
On May 21, 2015 12:13:19 AM GMT+02:00, Jan Hubicka wrote: >Hi, >this patch extends statistics from tree-ssa-alias to also cover TBAA >oracle. >This is useful to keep track of aliasing effectivity. For example the >hack >in alias.c putting globbing all pointers to one costs about 20% of all >answe

Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with "--with-sysroot=/"

2015-05-20 Thread Yvan Roux
Hi, On 8 May 2015 at 00:07, Joseph Myers wrote: > On Mon, 20 Apr 2015, Pavel Kopyl wrote: > >> Hi all, >> >> >> To build a GCC-4.9.2 ARM cross-compiler for my setting I need to configure it >> with "--with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.9.2". >> But I found that gcc driver r

Re: OpenACC: initialization with unsupported acc_device_t

2015-05-20 Thread Thomas Schwinge
Hi Julian! On Thu, 7 May 2015 16:56:11 +0100, Julian Brown wrote: > On Tue, 5 May 2015 16:09:18 +0200 > Thomas Schwinge wrote: > > On Tue, 5 May 2015 08:43:48 -0400, John David Anglin > > wrote: > > > On 2015-05-05 5:43 AM, Thomas Schwinge wrote: > > > >> FAIL: libgomp.oacc-c/../libgomp.oacc-c-

<    1   2