ipa-icf::merge TLC

Hi, 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. 1) The logic whether address matters was wrong and ignored symbol aliases. I separated it into symtab_node::address_matters predi

Re: [patch, avr] Tidy up avr-log.c

2015-02-24 16:09 GMT+03:00 Georg-Johann Lay : > avr-log.c and respective macros in avr-protos.h still assume that the > implementation language is C90, i.e. no variadic macros are available. > > This patch cleans up the code from the cumbersome old approach and uses > variadic macros for avr_dump,

Re: [PATCH] Fix up LTO TARGET_OPTION_NODE handling on x86 (PR lto/64374)

On Tue, 24 Feb 2015, Jakub Jelinek wrote: > On Tue, Feb 24, 2015 at 08:48:19PM +0100, Jan Hubicka wrote: > > Thanks, the i386 parts of the patch are OK, but I think you want to add the > > reverse > > transformation, too. I.e. if someone compiles with -fPIC but links without. > > I've only done

Re: Patch ping

On Wed, 25 Feb 2015, Jakub Jelinek wrote: > On Wed, Feb 18, 2015 at 11:00:35AM +0100, Jakub Jelinek wrote: > > On Tue, Feb 17, 2015 at 11:00:14AM +0100, Richard Biener wrote: > > > I'm just looking for a way to make this less of a hack (and the LTO IL > > > less target dependent). Not for GCC 5 f

[PATCH, CHKP, PR target/65183] Avoid wrong pass local data usage

Hi, This patch fixes a case when outdated checker local data is used to process external calls. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2015-02-25 Ilya Enkovich PR target/65183 * tree-chkp.c (chkp_check_lower): Don't check a

Re: [PATCH PR65161]

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. Is it OK for trunk? 2015-02-25 Yuri Rumyantsev PR target/65161 * confi

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

Hi! On Tue, 24 Feb 2015 11:29:51 +, Julian Brown wrote: > On Wed, 4 Feb 2015 15:05:45 + > Julian Brown wrote: > > > The major changes are: [...] Thanks for looking into this! > This is a version of the previously-posted patch to rework > initialisation and support the proposed load/u

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

This patch fixes a problem that has been with us for several years. Variable tracking has no idea about the end of the lifetime of inlined variables, so it keeps on computing locations for them over and over, even though the computed locations make no sense whatsoever because the variable can't eve

Re: [PATCH][wwwdocs] Mention xgene-1 in arm and aarch64, FreeBSD support for arm

On 13/02/15 10:14, Richard Earnshaw wrote: On 13/02/15 09:52, Kyrill Tkachov wrote: Hi all, This patch to changes.html mentions the xgene1 support in GCC 5 for arm and aarch64 and also the FreeBSD support for ARM. Is this ok? The repetitive nature of all these new cpus being added looks rath

[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

Hello, ChangeLog 2015-02-25 Kai Tietz PR tree-optimization/61917 * tree-vect-loop.c (vectorizable_reduction): Allow vect_internal_def without reduction to exit graceful. ChagneLog testsuite/ 2015-02-25 Kai Tietz PR tree-optimization/61917 * gcc.dg/vect/vect-pr61917.c

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

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 b

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

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 Date: Wed Feb 25 15:36:29 2015 +0530 [Patch,microblaze]: Optimized usage of fint i

Re: [PATCH PR65161]

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 ho

Re: [PATCH] Optimize bfi by remove redundant zero_extend

On 13/02/15 17:04, Richard Henderson wrote: On 02/13/2015 08:26 AM, Renlin Li wrote: + /* Complete overlap. We can remove the source ZERO_EXTEND. */ + if (width == inner_size + && (regno < FIRST_PSEUDO_REGISTER) + && HARD_REGNO_MODE_OK (regno, mode)) + { +

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

Hi! On Wed, 11 Feb 2015 15:50:20 +0100, I wrote: > On Wed, 11 Feb 2015 15:44:26 +0100, I wrote: > > If -freorder-blocks-and-partition is active, this results in PTX code > > such as: [...] > Such partitioning might not make a lot of sense for the virtual ISA that > PTX is, but disabling it in nvp

RE: [Patch][ARM]Don't put volatile memory access in IT block for cortex-m7

> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Richard Earnshaw > Sent: Wednesday, February 18, 2015 2:45 AM > To: Terry Guo; gcc-patches@gcc.gnu.org > Cc: Richard Earnshaw; Ramana Radhakrishnan > Subject: Re: [Patch][ARM]

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

On Wed, Feb 25, 2015 at 10:40 AM, Alexandre Oliva wrote: > This patch fixes a problem that has been with us for several years. > Variable tracking has no idea about the end of the lifetime of inlined > variables, so it keeps on computing locations for them over and over, > even though the computed

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

On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz wrote: > Hello, > > ChangeLog > > 2015-02-25 Kai Tietz > > PR tree-optimization/61917 > * tree-vect-loop.c (vectorizable_reduction): Allow > vect_internal_def without reduction to exit graceful. > > ChagneLog testsuite/ > > 2015-02-25 Kai

Re: [Ada] convert GNAT doc to sphinx

On Sun, 22 Feb 2015, Arnaud Charlet wrote: > Two .png files were missing, now added: > > 2015-02-22 Arnaud Charlet > > * doc/gnat_ugn/project-manager-figure.png, > doc/gnat_ugn/rtlibrary-structure.png: New. The maintainer-scripts/update_web_docs_svn scr

Re: [Patch][ARM]Don't put volatile memory access in IT block for cortex-m7

On 25/02/15 10:42, Terry Guo wrote: -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Richard Earnshaw Sent: Wednesday, February 18, 2015 2:45 AM To: Terry Guo; gcc-patches@gcc.gnu.org Cc: Richard Earnshaw; Ramana Radhakrishnan

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 11:57 GMT+01:00 Richard Biener : > On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz wrote: >> Hello, >> >> ChangeLog >> >> 2015-02-25 Kai Tietz >> >> PR tree-optimization/61917 >> * tree-vect-loop.c (vectorizable_reduction): Allow >> vect_internal_def without reduction to exit

Re: [Ada] convert GNAT doc to sphinx

> > Two .png files were missing, now added: > > > > 2015-02-22 Arnaud Charlet > > > > * doc/gnat_ugn/project-manager-figure.png, > > doc/gnat_ugn/rtlibrary-structure.png: New. > > The maintainer-scripts/update_web_docs_svn script is still producing > errors

Re: [Ada] convert GNAT doc to sphinx

On Wed, 25 Feb 2015, Arnaud Charlet wrote: > > > Two .png files were missing, now added: > > > > > > 2015-02-22 Arnaud Charlet > > > > > > * doc/gnat_ugn/project-manager-figure.png, > > > doc/gnat_ugn/rtlibrary-structure.png: New. > > > > The maintainer-scripts

Re: [Ada] convert GNAT doc to sphinx

> 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 as my knowledge goes for this script so I hope

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

On Wed, Feb 25, 2015 at 12:05 PM, Kai Tietz wrote: > 2015-02-25 11:57 GMT+01:00 Richard Biener : >> On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz wrote: >>> Hello, >>> >>> ChangeLog >>> >>> 2015-02-25 Kai Tietz >>> >>> PR tree-optimization/61917 >>> * tree-vect-loop.c (vectorizable_reduc

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

Hi! On Mon, 23 Feb 2015 18:14:35 +0100, Tom de Vries 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 expansion using > >> the apply pragmas. It maximizes factoring out

Re: [Ada] convert GNAT doc to sphinx

On Sun, Feb 22, 2015 at 8:15 PM, Arnaud Charlet wrote: >> Your patch removes these arguments to dircategory: >> ... >> $ git show bf5dffd3a47fe12ace71fe48e87cfb1b9ada1344 | grep dircategory >> +@dircategory >> -@dircategory GNU Ada tools >> -@dircategory GNU Ada tools >> +@dircategory >> ... > > W

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)

On Wed, 25 Feb 2015 10:36:08 +0100 Thomas Schwinge wrote: > Hi! > > On Tue, 24 Feb 2015 11:29:51 +, Julian Brown > wrote: > > Test results look OK, barring a suspected harness issue (lib-83 > > failing with a timeout for nvptx > > However, I'm seeing a class of testsuite regressions: all v

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)

On Wed, Feb 25, 2015 at 10:36:08 +0100, Thomas Schwinge wrote: > > Julian Brown 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]: [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 12:35 GMT+01:00 Richard Biener : > On Wed, Feb 25, 2015 at 12:05 PM, Kai Tietz wrote: >> 2015-02-25 11:57 GMT+01:00 Richard Biener : >>> On Wed, Feb 25, 2015 at 11:06 AM, Kai Tietz wrote: Hello, ChangeLog 2015-02-25 Kai Tietz PR tree-optimizati

Re: [PATCH PR65161]

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 : > > > On Wed, 25 Feb 2015, Yuri Rumyantsev wrote: > >> Hi All, >>

Re: [PATCH PR65161]

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

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: Option overriding in the offloading code path

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

Hello, So, I did full regression-test for following patch: ChangeLog 2015-02-25 Richard Biener Kai Tietz PR tree-optimization/61917 * tree-vect-loop.c (vectorizable_reduction): Allow vect_internal_def without reduction to exit graceful. ChagneLog testsuite/ 2015-02-25 Ka

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

On Wed, Feb 25, 2015 at 2:10 PM, Kai Tietz wrote: > Hello, > > So, I did full regression-test for following patch: > > ChangeLog > > 2015-02-25 Richard Biener > Kai Tietz > > PR tree-optimization/61917 > * tree-vect-loop.c (vectorizable_reduction): Allow > vect_internal_def wi

[PATCH] Fix PR65204

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 PR tree-optimization/65204 * tree-ssa-ccp.c (evaluate_stmt): Always evaluate address t

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

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 Date: Wed Feb 25 10:24:40 2015 +0800 2015-02-25 Xingxing Pan * config/arm/arm-c

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

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 PR target/65167 * gcc/config/i386/i386.c (ix86_function_arg

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

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 neon_f

Re: [PATCH PR65161]

I modified patch accordingly to Alexander comments. Is it OK for trunk? 2015-02-25 15:38 GMT+03:00 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_i

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

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. (n

Re: [PATCH PR65161]

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

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

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 an

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

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

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

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 recog_

Re: [PATCH PR65161]

Hi All, Here is updated patch to fix ICE. Is it OK for trunk? 2015-02-25 Yuri Rumyantsev 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. 2015-02-25 17:04 GMT+03:0

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

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 PR target/65184

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

On 02/24/15 22:47, augustine.sterl...@gmail.com wrote: On Tue, Feb 24, 2015 at 4:45 PM, Max Filippov 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, your patch is OK for

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

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: /ho

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

Hello! > 2015-02-25 Ilya Enkovich > > 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 > > PR target/65167 > * gcc.target/i386/pr65

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

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: [PATCH, CHKP, i386, PR target/65167] Avoid motion of bounds args during scheduling

On Wed, Feb 25, 2015 at 3:47 PM, Uros Bizjak wrote: > Hello! > >> 2015-02-25 Ilya Enkovich >> >> 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-0

[patch]: Fix PR target/64212

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 clon

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

On Wed, Feb 25, 2015 at 6:39 AM, Jeff Law wrote: > > Done. > jeff Thanks!

Re: [patch] Fix ICE on unaligned record field

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.

Re: [Ada] convert GNAT doc to sphinx

> > 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

C++ PATCH for useless NOTE_INSN_DELETED_DEBUG_LABELs

This isn't the main problem in debug/58315, but when looking at it I saw a bunch of useless # DEBUG => 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 inf

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

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 was

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

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 n

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

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 after

Re: [Ada] convert GNAT doc to sphinx

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 appa

Re: [Ada] convert GNAT doc to sphinx

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

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

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 -felimin

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

On 25-02-15 12:40, Thomas Schwinge wrote: Hi! On Mon, 23 Feb 2015 18:14:35 +0100, Tom de Vries 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 expansion using the apply prag

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

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 for PR ipa/64693

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 d7472e55b345214d

Re: Patch ping

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 r

Re: [patch]: Fix PR target/64212

Applied at revision 22098 to trunk. Jan approved patch on IRC. Regards, Kai

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

On Wed, Feb 25, 2015 at 5:10 AM, Kai Tietz wrote: > Hello, > > So, I did full regression-test for following patch: > > ChangeLog > > 2015-02-25 Richard Biener > Kai Tietz > > PR tree-optimization/61917 > * tree-vect-loop.c (vectorizable_reduction): Allow > vect_internal_def wi

Re: [PATCH] Fix for PR ipa/64693

> 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 te

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

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)

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 scenar

Re: ipa-icf::merge TLC

> 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 >

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

On Tue, Feb 17, 2015 at 4:47 AM, H.J. Lu wrote: > On Mon, Feb 16, 2015 at 5:24 AM, H.J. Lu wrote: >> On Mon, Feb 16, 2015 at 5:18 AM, Jakub Jelinek wrote: >>> On Mon, Feb 16, 2015 at 05:15:02AM -0800, H.J. Lu wrote: On Mon, Feb 16, 2015 at 4:30 AM, H.J. Lu wrote: > On Mon, Feb 16, 201

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

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 gcc/testsuite: * gcc.target/powerpc/direct-move.h: Include

[PATCH] ICF: handle correctly hard register variables

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 Date: Wed, 25 Feb 2015 18:26:

[patch]: Fix regression caused by fix for 61917

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 Kai Tietz PR tr

Re: [PATCH] Fix for PR ipa/64693

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 exp

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

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 nex

Re: ipa-icf::merge TLC

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 Chr

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

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 t

Re: ipa-icf::merge TLC

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 th

Re: [PATCH] Fix for PR ipa/64693

> > >From dd240028726cb7fdc777acd0b6d14c4f89aed714 Mon Sep 17 00:00:00 2001 > From: mliska > Date: Thu, 19 Feb 2015 16:08:09 +0100 > Subject: [PATCH 1/3] Fix PR ipa/64693 > > 2015-02-25 Martin Liska > Jan Hubicka > > * gcc.dg/ipa/ipa-icf-26.c: Update test. > * gcc.dg

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 ..."

On Mon, Jul 28, 2014 at 7:02 PM, Richard Henderson wrote: > On 07/26/2014 05:35 AM, Uros Bizjak wrote: >> On Mon, May 2, 2011 at 9:21 AM, Uros Bizjak wrote: >> >>> It looks that GP relative relocations do not fit anymore into GPREL16 >>> reloc, so bootstrap on alpha hosts fail in stage2 with "re

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 ..."

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

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.

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

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 i

Re: Fix alignment propagation

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) > >

Re: [PATCH PR65161]

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

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

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 import libstdcxx.v6 File "/usr/lib/i386-linux-gnu/../../share/gcc-5/python/libstdcxx/v

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

On Feb 25, 2015, Richard Biener 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 ceased to exist, so what is t

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

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 t

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

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

On Feb 25, 2015, Jakub Jelinek 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 (assuming you have c

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

On Feb 25, 2015, at 1: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

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

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 ch

[SH] Adding some peepholes (PR 61142)

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 garbag

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

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 b

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

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, app

  1   2   >