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

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 1/2][ARM]: New CPU support for Marvell Whitney

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 * config/arm/arm

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

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 throu

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

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

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

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 unnece

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

Hi, This patch fix the type of mov_aarch64 in aarch64.md. Is it OK for trunk? -- Regards, Xingxing [AArch64] Fix define_insn type in aarch64.md. 2015-02-26 Xingxing Pan * config/aarch64/aarch64.md: (mov_aarch64): Change type to neon_to_gp. diff --git a/gcc

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

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 of

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

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

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

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

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

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 a

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

On Wed, Feb 25, 2015 at 9:42 PM, Peter Bergner 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. Should I > just

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

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

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

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

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 cas

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

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

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

On Tue, Feb 17, 2015 at 11:39 AM, Terry Guo wrote: > On Sun, Feb 15, 2015 at 7:35 PM, Segher Boessenkool > wrote: >> Hi Terry, >> >> I still think this is stage1 material. >> >>> + /* Don't combine if dest contains a user specified register and i3 >>> contains >>> + ASM_OPERANDS, because th

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

On Feb 25, 2015, Jakub Jelinek 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 collecting ev

Re: [PATCH, rs6000] Fix PR65171

On Wed, Feb 25, 2015 at 5:08 PM, Bill Schmidt 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 TImode is > supposed to di

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

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 Date: Wed Feb 25 15:36:29 2015 +0530

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

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

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

On Mon, 2015-02-23 at 13:42 -0500, David Edelsohn wrote: > On Mon, Feb 23, 2015 at 1:26 PM, Peter Bergner 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. I'll commit it to

[PATCH, rs6000] Fix PR65171

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 incl

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

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 t

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

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

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

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

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

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 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: [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, 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, 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] 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: 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 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 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 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: [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

[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

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

[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

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 >

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: [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: [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]: [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 PR target/64212

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

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 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: [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] 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: [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: [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: [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

[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

[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

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

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

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

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: [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!

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

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

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

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

[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 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, 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_

[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

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

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

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

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

[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 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] 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]: [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

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: 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: 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: [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: [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]: [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: 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: 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: [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: [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: [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: [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

  1   2   >