Re: [PATCH][SPARC] sparc: switch -fasynchronous-unwind-tables on by default.

2016-03-19 Thread Jose E. Marchesi
> The backtrace(3) implementation for sparc contains a simple unwinder > that works well in most cases, but that unwinder is not used if > libgcc_s.so can be dlopened and it provides _Unwind_Backtrace. There's no reason that simple unwinder can't be put into MD_FALLBACK_FR

Re: C++ PATCH to fix missing warning (PR c++/70194)

2016-03-19 Thread Jason Merrill
OK. Jason

[PATCH] Add debug_varinfo and debug_varmap

2016-03-19 Thread Tom de Vries
[ was: Re: [RFC] dump_varmap in tree-ssa-structalias.c ] On 10/03/16 10:07, Richard Biener wrote: On Thu, 10 Mar 2016, Tom de Vries wrote: Hi, I wrote attached patch to print the actual contents of the varmap variable in tree-ssa-structalias.c. Does it make sense to rewrite this into a dump_v

Re: [i386] Support .lbss etc. sections with Solaris as (PR target/59407)

2016-03-19 Thread Uros Bizjak
On Thu, Mar 17, 2016 at 11:40 PM, Rainer Orth wrote: > gcc.target/i386/pr58218.c currently FAILs on 64-bit Solaris/x86 with the > native assembler: > > FAIL: gcc.target/i386/pr58218.c (test for excess errors) > > Excess errors: > Assembler: pr58218.c > "/var/tmp//cciHFIO7.s", line 3 : Sect

[C++ PATCH] Fix -flifetime-dse bug in dtors too (PR c++/70272)

2016-03-19 Thread Jakub Jelinek
Hi! This is a follow-up to the PR70259 that got fixed yesterday. As the testcase shows, clobbers for empty classes are harmful not just in the ctors, but also in the dtors, where they can result in stores being removed before the corresponding dtor has acted upon them. The ehcleanup1.C part of th

Contents of PO file 'cpplib-6.1-b20160131.fr.po'

2016-03-19 Thread Translation Project Robot
cpplib-6.1-b20160131.fr.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

Re: [PATCH] Fix PR c++/70121 (premature folding of const var that was implicitly captured)

2016-03-19 Thread Jason Merrill
On 03/10/2016 05:58 PM, Patrick Palka wrote: This patch reverses the behavior of process_outer_var_ref, so that we always implicitly capture a const variable if it's capturable, instead of always trying to first fold it to a constant. This behavior however is wrong too, and introduces a differen

Re: C++ PATCH to fix missing warning (PR c++/70194)

2016-03-19 Thread Jason Merrill
On 03/17/2016 02:51 PM, Martin Sebor wrote: On 03/17/2016 10:48 AM, Patrick Palka wrote: On Thu, Mar 17, 2016 at 12:27 PM, Jeff Law wrote: On 03/16/2016 06:43 PM, Martin Sebor wrote: @@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, return build3 (VEC_COND_EXPR, type, cmp

Re: [Patch, testsuite] Skip testcase for avr

2016-03-19 Thread Jeff Law
On 03/16/2016 11:33 AM, Senthil Kumar Selvaraj wrote: 2016-03-16 Senthil Kumar Selvaraj * gcc.c-torture/compile/20151204.c: Skip for avr. THanks. Applied to the trunk. jeff

Re: [PATCH][ARM][testsuite][committed] Do not override -mcpu in no-volatile-in-it.c

2016-03-19 Thread Andre Vieira (lists)
On 16/07/15 16:31, Kyrill Tkachov wrote: > Hi all, > > This scan-assembler test was failing for me when testing with an > explicit /-march=armv7-a variant because > it clashed with the -mcpu=cortex-m7 and overrode it. > > This patch skips the test if the user forces an incompatible -march or > -m

[omp] Create openmp -fopt-info optimization group

2016-03-19 Thread Martin Jambor
Hi, the following patch does two things. First, it creates a new optinfo group for OpenMP and moves OpenMP lowering and expansion to this group. Second, it changes all gridification MSG_NOTE dumps to MSG_MISSED_OPTIMIZATION, which is more appropriate. (Apparently, I remembered to change the dum

Re: Fix 70278 (LRA split_regs followup patch)

2016-03-19 Thread Jeff Law
On 03/18/2016 06:25 AM, Bernd Schmidt wrote: This fixes an oversight in my previous patch here. I used biggest_mode in the assumption that if the reg was used in the function, it would be set to something other than VOIDmode, but that fails if we have a multiword access - only the first hard reg

[gomp-nvptx 6/7] nvptx backend: change mul.u32 to mul.lo.u32

2016-03-19 Thread Alexander Monakov
Recent testing uncovered that PTX JIT may reject attempts to use 'mul.u32' as a non-widening 32-bit multiply instruction. Use 'mul.lo.u32' to fix 32-bit code generation and conform to the PTX spec better. * config/nvptx/nvptx.c (nvptx_init_unisimt_predicate): Emit 'mul.lo.u32' ins

Re: Please include ada-hurd.diff upstream (try2)

2016-03-19 Thread Svante Signell
Sorry for manually creating this mail. Replying to Arnauds message crashes my mail client (evolution). On Wed, 2016-03-16 at 11:53 +0100, Arnaud Charlet wrote: > Yes, you are right, somehow I submitted the old patch. Attached is the > updated one, with a proposed ChangeLog entry included. > > The

Re: [PATCH, PR70161] Fix fdump-ipa-all-graph

2016-03-19 Thread Tom de Vries
On 18/03/16 10:23, Tom de Vries wrote: On 15/03/16 12:37, Richard Biener wrote: On Mon, 14 Mar 2016, Tom de Vries wrote: Hi, this patch fixes PR70161, a 4.9/5/6 regression. Currently when using -fdump-ipa-all-graph, the compiler ICEs in execute_function_dump when testing for pass->graph_dump

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

2016-03-19 Thread Jakub Jelinek
On Wed, Mar 16, 2016 at 01:38:21PM -0600, Jeff Law wrote: > On 03/14/2016 04:13 PM, Jakub Jelinek wrote: > >On Mon, Mar 14, 2016 at 03:25:07PM -0600, Martin Sebor wrote: > >>PR c++/67376 - [5/6 regression] Comparison with pointer to past-the-end > >>of array fails inside constant expression > >

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

2016-03-19 Thread H.J. Lu
On Thu, Mar 17, 2016 at 8:23 AM, Bernd Schmidt wrote: > On 03/17/2016 04:13 PM, H.J. Lu wrote: >> >> On Thu, Mar 17, 2016 at 8:09 AM, Bernd Schmidt >> wrote: >>> >>> On 03/17/2016 04:06 PM, H.J. Lu wrote: This is the patch I am going to check in. >>> >>> That still mentions darwin

[PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

2016-03-19 Thread Jakub Jelinek
Hi! On Wed, Mar 16, 2016 at 05:48:33PM -0500, Segher Boessenkool wrote: > On Wed, Mar 16, 2016 at 01:59:29PM +0100, Bernd Schmidt wrote: > > On 03/16/2016 01:22 PM, Jakub Jelinek wrote: > > >So, this is what we've converged to on IRC and passed bootstrap/regtest > > >on x86_64-linux and i686-linux

Re: [PATCH] Fix PR64764

2016-03-19 Thread H.J. Lu
On Mon, Feb 9, 2015 at 2:30 AM, Tom de Vries wrote: > On 09-02-15 09:59, Richard Biener wrote: >> >> On Thu, 5 Feb 2015, Tom de Vries wrote: >> >>> On 26-01-15 15:47, Richard Biener wrote: Index: gcc/testsuite/gcc.dg/uninit-19.c ==

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245)

2016-03-19 Thread Segher Boessenkool
On Wed, Mar 16, 2016 at 01:59:29PM +0100, Bernd Schmidt wrote: > On 03/16/2016 01:22 PM, Jakub Jelinek wrote: > >So, this is what we've converged to on IRC and passed bootstrap/regtest > >on x86_64-linux and i686-linux. Is this ok for trunk? > > The explanation was a bit confusing at first, but I

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

2016-03-19 Thread Jeff Law
On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. I guess so, but it's one allocation per compiled function

Re: [Patch, testsuite] Skip testcase for avr

2016-03-19 Thread Senthil Kumar Selvaraj
Rainer Orth writes: > Hi Senthil, > >> diff --git a/gcc/testsuite/gcc.c-torture/compile/20151204.c >> b/gcc/testsuite/gcc.c-torture/compile/20151204.c >> index 036316c..0a60871 100644 >> --- a/gcc/testsuite/gcc.c-torture/compile/20151204.c >> +++ b/gcc/testsuite/gcc.c-torture/compile/20151204.c

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

2016-03-19 Thread Alan Modra
On Thu, Mar 17, 2016 at 12:16:58PM +0100, Jakub Jelinek wrote: > the rs6000 backend for whatever strange reason I haven't understood > really wants pointer equality instead of REGNO comparison (even when the > modes match), one (reg:DI 12) should be replaced, another (reg:DI 12) > should not. By t

Re: [05/05] Fix PR 69102

2016-03-19 Thread Jeff Law
On 03/15/2016 09:55 AM, Andrey Belevantsev wrote: Hello, On 14.03.2016 12:52, Andrey Belevantsev wrote: Hello, The problem here is readonly dependence contexts in selective scheduler. We're trying to cache the effect of initializing a dependence context with remembering that context and settin

Re: C++ PATCH to fix missing warning (PR c++/70194)

2016-03-19 Thread Marek Polacek
On Wed, Mar 16, 2016 at 06:43:39PM -0600, Martin Sebor wrote: > >@@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, > >return build3 (VEC_COND_EXPR, type, cmp, minus_one_vec, zero_vec); > > } > > > >+/* Possibly warn about an address never being NULL. */ > >+ > >+static void > >

[PATCH] PR testsuite/70150: Check non-pic/ia32 in stackprotectexplicit2.C

2016-03-19 Thread H.J. Lu
For ia32, __stack_chk_fail isn't called in PIC. We need to check non-pic or non-ia32 before scanning for __stack_chk_fail. OK for trunk? H.J. --- PR testsuite/70150 * g++.dg/stackprotectexplicit2.C: Scanning for __stack_chk_fail only for non-pic or non-ia32 . --- gcc/tes

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

2016-03-19 Thread H.J. Lu
On Thu, Mar 17, 2016 at 7:55 AM, H.J. Lu wrote: > On Thu, Mar 17, 2016 at 7:18 AM, Bernd Schmidt wrote: >> On 03/17/2016 02:59 PM, H.J. Lu wrote: >>> >>> On Fri, Mar 11, 2016 at 9:09 AM, H.J. Lu wrote: We can't set flag_pie to the default when flag_pic == 0, which may be set by -f

Re: [PATCH] Fix PR c++/70205 (ICE on valid call to qualified static member function)

2016-03-19 Thread Jason Merrill
OK. Jason

Re: [PATCH][ARM][5 Backport] PR target/69875 Fix atomic_loaddi expansion

2016-03-19 Thread Ramana Radhakrishnan
On Wed, Feb 24, 2016 at 11:23 AM, Kyrill Tkachov wrote: > Hi all, > > This is the GCC 5 backport of > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01338.html. > The differences are that TARGET_HAVE_LPAE has to be defined in arm.h in a > different way because > the ARM_FSET_HAS_CPU1 mechanism does

Re: [AArch64] Add precision choices for the reciprocal square root approximation

2016-03-19 Thread Wilco Dijkstra
Hi Evandro, > For example, though this approximation is improves the performance > noticeably for DF on A57, for SF, not so much, if at all. I'm still skeptical that you ever can get any gain on scalars. I bet the only gain is on 4x vectorized floats. So what I would like to see is this impleme

[PATCH V2]PR other/70268: map one directory name (old) to another (new) in __FILE__

2016-03-19 Thread Hongxu Jia
Changed in V2: - Rebase to latest master (f958b9e9d4dfed811f36217c800194dd82becf01) - Fix incorrect PR in libcpp/ChangeLog and gcc/ChangeLog //Hongxu >From b8761da98466242e1d18b4e39eb0ea3c6cee4d73 Mon Sep 17 00:00:00 2001 From: Hongxu Jia Date: Wed, 16 Mar 2016 04:55:56 -0700 Subject: [PATCH V

[committed] PR c/70264: fix crash in compatible_locations_p with BUILTINS_LOCATION

2016-03-19 Thread David Malcolm
In r234088 my fix for PR c++/70105 didn't allow for the possibility that when comparing a pair of macro expansion histories that one of the macros in the history might not be located within a line-map, and PR c/70264 reports a crash due to encountering BUILTINS_LOCATION within the traversal. Fixed

Re: C++ PATCH to fix missing warning (PR c++/70194)

2016-03-19 Thread Jason Merrill
On 03/16/2016 08:43 PM, Martin Sebor wrote: @@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, return build3 (VEC_COND_EXPR, type, cmp, minus_one_vec, zero_vec); } +/* Possibly warn about an address never being NULL. */ + +static void +warn_for_null_address (location_t locat

[oacc, testsuite, comitted] Add goacc/kernels-alias-{3,4}.f95

2016-03-19 Thread Tom de Vries
Hi, I've translated the goacc/kernels-alias-{3,4}.c testcases to fortran. Committed to trunk. Thanks, - Tom Add goacc/kernels-alias-{3,4}.f95 2016-03-17 Tom de Vries * gfortran.dg/goacc/kernels-alias-3.f95: New test. * gfortran.dg/goacc/kernels-alias-4.f95: New test. --- gcc/testsuite/

Re: [PATCH][ARM][4.9 Backport] PR target/69875 Fix atomic_loaddi expansion

2016-03-19 Thread Ramana Radhakrishnan
On Wed, Feb 24, 2016 at 11:23 AM, Kyrill Tkachov wrote: > Hi all, > > This is the GCC 4.9 backport of > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01338.html. > The differences are that TARGET_HAVE_LPAE has to be defined in arm.h in a > different way because > the ARM_FSET_HAS_CPU1 mechanism do

Re: [AArch64] Emit square root using the Newton series

2016-03-19 Thread Evandro Menezes
On 03/08/16 16:08, Evandro Menezes wrote: On 02/16/16 14:56, Evandro Menezes wrote: On 12/08/15 15:35, Evandro Menezes wrote: Emit square root using the Newton series 2015-12-03 Evandro Menezes gcc/ * config/aarch64/aarch64-protos.h (aarch64_emit_swsqrt): Declare new

Re: Patch ping

2016-03-19 Thread Jason Merrill
OK. Jason

[AArch64] Add precision choices for the reciprocal square root approximation

2016-03-19 Thread Evandro Menezes
Add precision choices for the reciprocal square root approximation Allow a target to prefer such operation depending on the FP precision. gcc/ * config/aarch64/aarch64-protos.h (AARCH64_EXTRA_TUNE_APPROX_RSQRT): New macro. * config/a

Re: [C PATCH] Prevent -Wunused-value warning with __atomic_fetch_* (PR c/69407)

2016-03-19 Thread Uros Bizjak
On Mon, Mar 7, 2016 at 2:34 PM, Marek Polacek wrote: > On Fri, Mar 04, 2016 at 07:17:46PM +0100, Uros Bizjak wrote: >> Hello! >> >> > This is not a regression but I thought I'd post this anyway. Martin >> > reported >> > that we generate -Wunused-value warnings on the attached testcase, which >>

[PATCH] Change replace_rtx if from is a REG (PR target/70245)

2016-03-19 Thread Jakub Jelinek
Hi! The following testcase is miscompiled on ia32, because a peephole2 calls replace_rtx trying to replace SImode %ecx with SImode %edx, but replace_rtx (unlike e.g. simplify_replace_rtx or validate_replace_rtx), in addition to modifying the rtxes in place (fine in this case) only does pointer equ

Re: Wonly-top-basic-asm

2016-03-19 Thread David Wohlferd
On 3/14/2016 8:28 AM, Bernd Schmidt wrote: The example is not good, as discussed previously, and IMO the best option is to remove it. Otherwise I have no objections to the latest variant. Despite the problems I have with the existing sample, adding the information/warnings is more important t

Re: Please include ada-hurd.diff upstream (try2)

2016-03-19 Thread Arnaud Charlet
> Yes, you are right, somehow I submitted the old patch. Attached is the > updated one, with a proposed ChangeLog entry included. The copyright notices are wrong (or at least incomplete). Arno > > Thanks! > 2016-03-16 Svante Signell > > * gcc-interface/Makefile.in: Add support for x

[PING] [PATCH] c++/65579 - set readonly bit on static constexpr members of templates

2016-03-19 Thread Martin Sebor
I'm looking for a review of the patch below. The problem isn't new (it has existed since at least 4.9) but the fix seems trivial and safe. If it isn't appropriate at this stage please let me know and I'll resubmit it when stage 1 opens. https://gcc.gnu.org/ml/gcc-patches/2016-03/msg00593.html

[gomp-nvptx 3/7] libgomp plugin: make cuMemFreeHost error non-fatal

2016-03-19 Thread Alexander Monakov
Unlike cuMemFree and other resource-releasing functions called on exit, cuMemFreeHost appears to re-report errors encountered in kernel launch. This leads to a deadlock after GOMP_PLUGIN_fatal is reentered. While the behavior on libgomp side is suboptimal (there's no need to call resource-releasin

[COMMITTED] Fix 70240

2016-03-19 Thread Richard Henderson
As discussed in the PR, the fix for 68215 was a bit too aggressive and caused this one. There's a simple alternate fix, first suggested by Richi in 68714, that cures both. Thus I apply one patch and revert another, in order, so that nothing breaks in between yet keeps the two commits separate

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

2016-03-19 Thread Bernd Schmidt
On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. + bitmap_set_bit (seen_insns, INSN_UID (insn)); + if (! INSN_P (insn)) continue

[C PATCH] Fix up composite_types (PR c/70280)

2016-03-19 Thread Jakub Jelinek
Hi! Zdenek reported a compare debug issue, where it is dumping used function prototypes and there is a difference between -g0 and -g in -2: static int BIO_vsnprintf (char *, size_t, const char *, struct *, void, ...); +2: static int BIO_vsnprintf (char *, size_t, const char *, struct *); Th

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

2016-03-19 Thread Jeff Law
On 03/17/2016 03:16 PM, Martin Sebor wrote: static tree cxx_eval_constant_expression (const constexpr_ctx *, tree, - bool, bool *, bool *, tree * = NULL); + bool, bool *, bool *, bool * = NULL, + tree * = NULL);

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

2016-03-19 Thread Jeff Law
On 03/17/2016 03:16 PM, Martin Sebor wrote: gcc-67376.patch PR c++/67376 - [5/6 regression] Comparison with pointer to past-the-end of array fails inside constant expression PR c++/70170 - [6 regression] bogus not a constant expression error comparing pointer to array to null

[Patch, fortran] PR68566 ICE on using unusable array in reshape (double free or corruption)

2016-03-19 Thread Harald Anlauf
Hi, the above ICE is fixed by the following simple/trivial fix: Index: gcc/fortran/simplify.c === --- gcc/fortran/simplify.c (revision 234170) +++ gcc/fortran/simplify.c (working copy) @@ -5163,6 +5163,9 @@ || !is_co

Re: [PATCH V3]PR other/70268: map one directory name (old) to another (new) in __FILE__

2016-03-19 Thread Hongxu Jia
On 03/18/2016 04:05 PM, Hongxu Jia wrote: Please explain why you first alloca() and then strdup the result instead of XNEWVEC BTW, I just refer the implement of -fdebug-prefix-map: In gcc/final.c const char * remap_debug_filename (const char *filename) //

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

2016-03-19 Thread Jason Merrill
On 03/16/2016 08:38 AM, H.J. Lu wrote: FAIL: g++.dg/abi/pr60336-1.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL: g++.dg/abi/pr60336-5.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL: g++.dg/abi/pr60336-6.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL: g++.dg/abi/pr60336

Re: [PATCH, rs6000] Add support for xxpermr and vpermr instructions

2016-03-19 Thread David Edelsohn
On Thu, Mar 17, 2016 at 2:58 PM, Kelvin Nilsen wrote: > > This patch adds support for two new Power9 instructions, xxpermr and vpermr, > providing more efficient vector permutation operations on > little-endian configurations. These new instructions are described in > the Power ISA 3.0 document.

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

2016-03-19 Thread Jeff Law
On 03/17/2016 02:02 PM, Jeff Law wrote: On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. + bitmap_

Emit square root using the Newton series

2016-03-19 Thread Evandro Menezes
2016-03-16 Evandro Menezes Wilco Dijkstra gcc/ * config/aarch64/aarch64-tuning-flags.def (AARCH64_EXTRA_TUNE_APPROX_SQRT_{SF,DF}): New tuning macros. * config/aarch64/aarch64-protos.h (aarch64_emit_approx_rsqrt): Replace with "aarch64_e

Re: [RFC] Getting LTO incremental linking work

2016-03-19 Thread H.J. Lu
On Wed, Nov 25, 2015 at 12:59 AM, Jan Hubicka wrote: > Hi, > PR 67548 is about LTO not supporting incremental linking. I never really > considered our current incremental linking very useful, because it triggers > code generation at the incremental link time basically nullifying any > benefits of

Re: [PATCH] c++/65579 - set readonly bit on static constexpr members of templates

2016-03-19 Thread Jason Merrill
On 03/09/2016 05:09 PM, Martin Sebor wrote: While going through constexpr bugs looking for background on one I'm currently working on I came across bug 65579 - [C++11] gcc requires definition of a static constexpr member even though it is not odr-used. The bug points out that GCC (sometimes) emi

[committed patch] Sync top-level configure.ac with binutils-gdb

2016-03-19 Thread Cary Coutant
I'm committing this patch to sync the top-level configure with binutils-gdb. -cary 2016-03-17 Cary Coutant * configure.ac: Add mips and s390 to the gold target check. * configure: Regenerate. Index: configure =

Re: [PATCH, PR70183] Propagate dump flags in pass_manager::register_pass

2016-03-19 Thread Richard Biener
On Wed, Mar 16, 2016 at 12:26 PM, Tom de Vries wrote: > Hi, > > atm dumpfile vzeroupper is not influenced by the flags in > -fdump-rtl-all-flags. > > The patch fixes this by copying the flags in pass_manager::register_pass. > > OK for stage1 if bootstrap and reg-test succeeds? Ok. Richard. > T

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

2016-03-19 Thread H.J. Lu
On Wed, Mar 16, 2016 at 7:33 AM, Jason Merrill wrote: > On 03/16/2016 07:55 AM, H.J. Lu wrote: >> >> On Tue, Mar 15, 2016 at 7:51 PM, Jason Merrill wrote: >>> >>> On 03/15/2016 08:25 PM, Joseph Myers wrote: On Tue, 15 Mar 2016, H.J. Lu wrote: > On Tue, Mar 15, 2016 at 3:34

Re: [PATCH] Fix 70199

2016-03-19 Thread Richard Henderson
On 03/16/2016 02:40 AM, Richard Biener wrote: I like patch 2 more - btw, you need to add has_forced_label_in_static streaming to lto-streamer-{in,out}.c, just look for has_nonlocal_label streaming. Also has_label_address_in_static_1 is now unused and should be removed. I'd already found the la

Re: [PATCH] PR c/70281: C FE: fix uninitialized range for __builtin_types_compatible_p

2016-03-19 Thread Jeff Law
On 03/18/2016 10:25 AM, David Malcolm wrote: PR c/70281 reports another case where Valgrind identified an uninitialized src_range in a c_expr in the C frontend, this time in the parsing of __builtin_types_compatible_p. For gcc 7 I hope to fix this more robustly (via poisoning the values in a c_e

[PATCH, aarch64] Fix target/70120

2016-03-19 Thread Richard Henderson
With -g, and a code section that ends unaligned, the assembler complains of "unaligned opcodes detected". Except there are no such unaligned opcodes, nor dwarf2 code ranges covering the end of the section, which arguably makes this an assembler bug. However, it's reasonably easy to work around in

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245)

2016-03-19 Thread Bernd Schmidt
On 03/16/2016 01:22 PM, Jakub Jelinek wrote: So, this is what we've converged to on IRC and passed bootstrap/regtest on x86_64-linux and i686-linux. Is this ok for trunk? The explanation was a bit confusing at first, but I think this looks reasonable. The assert worries me, but triggering it

[PATCH] Fix PR c++/70205 (ICE on valid call to qualified static member function)

2016-03-19 Thread Patrick Palka
adjust_result_of_qualified_name_lookup() may erroneously clobber the BASELINK_BINFO of its DECL argument if the BINFO_TYPE of DECL is an ambiguous base of the qualifying scope that's used to refer to DECL. But as the comment in the function suggests, this base ambiguity is not necessarily a problem

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

2016-03-19 Thread Jeff Law
On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. + bitmap_set_bit (seen_insns, INSN_UID (insn)); +

Re: Re: [PATCH][Testsuite] Add --param sra-max-scalarization-size-Ospeed to sra-12.c

2016-03-19 Thread Andre Vieira (lists)
On 21/10/15 16:59, Jeff Law wrote: > On 10/21/2015 09:52 AM, Alan Lawrence wrote: >> gcc.dg/tree-ssa/sra-12.c is skipped on a bunch of targets, including >> AArch64, >> because the default max-scalarization-size depends on MOVE_RATIO, and >> on those >> targets thus ends up being too small for SRA

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

2016-03-19 Thread H.J. Lu
On Thu, Mar 17, 2016 at 8:09 AM, Bernd Schmidt wrote: > On 03/17/2016 04:06 PM, H.J. Lu wrote: >> >> This is the patch I am going to check in. > > > That still mentions darwin which I imagine might not be an exhaustive test. > We can add an effective target, something like ignore_pic_pie, and use

Re: [RFA][PATCH][PR tree-optimization/64058] Improve and stabilize sorting of coalesce pairs

2016-03-19 Thread Jeff Law
On 03/15/2016 08:22 AM, Richard Biener wrote: To work around the narrow API in the comparison function we have to either store additional data in each node or have them available in globals. The former would be horribly wasteful, the latter is just ugly. I choose the latter in the lazy evaluati

[Patch] [x86_64]: minor latency changes for znver1.md

2016-03-19 Thread Kumar, Venkataramanan
Hi Uros, The below patch changes the latency values for fp type load reservations. It passes normal bootstrap and bootstrap with BOOT_CFLAGS="-O2 -g - march=znver1 -mno-clzero -mno-sha " on avx2 target. Also compiled and ran SPEC2006 with -march=znver1 and -Ofast . Ok for trunk? ChangeLog 2

C++ PATCH for c++/70139 (-fno-elide-constructors breaks regex)

2016-03-19 Thread Jason Merrill
The constexpr code for shortcutting trivial copy ctor/op= didn't get updated for the C++14 constexpr implementation, where we need to consider side effects. For GCC 5.4 I'm just going to disable the shortcut. Tested x86_64-pc-linux-gnu, applying to trunk. commit a805189949e8ed36713d5eb78c283a

Re: [PATCH][PR rtl-optimization/70024] Fix argument to CROSSING_JUMP_P

2016-03-19 Thread Jeff Law
On 03/16/2016 11:37 AM, Andreas Schwab wrote: Jeff Law writes: PR rtl-optimization/70024 That's probably a typo. Already fixed. jeff

[gomp-nvptx 4/7] nvptx backend: re-enable line info generation

2016-03-19 Thread Alexander Monakov
* config/nvptx/nvptx.c (nvptx_option_override): Remove custom handling of debug info options. --- gcc/ChangeLog.gomp-nvptx | 5 + gcc/config/nvptx/nvptx.c | 9 - 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/gcc/config/nvptx/nvptx.c b/gcc/config/nvptx/n

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

2016-03-19 Thread Jakub Jelinek
On Thu, Mar 17, 2016 at 11:07:03PM +1030, Alan Modra wrote: > On Thu, Mar 17, 2016 at 12:16:58PM +0100, Jakub Jelinek wrote: > > the rs6000 backend for whatever strange reason I haven't understood > > really wants pointer equality instead of REGNO comparison (even when the > > modes match), one (re

Re: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

2016-03-19 Thread Andre Vieira (lists)
On 23/10/15 12:31, Bernd Schmidt wrote: > On 10/12/2015 11:58 AM, Ulrich Weigand wrote: >> >> Index: gcc/configure.ac >> === >> --- gcc/configure.ac(revision 228530) >> +++ gcc/configure.ac(working copy) >> @@ -1993,7 +1993,7 @

[PATCH, PR tree-optimization/70252] Fix boolean vectors conversion

2016-03-19 Thread Ilya Enkovich
Hi, Current widening and narrowing vectorization functions may work incorrectly for scalar masks because we may have different boolean vector types having the same mode. E.g. vec(4) and vec(8) both have QImode. That means if we need to convert vec(4) into vec(16) we may actually find QImode->HIm

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

2016-03-19 Thread Bernd Schmidt
On 03/17/2016 12:16 PM, Jakub Jelinek wrote: Thus, I've reverted the patch (kept the testcase), and after some discussions on IRC bootstrapped/regtested on x86_64-linux and i686-linux following version, which right now should change behavior just for the i?86 case and nothing else, so shouldn't

Re: [gomp4.1] map clause parsing improvements

2016-03-19 Thread Thomas Schwinge
Hi! On Mon, 19 Oct 2015 12:34:08 +0200, Jakub Jelinek wrote: > On Mon, Oct 19, 2015 at 12:20:23PM +0200, Thomas Schwinge wrote: > > > @@ -77,7 +79,21 @@ enum gomp_map_kind > > > +/* OpenMP 4.1 alias for forced deallocation. */ > > > +GOMP_MAP_DELETE =GOMP_MAP_FORCE_D

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

2016-03-19 Thread Jason Merrill
On 03/16/2016 07:55 AM, H.J. Lu wrote: On Tue, Mar 15, 2016 at 7:51 PM, Jason Merrill wrote: On 03/15/2016 08:25 PM, Joseph Myers wrote: On Tue, 15 Mar 2016, H.J. Lu wrote: On Tue, Mar 15, 2016 at 3:34 PM, Joseph Myers wrote: On Tue, 15 Mar 2016, H.J. Lu wrote: On Tue, Mar 15, 2016 at

Re: [patch] libstdc++/69945 Add __gnu_cxx::__freeres hook

2016-03-19 Thread Mark Wielaard
On Thu, 2016-03-03 at 16:34 +0100, Mark Wielaard wrote: > On Wed, 2016-02-24 at 18:35 +, Jonathan Wakely wrote: > > This adds a new function to libsupc++ which will free the memory still > > in use by the pool used for allocating exceptions when malloc fails. > > > > This is similar to glibc's

[GCC][ARM] Skip tests that assume target supports arm mode, when testing M profiles

2016-03-19 Thread Andre Vieira (lists)
Hello, This patch skips four tests that assume a target supports ARM mode when testing M-profiles. Tested it by running the four tests for A-profiles and M-profiles. Is this ok? Cheers, Andre gcc/testsuite/ChangeLog: 2016-03-17 Andre Vieira * gcc/testsuite/gcc.target/arm/attr-align1.c:

[PATCH] PR testsuite/70150: Check pie_enabled target in PIC tests

2016-03-19 Thread H.J. Lu
We need to check pie_enabled target in PIC tests to support GCC where PIE is enabled by default when configured with --enable-default-pie. OK for master? H.J. --- PR testsuite/70150 * gcc.dg/20020312-2.c (dg-additional-options): Set to "-no-pie" for pie_enabled target.

Re: [PATCH] Fix PR64764

2016-03-19 Thread H.J. Lu
On Wed, Mar 16, 2016 at 9:41 AM, H.J. Lu wrote: > On Wed, Mar 16, 2016 at 9:35 AM, Tom de Vries wrote: >> On 16/03/16 17:15, H.J. Lu wrote: >>> >>> On Wed, Mar 16, 2016 at 9:12 AM, H.J. Lu wrote: >> >> Any particular reason why this test was changed to DOS format? >> >> >> FWIW, the test wa

Re: RFA: PATCH to load_register_parameters for empty structs and sibcalls

2016-03-19 Thread Bernd Schmidt
On 03/16/2016 07:45 PM, Jason Merrill wrote: Discussion of empty class parameter passing ABI led me to notice that r162402 broke sibcalls with arguments of size 0 in some cases. Before that commit, the code read else if ((partial == 0 || args[i].pass_on_stack) && size != 0) { [...]

Re: [PATCH, PR70185] Only finalize dot files that have been initialized

2016-03-19 Thread Richard Biener
On Wed, Mar 16, 2016 at 11:57 AM, Tom de Vries wrote: > Hi, > > Atm, using fdump-tree-all-graph produces invalid dot files: > ... > $ rm *.c.* ; gcc test.c -O2 -S -fdump-tree-all-graph > $ for f in *.dot; do dot -Tpdf $f -o dot.pdf; done > Warning: test.c.006t.omplower.dot: syntax error in line 1

Re: Please include ada-hurd.diff upstream (try2)

2016-03-19 Thread Arnaud Charlet
> > The copyright notices are wrong (or at least incomplete). > > Hi, what is wrong then, copyright years and/or the text? Both. The copyright year should include 2016 and the text should be copyright FSF, not AdaCore. Arno

[gomp4] Merge trunk r234323 (2016-03-18) into gomp-4_0-branch

2016-03-19 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r234351: commit 4514391426e57f78cb3bfd66d09f5065eff66243 Merge: 2d2924a 666094f Author: tschwinge Date: Sat Mar 19 15:34:52 2016 + svn merge -r 232931:234323 svn+ssh://gcc.gnu.org/svn/gcc/trunk git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc

[PATCH] PR lto/70258: [6 Regression] flag_pic is cleared for PIE in lto_post_options

2016-03-19 Thread H.J. Lu
Since PIE implies PIC, we should set flag_pic to flag_pie for PIE in LTO. Tested on x86-64. OK for trunk? H.J. --- PR lto/70258 * lto-lang.c (lto_post_options): Set flag_pic to flag_pie for PIE. --- gcc/lto/lto-lang.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH] Fix PR64764

2016-03-19 Thread H.J. Lu
On Wed, Mar 16, 2016 at 9:12 AM, H.J. Lu wrote: > On Mon, Feb 9, 2015 at 2:30 AM, Tom de Vries wrote: >> On 09-02-15 09:59, Richard Biener wrote: >>> >>> On Thu, 5 Feb 2015, Tom de Vries wrote: >>> On 26-01-15 15:47, Richard Biener wrote: > > Index: gcc/testsuite/gcc.dg/uninit-19.c >

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

2016-03-19 Thread Jason Merrill
On 03/16/2016 03:39 PM, H.J. Lu wrote: On Wed, Mar 16, 2016 at 10:02 AM, H.J. Lu wrote: On Wed, Mar 16, 2016 at 9:58 AM, Jason Merrill wrote: On 03/16/2016 08:38 AM, H.J. Lu wrote: FAIL: g++.dg/abi/pr60336-1.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL: g++.dg/abi/pr60336-5.C s

Re: [PATCH, 16/16] Add libgomp.oacc-fortran/kernels-*.f95

2016-03-19 Thread Thomas Schwinge
Hi! On Wed, 9 Mar 2016 10:19:09 +0100, Tom de Vries wrote: > On 09/11/15 21:12, Tom de Vries wrote: > > This patch adds Fortran oacc kernels execution tests. > > Retested on current trunk. > > Committed, minus the kernels-parallel-loop-data-enter-exit.f95 test. As obvious, committed in r234257

Re: [PATCH PR69489/01]Improve tree ifcvt by storing/tracking DR against its innermost loop bahavior if possible

2016-03-19 Thread Richard Biener
On Wed, Mar 16, 2016 at 5:17 PM, Bin.Cheng wrote: > On Wed, Mar 16, 2016 at 12:20 PM, Richard Biener > wrote: >> >> On Wed, Mar 16, 2016 at 10:59 AM, Bin Cheng wrote: >> > Hi, >> > .. >> > Bootstrap and test on x86_64 and AArch64. Is it OK, not sure if it's GCC >> > 7? >> >> Hmm. > Hi, > T

Re: [PATCH, match] Fix pr68714

2016-03-19 Thread Jakub Jelinek
On Tue, Mar 15, 2016 at 08:09:54AM -0700, Richard Henderson wrote: > Ah, sure. I should have simply tested the reassoc1 dump file, before > generic vector lowering. The testcase fails on i386 (and I assume fails on powerpc too), due to the psABI warnings/notes. I've committed this as obvious. 2

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

2016-03-19 Thread Bernd Schmidt
On 03/17/2016 04:26 PM, H.J. Lu wrote: On Thu, Mar 17, 2016 at 8:23 AM, Bernd Schmidt wrote: On 03/17/2016 04:13 PM, H.J. Lu wrote: We can add an effective target, something like ignore_pic_pie, and use it instead of *-*-darwin*. That should have been done _before_ committing the patch in a

Re: [AArch64] Disable pcrelative_literal_loads with fix-cortex-a53-843419

2016-03-19 Thread Richard Earnshaw (lists)
On 14/03/16 15:34, Christophe Lyon wrote: > On 10 March 2016 at 14:24, James Greenhalgh wrote: >> On Thu, Mar 10, 2016 at 01:37:50PM +0100, Christophe Lyon wrote: >>> On 10 March 2016 at 12:43, James Greenhalgh >>> wrote: On Tue, Jan 26, 2016 at 03:43:36PM +0100, Christophe Lyon wrote:

Re: [PATCH, PR tree-optimization/70252] Fix boolean vectors conversion

2016-03-19 Thread Richard Biener
On Thu, Mar 17, 2016 at 1:02 PM, Ilya Enkovich wrote: > Hi, > > Current widening and narrowing vectorization functions may work > incorrectly for scalar masks because we may have different boolean > vector types having the same mode. E.g. vec(4) and vec(8) > both have QImode. That means if we ne

New French PO file for 'gcc' (version 6.1-b20160131)

2016-03-19 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the French team of translators. The file is available at: http://translationproject.org/latest/gcc/fr.po (This file, 'gcc-6.1-b20160131.fr.po', h

C++ PATCH for c++/70259 (-flifetime-dse vs. empty bases)

2016-03-19 Thread Jason Merrill
The constructor for an empty class can't do the -flifetime-dse clobber because when the class is used as a base it might be assigned the same offset as a real base, so the clobber would mess with real data. Tested x86_64-pc-linux-gnu, applying to trunk. commit e1a5f038350d1881153d8f65359bd883f

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

2016-03-19 Thread H.J. Lu
On Thu, Mar 17, 2016 at 7:18 AM, Bernd Schmidt wrote: > On 03/17/2016 02:59 PM, H.J. Lu wrote: >> >> On Fri, Mar 11, 2016 at 9:09 AM, H.J. Lu wrote: >>> >>> We can't set flag_pie to the default when flag_pic == 0, which may be >>> set by -fno-pic or -fno-PIC, since the default value of flag_pie i

[PATCH]PR other/70268: map one directory name (old) to another (new) in __FILE__

2016-03-19 Thread Hongxu Jia
Similar -fdebug-prefix-map, add option -ffile-prefix-map to map one directory name (old) to another (new) in __FILE__, __BASE_FILE__ and __builtin_FILE(). PR other/70268 * gcc/c-family/c.opt(-ffile-prefix-map=): New option. * gcc/c-family/c-opts.c: Include file-map.h (c_common_handle_option): Han

  1   2   >