Re: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-02-25 Thread Julian Brown
On Wed, 25 Feb 2015 10:36:08 +0100 Thomas Schwinge tho...@codesourcery.com wrote: Hi! On Tue, 24 Feb 2015 11:29:51 +, Julian Brown jul...@codesourcery.com wrote: Test results look OK, barring a suspected harness issue (lib-83 failing with a timeout for nvptx However, I'm seeing a

Re: [PATCH PR65161]

2015-02-25 Thread Alexander Monakov
On Wed, 25 Feb 2015, Yuri Rumyantsev wrote: Here is updated patch accordingly to Alexander comments. BTW another function using HID interface is do_reorder_for_imul and it is called from ix86_sched_reorder. do_reorder_for_imul uses dependency list iteration macros, which use HDID, not

Re: ipa-icf::merge TLC

2015-02-25 Thread Markus Trippelsdorf
On 2015.02.25 at 09:38 +0100, Jan Hubicka wrote: this patch reorganize sem_function::merge and sem_variable::merge. I read the code in detail and found several issues that are fixed in the following patch. I gave your patch a quick spin. It breaks Chromium. Its protocol buffer compiler gets

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Kai Tietz
2015-02-25 12:35 GMT+01:00 Richard Biener richard.guent...@gmail.com: On Wed, Feb 25, 2015 at 12:05 PM, Kai Tietz ktiet...@googlemail.com wrote: 2015-02-25 11:57 GMT+01:00 Richard Biener richard.guent...@gmail.com: On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz ktiet...@googlemail.com wrote:

Re: [PATCH] gcc/reload.c: Initialize several arrays before use them in find_reloads()

2015-02-25 Thread Jeff Law
On 02/24/15 22:47, augustine.sterl...@gmail.com wrote: On Tue, Feb 24, 2015 at 4:45 PM, Max Filippov jcmvb...@gmail.com wrote: Sterling, I was referring Jeff's patch, do you say that his patch is not the proper fix? No, I was thinking of Chen's patch. Jeff's patch is the right one. Jeff,

Re: [PATCH, 2/2][ARM]: New CPU support for Marvell Whitney

2015-02-25 Thread Xingxing Pan
Hi, This patch expanding the following RTL types. And it has been merged to the latest code base. (neon_logic): Expand to neon_logic_reg and neon_logic_imm. (neon_logic_q): Expand to neon_logic_reg_q and neon_logic_imm_q. (neon_from_gp): Expand to neon_from_gp and

Re: [PATCH, 2/2][ARM]: New CPU support for Marvell Whitney

2015-02-25 Thread Xingxing Pan
On 02/25/2015 09:42 PM, Xingxing Pan wrote: Hi, This patch expanding the following RTL types. And it has been merged to the latest code base. (neon_logic): Expand to neon_logic_reg and neon_logic_imm. (neon_logic_q): Expand to neon_logic_reg_q and neon_logic_imm_q.

Re: [PATCH PR65161]

2015-02-25 Thread Alexander Monakov
On Wed, 25 Feb 2015, Yuri Rumyantsev wrote: I modified patch accordingly to Alexander comments. Is it OK for trunk? If possible, please add a short comment explaining why a shortcut is necessary, for example HID is not populated during selective scheduling. OK for trunk from selective

[PATCH, CHKP, i386, PR target/65184] Fix pass_by_reference for MS ABI for bounds

2015-02-25 Thread Ilya Enkovich
Hi, Currenly ix86_pass_by_reference may return 1 for bounds if MS ABI is used. This patch explicitly says bounds are never passed by reference. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2015-02-25 Ilya Enkovich ilya.enkov...@intel.com

Re: [Ada] convert GNAT doc to sphinx

2015-02-25 Thread Richard Biener
On Sun, Feb 22, 2015 at 8:15 PM, Arnaud Charlet char...@adacore.com wrote: Your patch removes these arguments to dircategory: ... $ git show bf5dffd3a47fe12ace71fe48e87cfb1b9ada1344 | grep dircategory +@dircategory -@dircategory GNU Ada tools -@dircategory GNU Ada tools +@dircategory ...

[PATCH] Fix PR65204

2015-02-25 Thread Richard Biener
This fixes missed tracking of alignment of non-invariant addresses in CCP. Bootstrapped and tested on x86_64-unknown-linux-gnu, queued for GCC 6. Richard. 2015-02-25 Richard Biener rguent...@suse.de PR tree-optimization/65204 * tree-ssa-ccp.c (evaluate_stmt): Always evaluate

[PATCH, CHKP, i386, PR target/65167] Avoid motion of bounds args during scheduling

2015-02-25 Thread Ilya Enkovich
Hi, This patch adds support for bounds registers into args recognition mechanism used by scheduler. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2015-02-25 Ilya Enkovich ilya.enkov...@intel.com PR target/65167 *

[Ping v2] [PATCH PR64820] Fix ASan UAR detection fails on 32-bit targets if SSP is enabled.

2015-02-25 Thread Maxim Ostapenko
On 02/16/2015 10:58 AM, Maxim Ostapenko wrote: Hi, when testing I noticed, that if compile with both -fsanitize=address and -fstack-protector for 32-bit architectures and run with ASAN_OPTIONS=detect_stack_use_after_return=1, libsanitizer fails with: ==7299==AddressSanitizer CHECK failed:

Re: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-02-25 Thread Ilya Verbin
On Wed, Feb 25, 2015 at 10:36:08 +0100, Thomas Schwinge wrote: Julian Brown jul...@codesourcery.com wrote: OK for gomp4 branch? I could commit Ilya's patch there too if so. I'll leave the decision to Jakub, but, what about trunk? As Ilya indicated in

Re: [PATCH, 2/2][ARM]: New CPU support for Marvell Whitney

2015-02-25 Thread James Greenhalgh
On Wed, Feb 25, 2015 at 01:42:39PM +, Xingxing Pan wrote: Hi, This patch expanding the following RTL types. And it has been merged to the latest code base. (neon_logic): Expand to neon_logic_reg and neon_logic_imm. (neon_logic_q): Expand to neon_logic_reg_q and

[patch, avr, committed]: Fix PR65196 (ice-checking)

2015-02-25 Thread Georg-Johann Lay
http://gcc.gnu.org/r220963 http://gcc.gnu.org/r220964 http://gcc.gnu.org/r220965 Applied this obvious fix for ICE with checking enabled (recog_memoized used with invalid rtx, e.g. jump_table_data). Johann PR target/65196 * config/avr/avr.c (avr_adjust_insn_length): Call

Re: Option overriding in the offloading code path

2015-02-25 Thread Bernd Schmidt
On 02/25/2015 11:28 AM, Thomas Schwinge wrote: Am I on the right track with my assumption that it is correct that nvptx.c:nvptx_option_override is not invoked in the offloading code path, so we'd need a new target hook (?) to consolidate/override the options in this scenario? I'm surprised by

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Richard Biener
On Wed, Feb 25, 2015 at 2:10 PM, Kai Tietz ktiet...@googlemail.com wrote: Hello, So, I did full regression-test for following patch: ChangeLog 2015-02-25 Richard Biener rguent...@suse.de Kai Tietz kti...@redhat.com PR tree-optimization/61917 * tree-vect-loop.c

[wwwdocs] List IPA-CP of alignments in changes.html

2015-02-25 Thread Martin Jambor
Hi, I'd like to commit the following to gcc-5/changes.html so that IPA-CP alignment propagation is listed among other new features. OK? Thanks, Martin Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v

Re: [PATCH PR65161]

2015-02-25 Thread Yuri Rumyantsev
Here is updated patch accordingly to Alexander comments. BTW another function using HID interface is do_reorder_for_imul and it is called from ix86_sched_reorder. Is it OK for trunk? 2015-02-25 13:26 GMT+03:00 Alexander Monakov amona...@ispras.ru: On Wed, 25 Feb 2015, Yuri Rumyantsev wrote:

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread Kai Tietz
Hello, So, I did full regression-test for following patch: ChangeLog 2015-02-25 Richard Biener rguent...@suse.de Kai Tietz kti...@redhat.com PR tree-optimization/61917 * tree-vect-loop.c (vectorizable_reduction): Allow vect_internal_def without reduction to exit graceful.

Re: [patch 1/2][ARM]: New CPU support for Marvell Whitney

2015-02-25 Thread Xingxing Pan
Hi, This patch merges pipeline description for marvell-whitney to latest code base. Is it OK for trunk? -- Regards, Xingxing commit 83974dde8d9f773df1004aa1d5e3b05d8a33f5e0 Author: Xingxing Pan xxing...@marvell.com Date: Wed Feb 25 10:24:40 2015 +0800 2015-02-25 Xingxing Pan

Re: [PATCH PR65161]

2015-02-25 Thread Yuri Rumyantsev
I modified patch accordingly to Alexander comments. Is it OK for trunk? 2015-02-25 15:38 GMT+03:00 Alexander Monakov amona...@ispras.ru: On Wed, 25 Feb 2015, Yuri Rumyantsev wrote: Here is updated patch accordingly to Alexander comments. BTW another function using HID interface is

Re: [PATCH PR65161]

2015-02-25 Thread Yuri Rumyantsev
Hi All, Here is updated patch to fix ICE. Is it OK for trunk? 2015-02-25 Yuri Rumyantsev ysrum...@gmail.com PR target/65161 * config/i386/i386.c (ix86_sched_reorder): Skip instruction reordering for selective scheduling. gcc/testsuite/ChangeLog * gcc.target/i386/pr65161.c: New test.

Re: [wwwdocs] List IPA-CP of alignments in changes.html

2015-02-25 Thread Gerald Pfeifer
On Wed, 25 Feb 2015, Martin Jambor wrote: I'd like to commit the following to gcc-5/changes.html so that IPA-CP alignment propagation is listed among other new features. OK? Looks good, thank you! (Just watch out for overly long lines.) Gerald

Re: [Ada] convert GNAT doc to sphinx

2015-02-25 Thread Joseph Myers
On Wed, 25 Feb 2015, Arnaud Charlet wrote: So you need to update the find command therein not to remove anything that's part of the sources for this documentation, and possibly update -I options for building manuals as well. I've added a -I gcc/gcc/ada/doc/gnat_ugn there, that's as far

Re: [PATCH, CHKP, i386, PR target/65167] Avoid motion of bounds args during scheduling

2015-02-25 Thread Uros Bizjak
On Wed, Feb 25, 2015 at 3:47 PM, Uros Bizjak ubiz...@gmail.com wrote: Hello! 2015-02-25 Ilya Enkovich ilya.enkov...@intel.com PR target/65167 * gcc/config/i386/i386.c (ix86_function_arg_regno_p): Support bounds registers. (avoid_func_arg_motion): Add dependencies for BNDSTX insns.

C++ PATCH for useless NOTE_INSN_DELETED_DEBUG_LABELs

2015-02-25 Thread Jason Merrill
This isn't the main problem in debug/58315, but when looking at it I saw a bunch of useless # DEBUG L0 = NULL lines, which turned out to be deleted debug labels notes for the cdtor_label created in start_preparsed_function. Since this is an internal, unnamed label, we shouldn't have debug

[patch, avr-tiny]: Fix handling of constant addresses.

2015-02-25 Thread Georg-Johann Lay
The current avr-gcc ICEs in avr.c::tiny_valid_direct_memory_access_range because XEXP (op, 0) is used on op which are not MEM_P (e.g. REG or SUBREG). If op is MEM_P then INTVAL might be used for on RTXes which are not CONST_INT, e.g. CONST. Anyway, using such functions in insn conditions is

Re: [PR58315] reset inlined debug vars at return-to point

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 11:54:16AM +0100, Richard Biener wrote: Regstrapped on x86_64-linux-gnu and i686-linux-gnu. Ok to install? But code-motion could still move stmts from the inlined functions across these resets? That said - shouldn't this simply performed by proper var-tracking

Re: [patch] Fix ICE on unaligned record field

2015-02-25 Thread Martin Jambor
Hi Eric and Richard, On Tue, Jan 06, 2015 at 06:07:12PM +0100, Eric Botcazou wrote: Martin, I suppose that could be done by something like the following, which I have tested only very mildly so far, in particular I have not double checked that get_inner_reference is cfun-agnostic. The

[hsa] Do not ICE when regallocating function with zero pseudoregisters

2015-02-25 Thread Martin Jambor
Hi, the new HSA register allocator ICEs when it tries to resize a vector to zero length, which is something that our vectors do not take well, when it processes a simple function which just returns a constant and does not actually use any registers. Fixed thusly, committed to the hsa branch

Re: [Ada] convert GNAT doc to sphinx

2015-02-25 Thread Joseph Myers
On Wed, 25 Feb 2015, Arnaud Charlet wrote: See the existing code to handle Sphinx documentation for the JIT. That's a good reference. We'll need a more recent version of sphinx than 1.0 though (at least 1.2.2, or even better, 1.3b2 which is the version we use at AdaCore). 1.0 is

Re: [PATCH] gcc/reload.c: Initialize several arrays before use them in find_reloads()

2015-02-25 Thread augustine.sterl...@gmail.com
On Wed, Feb 25, 2015 at 6:39 AM, Jeff Law l...@redhat.com wrote: Done. jeff Thanks!

Re: [PATCH, CHKP, i386, PR target/65167] Avoid motion of bounds args during scheduling

2015-02-25 Thread Uros Bizjak
Hello! 2015-02-25 Ilya Enkovich ilya.enkov...@intel.com PR target/65167 * gcc/config/i386/i386.c (ix86_function_arg_regno_p): Support bounds registers. (avoid_func_arg_motion): Add dependencies for BNDSTX insns. gcc/testsuite/ 2015-02-25 Ilya Enkovich ilya.enkov...@intel.com PR

[patch]: Fix PR target/64212

2015-02-25 Thread Kai Tietz
Hi, The issue here is a pe-coff target specific thing that dllimported-symbols have an noninterposable, and an interposable part. The dllimport address itself is not interposable, but its stubbing function/var is. So the hook binds_to_local has to return false for dllimport, nevertheless for

Re: [Ada] convert GNAT doc to sphinx

2015-02-25 Thread Arnaud Charlet
I've added a -I gcc/gcc/ada/doc/gnat_ugn there, that's as far as my knowledge goes for this script so I hope this is enough. Well, since by default the find command deletes all files except those known to be documentation sources, you need at least to change it not to delete those

PING Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Aldy Hernandez
On 02/19/2015 10:41 AM, Jakub Jelinek wrote: On Thu, Feb 19, 2015 at 10:33:20AM -0800, Aldy Hernandez wrote: Well, any PCH file we generate will have some sort of early DIE in it (at the very least the compilation unit DIE) and we will read these in at PCH read-in time, obliterating whatever

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Jason Merrill
On 02/19/2015 11:50 AM, Jakub Jelinek wrote: Wouldn't it be better to disable PCH reading if -feliminate-dwarf2-dups is used? In the abstract, perhaps, but given https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53118 I'd prefer to disable the useless thing. :) We might actually disable

Re: [PATCH] Use DO_PRAGMA in libgomp.oacc-c-c++-common/reduction-1.c

2015-02-25 Thread Tom de Vries
On 25-02-15 12:40, Thomas Schwinge wrote: Hi! On Mon, 23 Feb 2015 18:14:35 +0100, Tom de Vries tom_devr...@mentor.com wrote: On 23-02-15 17:08, Jakub Jelinek wrote: On Mon, Feb 23, 2015 at 04:52:56PM +0100, Tom de Vries wrote: The only thing I'm not sure about is the two-level pragma

Re: [PATCH] Fix for PR ipa/64693

2015-02-25 Thread Martin Liška
On 02/20/2015 07:39 PM, Jan Hubicka wrote: Hello. There's updated version that reflects how should we handle congruence classes that have any address reference. Patch can bootstrap x86_64-linux-pc and no new regression is introduced? Ready for trunk? Thanks, Martin From

Re: [patch]: [Bug tree-optimization/61917] [4.9/5 Regression] ICE on valid code at -O3 on x86_64-linux-gnu in vectorizable_reduction, at tree-vect-loop.c:4913

2015-02-25 Thread H.J. Lu
On Wed, Feb 25, 2015 at 5:10 AM, Kai Tietz ktiet...@googlemail.com wrote: Hello, So, I did full regression-test for following patch: ChangeLog 2015-02-25 Richard Biener rguent...@suse.de Kai Tietz kti...@redhat.com PR tree-optimization/61917 * tree-vect-loop.c

Re: ipa-icf::merge TLC

2015-02-25 Thread Jan Hubicka
On 2015.02.25 at 09:38 +0100, Jan Hubicka wrote: this patch reorganize sem_function::merge and sem_variable::merge. I read the code in detail and found several issues that are fixed in the following patch. I gave your patch a quick spin. It breaks Chromium. Its protocol buffer compiler

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Aldy Hernandez
On 02/25/2015 07:59 AM, Jason Merrill wrote: On 02/19/2015 11:50 AM, Jakub Jelinek wrote: Wouldn't it be better to disable PCH reading if -feliminate-dwarf2-dups is used? In the abstract, perhaps, but given https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53118 I'd prefer to disable the useless

Re: Option overriding in the offloading code path (was: [nvptx] -freorder-blocks-and-partition, -freorder-functions)

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 11:28:12AM +0100, Thomas Schwinge wrote: Am I on the right track with my assumption that it is correct that nvptx.c:nvptx_option_override is not invoked in the offloading code path, so we'd need a new target hook (?) to consolidate/override the options in this scenario?

[4.8 branch] PATCH: PR middle-end/53623: [4.7/4.8 Regression] sign extension is effectively split into two x86-64 instructions

2015-02-25 Thread H.J. Lu
On Tue, Feb 17, 2015 at 4:47 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Feb 16, 2015 at 5:24 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Feb 16, 2015 at 5:18 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Feb 16, 2015 at 05:15:02AM -0800, H.J. Lu wrote: On Mon, Feb 16, 2015 at 4:30

Re: Patch ping

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 10:10:52AM +0100, Richard Biener wrote: Oops, totally forgot about this one. Shouldn't + default: + error (unsupported mode %s\n, mname); be a fatal_error ()? After all if we hit this but continue we'll Ok, I'll change it. stream random

Re: [patch]: Fix PR target/64212

2015-02-25 Thread Kai Tietz
Applied at revision 22098 to trunk. Jan approved patch on IRC. Regards, Kai

Re: [PATCH] Fix for PR ipa/64693

2015-02-25 Thread Jan Hubicka
Hello Honza. I've updated the patch so that your notes are resolved. Moreover, I've added comparison for interposable symbols that are either target of reference or are called by a function. Please read the patch to verify the comparison is as you expected. I'm going to run testsuite.

[PATCH, rs6000 testsuite] Fix failures for implicit function declaration

2015-02-25 Thread Pat Haugen
The following patch fixes excess errors failures for implicit function declarations (memcmp/random) for the direct-move-*/pack01 tests. Tested on powerpc64le-unknown-linux-gnu. Committed as obvious. 2015-02-25 Pat Haugen pthau...@us.ibm.com gcc/testsuite: *

[PATCH] ICF: handle correctly hard register variables

2015-02-25 Thread Martin Liška
Hello. This patch adds support for hard register variables in ICF and it's pre-approved by Honza, I'm going to install the patch. No regressing on x86_64-linux-pc. Thanks, Martin From eff93050904e0aeaf26b47fb1d1e8eeb803f9af6 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Wed, 25

[patch]: Fix regression caused by fix for 61917

2015-02-25 Thread Kai Tietz
Hi, The patch didn't handled the case for dt being vect_constant_def, where of course the reduc_def_stmt is NULL. By checking for NULL before testing for PHI, we now fallback for such cases to old behavior and return in the next if-statment. 2015-02-25 Richard Biener rguent...@suse.de Kai

Re: [PATCH] Fix for PR ipa/64693

2015-02-25 Thread Martin Liška
On 02/25/2015 06:00 PM, Jan Hubicka wrote: Hello Honza. I've updated the patch so that your notes are resolved. Moreover, I've added comparison for interposable symbols that are either target of reference or are called by a function. Please read the patch to verify the comparison is as you

Re: ipa-icf::merge TLC

2015-02-25 Thread Martin Liška
On 02/25/2015 06:15 PM, Jan Hubicka wrote: On 2015.02.25 at 09:38 +0100, Jan Hubicka wrote: this patch reorganize sem_function::merge and sem_variable::merge. I read the code in detail and found several issues that are fixed in the following patch. I gave your patch a quick spin. It breaks

Re: Fix alignment propagation

2015-02-25 Thread Martin Jambor
Hi, On Fri, Feb 20, 2015 at 07:22:02PM +0100, Jan Hubicka wrote: +/* Decrease alignment info DEST to be at most CUR. */ + +static bool +decrease_alignment (ipa_alignment *dest, ipa_alignment cur) +{ + bool changed = false; + + if (!cur.known) +return false;

Re: [PATCH PR65161]

2015-02-25 Thread Uros Bizjak
On Wed, Feb 25, 2015 at 3:24 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Hi All, Here is updated patch to fix ICE. Is it OK for trunk? 2015-02-25 Yuri Rumyantsev ysrum...@gmail.com PR target/65161 * config/i386/i386.c (ix86_sched_reorder): Skip instruction reordering for selective

Re: [patch]: Fix regression caused by fix for 61917

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 07:13:55PM +0100, Kai Tietz wrote: Hi, The patch didn't handled the case for dt being vect_constant_def, where of course the reduc_def_stmt is NULL. By checking for NULL before testing for PHI, we now fallback for such cases to old behavior and return in the next

Re: ipa-icf::merge TLC

2015-02-25 Thread Markus Trippelsdorf
On 2015.02.25 at 19:32 +0100, Martin Liška wrote: On 02/25/2015 06:15 PM, Jan Hubicka wrote: On 2015.02.25 at 09:38 +0100, Jan Hubicka wrote: this patch reorganize sem_function::merge and sem_variable::merge. I read the code in detail and found several issues that are fixed in the

Re: [PATCH, alpha]: Fix PR/47230 [4.6/4.7 Regression] gcc fails to bootstrap on alpha in stage2 with relocation truncated to fit: GPREL16 against ...

2015-02-25 Thread Richard Henderson
On 02/25/2015 09:02 AM, Uros Bizjak wrote: The patch was tested on alpha-linux-gnu and alphaev68-linux-gnu for all default languages plus obj-c++ and go. OK for mainline? Ok. Thanks. r~

Re: [patch] fix PR65048: check that jump-thread paths are still valid

2015-02-25 Thread Sebastian Pop
Jeff Law wrote: Registering FSM jump thread: (10, 12) (12, 13) (13, 15) (15, 3) [ snip ] Registering FSM jump thread: (7, 10) (10, 12) (12, 13) (13, 14) What I'm having a bit of trouble wrapping my head around is how can those two paths both be valid when you register them?

Re: [PATCH, alpha]: Fix PR/47230 [4.6/4.7 Regression] gcc fails to bootstrap on alpha in stage2 with relocation truncated to fit: GPREL16 against ...

2015-02-25 Thread Uros Bizjak
On Mon, Jul 28, 2014 at 7:02 PM, Richard Henderson r...@redhat.com wrote: On 07/26/2014 05:35 AM, Uros Bizjak wrote: On Mon, May 2, 2011 at 9:21 AM, Uros Bizjak ubiz...@gmail.com wrote: It looks that GP relative relocations do not fit anymore into GPREL16 reloc, so bootstrap on alpha hosts

Re: [PATCH] Fix for PR ipa/64693

2015-02-25 Thread Jan Hubicka
From dd240028726cb7fdc777acd0b6d14c4f89aed714 Mon Sep 17 00:00:00 2001 From: mliska mli...@suse.cz Date: Thu, 19 Feb 2015 16:08:09 +0100 Subject: [PATCH 1/3] Fix PR ipa/64693 2015-02-25 Martin Liska mli...@suse.cz Jan Hubicka hubi...@ucw.cz *

Re: [patch] fix PR65048: check that jump-thread paths are still valid

2015-02-25 Thread Sebastian Pop
Jeff Law wrote: On 02/18/15 15:27, Sebastian Pop wrote: The dumps for the FSM threads are a bit sparse -- they don't show the entire path. That makes it much harder to see what's going on. Would a patch improving the FSM dumps ok to commit separately to trunk? Most definitely. I realize

Re: [PATCH 1/n] OpenMP 4.0 offloading infrastructure

2015-02-25 Thread Ilya Verbin
On Fri, Feb 20, 2015 at 15:50:53 +0100, Jakub Jelinek wrote: On Fri, Feb 20, 2015 at 03:41:40PM +0100, Thomas Schwinge wrote: Well, but users (like Jakub, for example) ;-) may decide to build the offloading compilers without specifying --enable-languages, and that'll then default to include

Re: [PR58315] reset inlined debug vars at return-to point

2015-02-25 Thread Alexandre Oliva
On Feb 25, 2015, Richard Biener richard.guent...@gmail.com wrote: But code-motion could still move stmts from the inlined functions across these resets? Sure, just like it could still move stmts across any other debug stmts. Once you return from a function, it's as if all of its variables

[patch, libstdc++] Use explicit relative imports for the pretty printers

2015-02-25 Thread Matthias Klose
When gdb is linked/used with Python 3, import of the pretty printers fails: Traceback (most recent call last): File /usr/share/gdb/auto-load/usr/lib/i386-linux-gnu/libstdc++.so.6.0.21-gdb.py, line 58, in module import libstdcxx.v6 File

Re: [patch] fix PR65048: check that jump-thread paths are still valid

2015-02-25 Thread Jeff Law
On 02/25/15 12:18, Sebastian Pop wrote: Jeff Law wrote: On 02/18/15 15:27, Sebastian Pop wrote: The dumps for the FSM threads are a bit sparse -- they don't show the entire path. That makes it much harder to see what's going on. Would a patch improving the FSM dumps ok to commit separately

Re: [PR58315] reset inlined debug vars at return-to point

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 06:17:33PM -0300, Alexandre Oliva wrote: My measurements, for a not particularly unusual testcase, showed an overall reduction of 63% in compile time, as indicated yesterday. Now, who should bear the burden of collecting evidence to back up the claims against the

C++ PATCH for c++/65209 (linkage of anonymous namespace)

2015-02-25 Thread Jason Merrill
There are two issues with this testcase: one, when we internalize a decl because it involves an anonymous namespace we need to clear DECL_COMDAT, now that we're setting it early. Also, our handling of pointers to local functions also needs to handle references. Tested x86_64-pc-linux-gnu,

Re: [patch] fix PR65048: check that jump-thread paths are still valid

2015-02-25 Thread Jeff Law
On 02/13/15 16:50, Sebastian Pop wrote: Hi, the attached patch fixes PR65048 by checking before jump-threading that a path to be threaded is still valid: as the testcase shows, there may be paths that are not connected anymore because the cfg has changed in a previous jump-thread. PR

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Jason Merrill
On 02/25/2015 12:02 PM, Aldy Hernandez wrote: + if (flag_eliminate_dwarf2_dups) +{ + warning (0, ignoring unimplemented option -feliminate-dwarf2-dups); + flag_eliminate_dwarf2_dups = 0; +} I think we only want to disable it for C++, not all languages. Jason

Re: [PR58315] reset inlined debug vars at return-to point

2015-02-25 Thread Alexandre Oliva
On Feb 25, 2015, Jakub Jelinek ja...@redhat.com wrote: various tools and users really want to be able to inspect variables and parameters on the return statement. This patch won't affect the return statement. The resets are at the return-to statement; if you stop at the return statement

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Mike Stump
On Feb 25, 2015, at 1:13 PM, Jason Merrill ja...@redhat.com wrote: On 02/25/2015 12:02 PM, Aldy Hernandez wrote: + if (flag_eliminate_dwarf2_dups) +{ + warning (0, ignoring unimplemented option -feliminate-dwarf2-dups); + flag_eliminate_dwarf2_dups = 0; +} I think we

[SH] Adding some peepholes (PR 61142)

2015-02-25 Thread Oleg Endo
Hi, These are the peepholes as mentioned in PR 65153 and in PR 61142. They try to wallpaper some bad RA choices and reduce the CSiBE code size by approx. 3.9K bytes. A problem I ran into with this one is that the peephole2 pass drops REG_INC notes, which makes the following passes produce

Re: [patch] fix PR65048: check that jump-thread paths are still valid

2015-02-25 Thread Jeff Law
On 02/25/15 11:37, Sebastian Pop wrote: Jeff Law wrote: Registering FSM jump thread: (10, 12) (12, 13) (13, 15) (15, 3) [ snip ] Registering FSM jump thread: (7, 10) (10, 12) (12, 13) (13, 14) What I'm having a bit of trouble wrapping my head around is how can those two paths

[PATCH, rs6000] Fix PR65171

2015-02-25 Thread Bill Schmidt
Hi, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65171 identifies a bug when compiling portions of the Boost library. The problem occurs in the swap analysis phase. Any operand that is TImode or a subreg of TImode is supposed to disable the swap optimization for the web of instructions that

Re: [PATCH] rs6000: Fix HTM tcheck assembly encoding

2015-02-25 Thread Peter Bergner
On Mon, 2015-02-23 at 13:42 -0500, David Edelsohn wrote: On Mon, Feb 23, 2015 at 1:26 PM, Peter Bergner berg...@vnet.ibm.com wrote: This is broken on 4.9 and 4.8, so can we get those fixed as well? Yes, please backport. I committed this to trunk on Adhemerval's behalf as revision 220992.

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

2015-02-25 Thread Terry Guo
On Tue, Feb 17, 2015 at 11:39 AM, Terry Guo flame...@gmail.com wrote: On Sun, Feb 15, 2015 at 7:35 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: Hi Terry, I still think this is stage1 material. + /* Don't combine if dest contains a user specified register and i3 contains +

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Jason Merrill
On 02/25/2015 09:16 PM, Aldy Hernandez wrote: + warning (0, ignoring unimplemented option -feliminate-dwarf2-dups); Similarly, I'd rather say it's broken for C++. OK with that change. Jason

Re: [PATCH] rs6000: Fix HTM tcheck assembly encoding

2015-02-25 Thread Peter Bergner
On Mon, 2015-02-23 at 13:42 -0500, David Edelsohn wrote: Again, a testcase is highly encouraged. Actually, this additional change to the test case would be enough to have the assembler catch the bad opcode mnemonic. Should I just go ahead and make this change? I still plan on adding a test

Re: [PATCH] rs6000: Fix HTM tcheck assembly encoding

2015-02-25 Thread David Edelsohn
On Wed, Feb 25, 2015 at 9:42 PM, Peter Bergner berg...@vnet.ibm.com wrote: On Mon, 2015-02-23 at 13:42 -0500, David Edelsohn wrote: Again, a testcase is highly encouraged. Actually, this additional change to the test case would be enough to have the assembler catch the bad opcode mnemonic.

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Aldy Hernandez
On 02/25/2015 06:47 PM, Jason Merrill wrote: On 02/25/2015 09:16 PM, Aldy Hernandez wrote: + warning (0, ignoring unimplemented option -feliminate-dwarf2-dups); Similarly, I'd rather say it's broken for C++. OK with that change. Interesting. As in ignoring broken C++ option

Re: [PATCH] rs6000: Fix HTM tcheck assembly encoding

2015-02-25 Thread Peter Bergner
On Wed, 2015-02-25 at 21:57 -0500, David Edelsohn wrote: Okay. Ok, committed as revision 220998 with the following ChangeLog entry: * gcc.target/powerpc/htm-builtin-1.c (dg-do) Change to assemble. (dg-options): Add -save-temps. (dg-final): Add cleanup-saved-temps. I'll

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-25 Thread Aldy Hernandez
On 02/25/2015 01:13 PM, Jason Merrill wrote: On 02/25/2015 12:02 PM, Aldy Hernandez wrote: + if (flag_eliminate_dwarf2_dups) +{ + warning (0, ignoring unimplemented option -feliminate-dwarf2-dups); + flag_eliminate_dwarf2_dups = 0; +} I think we only want to disable it for

Re: [Patch,microblaze]: Optimized usage of fint instruction.

2015-02-25 Thread Michael Eager
On 02/25/15 02:20, Ajit Kumar Agarwal wrote: Hello All: Please find the patch for the optimized usage of fint instruction changes. No regression is seen in the deja GNU tests. commit ed4dc0b96bf43c200cacad97f73a98ab7048e51b Author: Ajit Kumar Agarwal ajitkum@xhdspdgnu.(none) Date: Wed Feb

Re: [PR58315] reset inlined debug vars at return-to point

2015-02-25 Thread Alexandre Oliva
On Feb 25, 2015, Jakub Jelinek ja...@redhat.com wrote: On Wed, Feb 25, 2015 at 06:17:33PM -0300, Alexandre Oliva wrote: My measurements, for a not particularly unusual testcase, showed an overall reduction of 63% in compile time, as indicated yesterday. Now, who should bear the burden of

Re: [PATCH, rs6000] Fix PR65171

2015-02-25 Thread David Edelsohn
On Wed, Feb 25, 2015 at 5:08 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: Hi, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65171 identifies a bug when compiling portions of the Boost library. The problem occurs in the swap analysis phase. Any operand that is TImode or a subreg of

Re: [Patch,microblaze]: Optimized usage of pcmp conditional instruction.

2015-02-25 Thread Michael Eager
On 02/25/15 02:19, Ajit Kumar Agarwal wrote: Hello All: Please find the patch for the optimized usage of pcmp instructions in microblaze. No regressions is seen In deja GNU tests. There are many testcases that are already there in deja GNU to check the generation of pcmpne/pcmpeq instructions

[Patch, fortran] PR65024 - [4.9/5 Regression] [OOP] ICE concerning unlimited polymorphic pointer

2015-02-25 Thread Paul Richard Thomas
Dear All, This patch has something of a band aid flavour about it. However, the more I look at it the more I like it and it cannot do any harm. In any case, I spent a silly amount of time trying to understand why this component fails to get its backend_decl in the usual way and failed. That it is

[PATCH] ubsan: remove bogus check for flexible array members

2015-02-25 Thread Martin Uecker
Hi, this patch removes a bogus check for flexible array members which prevents array references to be instrumented in some interesting cases. Arrays accessed through pointers are now instrumented correctly. The check was unnecessary because flexible arrays are not instrumented anyway because

Re: [PATCH] ubsan: remove bogus check for flexible array members

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 10:01:07PM -0800, Martin Uecker wrote: this patch removes a bogus check for flexible array members which prevents array references to be instrumented in some interesting cases. Arrays accessed through pointers are now instrumented correctly. The check was unnecessary

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

2015-02-25 Thread Segher Boessenkool
On Tue, Feb 17, 2015 at 11:39:34AM +0800, Terry Guo wrote: On Sun, Feb 15, 2015 at 7:35 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: Hi Terry, I still think this is stage1 material. + /* Don't combine if dest contains a user specified register and i3 contains +

[PATCH][AArch64]: Fix rtl type in aarch64.md.

2015-02-25 Thread Xingxing Pan
Hi, This patch fix the type of movmode_aarch64 in aarch64.md. Is it OK for trunk? -- Regards, Xingxing [AArch64] Fix define_insn type in aarch64.md. 2015-02-26 Xingxing Pan xxing...@marvell.com * config/aarch64/aarch64.md: (movmode_aarch64): Change type to

Re: [PR58315] reset inlined debug vars at return-to point

2015-02-25 Thread Jakub Jelinek
On Wed, Feb 25, 2015 at 09:01:09PM -0300, Alexandre Oliva wrote: On Wed, Feb 25, 2015 at 06:17:33PM -0300, Alexandre Oliva wrote: My measurements, for a not particularly unusual testcase, showed an overall reduction of 63% in compile time, as indicated yesterday. Now, who should bear the

Re: [PATCH, 2/2][ARM]: New CPU support for Marvell Whitney

2015-02-25 Thread Xingxing Pan
On 02/25/2015 10:20 PM, James Greenhalgh wrote: On Wed, Feb 25, 2015 at 01:42:39PM +, Xingxing Pan wrote: Hi, This patch expanding the following RTL types. And it has been merged to the latest code base. (neon_logic): Expand to neon_logic_reg and neon_logic_imm.

Re: [PATCH] ubsan: remove bogus check for flexible array members

2015-02-25 Thread Marek Polacek
On Thu, Feb 26, 2015 at 07:36:54AM +0100, Jakub Jelinek wrote: On Wed, Feb 25, 2015 at 10:01:07PM -0800, Martin Uecker wrote: this patch removes a bogus check for flexible array members which prevents array references to be instrumented in some interesting cases. Arrays accessed through

Re: [patch 1/2][ARM]: New CPU support for Marvell Whitney

2015-02-25 Thread Xingxing Pan
On 02/25/2015 09:32 PM, Xingxing Pan wrote: Hi, This patch merges pipeline description for marvell-whitney to latest code base. Is it OK for trunk? Refactor the commit message. -- Regards, Xingxing Add pipeline description for marvell-whitney. 2015-02-26 Xingxing Pan xxing...@marvell.com

[Patch,microblaze]: Optimized usage of pcmp conditional instruction.

2015-02-25 Thread Ajit Kumar Agarwal
Hello All: Please find the patch for the optimized usage of pcmp instructions in microblaze. No regressions is seen In deja GNU tests. There are many testcases that are already there in deja GNU to check the generation of pcmpne/pcmpeq instructions and are used to check the validity. commit

[Patch,microblaze]: Optimized usage of fint instruction.

2015-02-25 Thread Ajit Kumar Agarwal
Hello All: Please find the patch for the optimized usage of fint instruction changes. No regression is seen in the deja GNU tests. commit ed4dc0b96bf43c200cacad97f73a98ab7048e51b Author: Ajit Kumar Agarwal ajitkum@xhdspdgnu.(none) Date: Wed Feb 25 15:36:29 2015 +0530 [Patch,microblaze]:

Re: [PATCH PR65161]

2015-02-25 Thread Alexander Monakov
On Wed, 25 Feb 2015, Yuri Rumyantsev wrote: Hi All, I prepared new patch which includes test-case. I can't agree with patch proposed by Alexander since other functions doing ready list reordering also use HID interface, so I put escape check in ix86_sched_reorder. I don't see how that

  1   2   >