[PATCH] Fixup PR69719 fix

The following patch improves the fix for PR69719 after spending another hour in trying to understand that code (and revisiting the original patch postings). The code assumes that dr_a1 is left of dr_a2 which is not always the case (sorting doesn't guarantee that), if that's not the case we can ob

RE: [PATCH] [ARC] Add single/double IEEE precission FPU support.

> > In the expand: > > 18: cc:CC_FPU=cmp(r159:DF,r162:DF) > > 19: r163:SI=cc:CC_FPU<0 > > 20: r161:QI=r163:SI#0 > > 21: r153:SI=zero_extend(r161:QI) > > 22: cc:CC_ZN=cmp(r153:SI,0) > > 23: pc={(cc:CC_ZN!=0)?L28:pc} > > > > Then after combine we get this: > > 18: cc:CC_FP

[patch] Fix timevar internal consistency failure

Hi, I just ran into a timevar internal consistency failure with -ftime-report: Execution times (seconds) phase setup : 0.00 ( 0%) usr 0.00 ( 0%) sys 0.01 ( 0%) wall 114 kB ( 0%) ggc phase parsing : 0.29 ( 7%) usr 0.01 ( 9%) sys 0.30 ( 7%) wall [...] rep

Re: libgcc: On AIX, increase chances to find landing pads for exceptions

On 02/08/2016 02:59 PM, David Edelsohn wrote: > Runtime linking is disabled by default on AIX, and I disabled it for > libstdc++. For large applications mainly developed on/for Linux I do prefer/need runtime linking even on AIX. Still I do believe there is no AIX-based reason to leave runtime li

Re: [patch] Fix timevar internal consistency failure

On Wed, Feb 10, 2016 at 10:55 AM, Eric Botcazou wrote: > Hi, > > I just ran into a timevar internal consistency failure with -ftime-report: > > Execution times (seconds) > phase setup : 0.00 ( 0%) usr 0.00 ( 0%) sys 0.01 ( 0%) wall > 114 kB ( 0%) ggc > phase parsing :

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: > Hi all, > > As part of the target attributes and pragmas support for GCC 6 I changed the > aarch64 port to emit a .arch assembly directive for each function that > describes the architectural features used by that function. This is

Re: [PATCH PR69652, Regression]

Thanks Richard for your comments. I changes algorithm to remove dead scalar statements as you proposed. Bootstrap and regression testing did not show any new failures on x86-64. Is it OK for trunk? Changelog: 2016-02-10 Yuri Rumyantsev PR tree-optimization/69652 * tree-vect-loop.c (optimize_m

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

Hi James, On 10/02/16 10:11, James Greenhalgh wrote: On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: Hi all, As part of the target attributes and pragmas support for GCC 6 I changed the aarch64 port to emit a .arch assembly directive for each function that describes the archite

Re: [PATCH][AArch64] Only update assembler .arch directive when necessary

On Wed, Feb 10, 2016 at 10:32:16AM +, Kyrill Tkachov wrote: > Hi James, > > On 10/02/16 10:11, James Greenhalgh wrote: > >On Thu, Feb 04, 2016 at 01:50:31PM +, Kyrill Tkachov wrote: > >>Hi all, > >> > >>As part of the target attributes and pragmas support for GCC 6 I changed the > >>aarch6

Re: [patch] libstdc++/69116 Constrain std::valarray functions and operators

On 22/01/16 21:15 +, Jonathan Wakely wrote: This is a regression, caused by the front end starting to diagnose the invalid library instantiations more eagerly. The fix seems simple and safe, so I plan to backport it to the branches too. Committed to the branches too.

Re: [Patch, fortran, pr67451, v1] [5/6 Regression] ICE with sourced allocation from coarray

Hi all, unfortunately was my last patch for pr67451 not perfect and introduced regressions occurring on s390(x) and with the sanitizer. These were caused, because when taking the array specs from the source=-expression also its attributes, like coarray state and so on where taken from there. This

Re: [PATCH] PR rtl-optimization/64081: Enable RTL loop unrolling for duplicated exit blocks and back edges.

Hi, Here is a quick status update. (Which comes a bit late due to bisection efforts) This patch still causes bootrstrap failure on AIX when applied on top of r219827. I tried to bisect first commit eliminating AIX problem - it may be useful anyway - but my current results seem misleading. Therefor

[PATCH PR68021]Set ratio to 1 when computing the value of biv cand by itself

Hi, This is another way to fix PR68021, and I think it's the least intrusive way. The issue is triggered in a special case in which cand is a original biv, and use denotes the value of the biv itself. In this case, the use is added specifically for the original biv, as a result, get_computatio

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

Hi! Ping. On Thu, 04 Feb 2016 15:47:25 +0100, I wrote: > Here is the patch re-worked for trunk. Instead of passing > -foffload-force in the affected libgomp test cases, I instead chose to > have them expect the warning. This way, we're testing more in line to > what users will be doing, and we'

Re: [PATCH] PR rtl-optimization/64081: Enable RTL loop unrolling for duplicated exit blocks and back edges.

On 02/10/2016 12:34 PM, Alexander Fomin wrote: This patch still causes bootrstrap failure on AIX when applied on top of r219827. I tried to bisect first commit eliminating AIX problem - it may be useful anyway - but my current results seem misleading. Therefore, I'll to continue the investigatio

RE: [PATCH] [ARC] Add single/double IEEE precission FPU support.

> That sound like a bug. Have you looked more closely what's going on? Right, I found it. Forgot to set the C_MODE for CC_FPU* modes in the arc_mode_class[]. I will prepare a new patch with the proper handling. Thanks!

[PATCH] Fix PR69291, RTL if-conversion bug

In this case if-conversion sees if () (set (reg/v:SI 224 [ ]) (plus:SI (plus:SI (reg/v:SI 160 [ mod_tlen ]) (reg/v:SI 224 [ ])) (const_int 11 [0xb]) else (set (reg/v:SI 224 [ ]) (plus:SI (plus:SI (reg/v:SI 160 [ mod_tlen ]) (reg

Re: [PATCH] s390: Add -fsplit-stack support

On 04/02/16 13:44, Marcin Kościelnicki wrote: On 03/02/16 18:27, Ulrich Weigand wrote: Marcin KoÅ›cielnicki wrote: libgcc/ChangeLog: * config.host: Use t-stack and t-stack-s390 for s390*-*-linux. * config/s390/morestack.S: New file. * config/s390/t-stack-s390: New file. * gene

Re: [PATCH PR69652, Regression]

On Wed, Feb 10, 2016 at 11:26 AM, Yuri Rumyantsev wrote: > Thanks Richard for your comments. > I changes algorithm to remove dead scalar statements as you proposed. > > Bootstrap and regression testing did not show any new failures on x86-64. > Is it OK for trunk? Ok. Thanks, Richard. > Changel

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

On 02/10/2016 12:49 PM, Thomas Schwinge wrote: Hi! Ping. I think this has to be considered after gcc-6. In general, what's the state of OpenACC these days? I'm slightly confused by the interface between offloaded code and libgomp. It looks like you're collecting avoid-offloading flags per

Re: libgcc: On AIX, increase chances to find landing pads for exceptions

On Wed, Feb 10, 2016 at 1:52 AM, Michael Haubenwallner wrote: > > On 02/08/2016 02:59 PM, David Edelsohn wrote: >> Runtime linking is disabled by default on AIX, and I disabled it for >> libstdc++. > > For large applications mainly developed on/for Linux I do prefer/need > runtime linking even on

Re: [PATCH PR68021]Set ratio to 1 when computing the value of biv cand by itself

On Wed, Feb 10, 2016 at 12:34 PM, Bin Cheng wrote: > Hi, > This is another way to fix PR68021, and I think it's the least intrusive way. > The issue is triggered in a special case in which cand is a original biv, > and use denotes the value of the biv itself. In this case, the use is added >

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On 02/10/2016 02:04 PM, Richard Biener wrote: where noce_try_store_flag_constants identifies (plus:SI (reg/v:SI 160 [ mod_tlen ]) (reg/v:SI 224 [ ])) as "common" and then tries to detect the case where setting the result would clobber that value. It doesn't seem to expect anything e

RE: [PATCH] [ARC] Add single/double IEEE precission FPU support.

Please find attached the amended patch for FPU instructions. Ok to apply? 0001-ARC-Add-single-double-IEEE-precission-FPU-support.patch Description: 0001-ARC-Add-single-double-IEEE-precission-FPU-support.patch

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On Wed, 10 Feb 2016, Bernd Schmidt wrote: > On 02/10/2016 02:04 PM, Richard Biener wrote: > > where noce_try_store_flag_constants identifies > > > > (plus:SI (reg/v:SI 160 [ mod_tlen ]) > > (reg/v:SI 224 [ ])) > > > > as "common" and then tries to detect the case where setting the > >

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On 02/10/2016 02:33 PM, Richard Biener wrote: But if you prefer I can instead test the following Index: gcc/ifcvt.c === --- gcc/ifcvt.c (revision 233262) +++ gcc/ifcvt.c (working copy) @@ -1274,7 +1274,7 @@ noce_try_store_flag_cons

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On Wed, 10 Feb 2016, Richard Biener wrote: > On Wed, 10 Feb 2016, Bernd Schmidt wrote: > > > On 02/10/2016 02:04 PM, Richard Biener wrote: > > > where noce_try_store_flag_constants identifies > > > > > > (plus:SI (reg/v:SI 160 [ mod_tlen ]) > > > (reg/v:SI 224 [ ])) > > > > > > as "c

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On 02/10/2016 02:35 PM, Richard Biener wrote: Index: gcc/ifcvt.c === --- gcc/ifcvt.c (revision 233262) +++ gcc/ifcvt.c (working copy) @@ -1274,7 +1274,8 @@ noce_try_store_flag_constants (struct no && CONST_INT_P (XEXP (a, 1)

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On Wed, 10 Feb 2016, Bernd Schmidt wrote: > On 02/10/2016 02:35 PM, Richard Biener wrote: > > > Index: gcc/ifcvt.c > > === > > --- gcc/ifcvt.c (revision 233262) > > +++ gcc/ifcvt.c (working copy) > > @@ -1274,7 +1274,8 @@ noce_try_st

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On 02/10/2016 02:50 PM, Richard Biener wrote: On Wed, 10 Feb 2016, Bernd Schmidt wrote: On 02/10/2016 02:35 PM, Richard Biener wrote: Index: gcc/ifcvt.c === --- gcc/ifcvt.c (revision 233262) +++ gcc/ifcvt.c (working copy) @@ -127

Patch ping

Hi! I'd like to ping a P1 patch: PR ipa/69241, PR c++/69649 https://gcc.gnu.org/ml/gcc-patches/2016-02/msg00192.html Jakub

Re: [PATCH PR68021]Set ratio to 1 when computing the value of biv cand by itself

On Wed, Feb 10, 2016 at 1:27 PM, Richard Biener wrote: > > On Wed, Feb 10, 2016 at 12:34 PM, Bin Cheng wrote: > > Hi, > > This is another way to fix PR68021, and I think it's the least intrusive > > way. The issue is triggered in a special case in which cand is a original > > biv, and use deno

Re: [PATCH] Fix PR69291, RTL if-conversion bug

On Wed, 10 Feb 2016, Bernd Schmidt wrote: > On 02/10/2016 02:50 PM, Richard Biener wrote: > > On Wed, 10 Feb 2016, Bernd Schmidt wrote: > > > > > On 02/10/2016 02:35 PM, Richard Biener wrote: > > > > > > > Index: gcc/ifcvt.c > > > > ===

Re: [patch] Fix timevar internal consistency failure

Hi, On Wed, 10 Feb 2016, Richard Biener wrote: > > The problem is that TV_PHASE_DBGINFO is now nested within > > TV_PHASE_OPT_GEN, which violates the above mutual exclusivity > > requirement. Therefore the attached patch simply gets rid of > > TV_PHASE_DBGINFO (as well as of the sibling TV_PH

Re: Patch ping

On Wed, 10 Feb 2016, Jakub Jelinek wrote: > Hi! > > I'd like to ping a P1 patch: > PR ipa/69241, PR c++/69649 > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg00192.html Ok. Thanks, Richard. > Jakub > > -- Richard Biener SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Graha

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

Hi! On Wed, 10 Feb 2016 14:25:50 +0100, Bernd Schmidt wrote: > On 02/10/2016 12:49 PM, Thomas Schwinge wrote: > > [...] > > I think this has to be considered after gcc-6. Hmm, I see. > In general, what's the > state of OpenACC these days? Much improved compared to GCC 5. :-) Anything speci

Re: Use plain -fopenacc to enable OpenACC kernels processing

Hi! Will this patch be acceptable for GCC trunk in the current development stage? In its current incarnation, this patch depends on my 'Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"' patch,

[PATCH, testsuite]: Add -mcpu=ev4 for alpha for gcc.dg/tree-ssa/sra-{18,19}.c

Hello! Due to various move cost functions, the expected transformation is triggered only for ev4. 2016-02-10 Uros Bizjak * gcc.dg/tree-ssa/sra-17.c: Add -mcpu=ev4 for target alpha*-*-*. * gcc.dg/tree-ssa/sra-18.c: Ditto. Tested on alphaev6-linux-gnu and committed to mainline SVN. Ur

Re: [patch] Fix timevar internal consistency failure

On Wed, 2016-02-10 at 15:15 +0100, Michael Matz wrote: > Hi, > > On Wed, 10 Feb 2016, Richard Biener wrote: > > > > The problem is that TV_PHASE_DBGINFO is now nested within > > > TV_PHASE_OPT_GEN, which violates the above mutual exclusivity > > > requirement. Therefore the attached patch simp

libgcc patch committed: Align stack in __stack_split_initialize

PR 68562 points out that the x86 stack is misaligned when __stack_split_initialize calls __generic_morestack_set_initial_sp, causing crashes with the trunk glibc. This patch fixes the problem by aligning the stack. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline a

Re: [patch] Fix timevar internal consistency failure

Hi, On Wed, 10 Feb 2016, David Malcolm wrote: > > +static timevar_id_t global_phase; > > FWIW I like the idea, but could this be a private field within class > timer, rather than a global? Sure, consider the patch amended accordingly. Ciao, Michael.

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

On 02/10/2016 03:39 PM, Thomas Schwinge wrote: Yes, we need a hammer that big: we have to ensure consistency between data regions on the device and code offloading to the device, as otherwise we'll very easily run into inconsistencies, because of the non-shared memory. In the general case, it's

C++ PATCH for c++/10200 (DR 141, template name lookup after ->)

After . or ->, when we see a name followed by < we look for a template name, first in the scope of the object expression and then in the enclosing scope. DR 141 clarified that when we look in the enclosing scope, we only consider class templates, since there's no way a non-member function temp

Re: C++ PATCH for c++/69657 (abs not inlined)

On 02/09/2016 03:29 PM, Rainer Orth wrote: This patch broke Solaris bootstrap (seen on i386-pc-solaris2.12): Fixed by pruning hidden names from the lookup result in more places. Jason commit 09bb9e19d3f284c2f9d8bf57959c99334363c3c9 Author: Jason Merrill Date: Tue Feb 9 16:20:46 2016 -0500

Small C++ PATCH for debug_tree(cp_expr)

David's cp_expr change made use of the 'pt' gdb macro less convenient in some cases, so I'm adding a debug_tree overload to handle it. commit 48d6e68c6b43b11d4f0bd696f19b490358954937 Author: Jason Merrill Date: Mon Feb 8 23:46:14 2016 -0500 * ptree.c (debug_tree): Implement for cp_expr.

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

Hi! On Wed, 10 Feb 2016 16:27:40 +0100, Bernd Schmidt wrote: > On 02/10/2016 03:39 PM, Thomas Schwinge wrote: > > > Yes, we need a hammer that big: we have to ensure consistency between > > data regions on the device and code offloading to the device, as > > otherwise we'll very easily run into

Re: [PATCH] PR driver/69265: improved suggestions for various misspelled options

On 02/09/2016 09:44 PM, David Malcolm wrote: This is a bug in a new feature, so it isn't a regression as such, but it's fairly visible, and I believe the fix is relatively low-risk (error-handling of typos of command-line options). This also now covers PR driver/69453 (and its duplicate PR drive

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

On 02/10/2016 05:23 PM, Thomas Schwinge wrote: Why? A user of GCC has no intrinsic interest in getting OpenACC kernels constructs' code offloaded; the user wants his code to execute as fast as possible. If you consider the whole of OpenACC kernels code offloading as a compiler optimization, the

[wwwdocs] New bits to porting_to

Some minor issues I noticed. Ok? Index: porting_to.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/porting_to.html,v retrieving revision 1.7 diff -u -r1.7 porting_to.html --- porting_to.html 9 Feb 2016 21:06:32 - 1.7 +

Re: [Patch, fortran, pr67451, v1] [5/6 Regression] ICE with sourced allocation from coarray

> Hi all, > > unfortunately was my last patch for pr67451 not perfect and introduced > regressions occurring on s390(x) and with the sanitizer. These were > caused, because when taking the array specs from the source=-expression > also its attributes, like coarray state and so on where taken from

Re: [PATCH, PR67709 ] Don't call call_cgraph_insertion_hooks in simd_clone_create

The patch fixes the PR on x86_64-apple-darwin15. > OK for stage1 trunk? What it the reason to delay the fix for a couple of months? Thanks for working on the issue. Dominique

Re: [wwwdocs] New bits to porting_to

On 10/02/16 17:46 +0100, Marek Polacek wrote: +int +foo (unsigned x) +{ + abs (x); Let's make this "return abs (x);" so we don't have a missing return. +The std::auto_ptr template class was deprecated in C++11, so GCC s/template class/class template/

Re: [PATCH, PR67709 ] Don't call call_cgraph_insertion_hooks in simd_clone_create

On 10/02/16 17:55, Dominique d'Humières wrote: The patch fixes the PR on x86_64-apple-darwin15. OK for stage1 trunk? What it the reason to delay the fix for a couple of months? We're in stage4 ( https://gcc.gnu.org/ml/gcc/2016-01/msg00168.html ): ... trunk is in regression and documentati

Re: [wwwdocs] New bits to porting_to

On Wed, Feb 10, 2016 at 05:14:40PM +, Jonathan Wakely wrote: > On 10/02/16 17:46 +0100, Marek Polacek wrote: > >+int > >+foo (unsigned x) > >+{ > >+ abs (x); > > Let's make this "return abs (x);" so we don't have a missing return. Ok. > >+The std::auto_ptr template class was deprecated in

Re: [PATCH][RFC][Offloading] Fix PR68463

Hi! On Tue, Jan 19, 2016 at 16:32:13 +0300, Ilya Verbin wrote: > On Tue, Jan 19, 2016 at 10:36:28 +0100, Jakub Jelinek wrote: > > On Tue, Jan 19, 2016 at 09:57:01AM +0100, Richard Biener wrote: > > > On Mon, 18 Jan 2016, Ilya Verbin wrote: > > > > On Fri, Jan 15, 2016 at 09:15:01 +0100, Richard Bi

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

Hi! On Wed, 10 Feb 2016 17:37:30 +0100, Bernd Schmidt wrote: > On 02/10/2016 05:23 PM, Thomas Schwinge wrote: > > Why? A user of GCC has no intrinsic interest in getting OpenACC kernels > > constructs' code offloaded; the user wants his code to execute as fast as > > possible. > > > > If you con

[patch] c++/61198 backport to gcc-4.9

I wanted to backport r232232 (aka 2edb91b1) which helps with the compile-time regression tracked by PR60976. On the gcc-4.9 branch it produces lots of ICEs due to PR61198, which was only fixed for gcc-5. This backports the PR61198 fix to the 4.9 branch, which resolves the ICEs I'm seeing and so w

Re: [PATCH, stage1] Better error recovery for merge-conflict markers

On Mon, 2016-02-08 at 10:07 +0100, Bert Wesarg wrote: > David, > > On Thu, Apr 9, 2015 at 10:29 AM, Bert Wesarg < > bert.wes...@googlemail.com> wrote: > > Hi David, > > > > > Various tools that operate on source code files will inject > > > markers > > > into them when an unfixable conflict occur

Re: [PATCH][cilkplus] fix c++ implicit conversions with cilk_spawn (PR/69024, PR/68997)

On 01/20/2016 10:57 AM, Ryan Burn wrote: This patch follows on from https://gcc.gnu.org/ml/gcc-patches/2015-12/msg02142.html As discussed, it creates a separate function cilk_cp_detect_spawn_and_unwrap in gcc/cp to handle processing cilk_spawn expressions for c++ and adds support for implicit co

Patch to fix PR69148

The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69148 The patch was successfully bootstrapped and tested on x86/x86-64 and s390x. Committed as rev. 233283 I'll wait for a few days before committing it into gcc-5 branch. Index: ChangeLog

libgo patch committed: Support pkg-config in go tool

Michael Hudson-Doyle has written a patch to support pkg-config for gccgo in the go tool. The patch has not been committed to the master sources, because they are in a freeze for the Go 1.6 release. I've decided to pick it up and commit it to the gccgo sources, so that it is available for the GCC

gcc-5-branch backports

Hi! I've committed following backports of my trunk commits to gcc-5-branch after bootstrapping/regtesting them on x86_64-linux and i686-linux. Jakub 2016-02-10 Jakub Jelinek Backported from mainline 2015-12-03 Jakub Jelinek PR preprocessor/57580 * c

C++ PATCH for c++/68926 (SFINAE failure with template-id)

A standard SFINAE problem of not passing 'complain' down far enough. Tested x86_64-pc-linux-gnu, applying to trunk, 5, 4.9. commit ca7de783e98c7b1163be0eefacca762e8aa3e685 Author: Jason Merrill Date: Wed Feb 10 12:45:55 2016 -0500 PR c++/68926 * pt.c (resolve_nondeduced_context

[PATCH] Fix PR c++/69098 (bogus errors with static data member template)

tsubst_qualified_id() is currently not prepared to handle a SCOPED_REF whose RHS is a variable template. r226642 made this deficiency more obvious by marking all variable templates as dependent (thus forcing them to be wrapped in a SCOPED_REF) but before that it was also possible to trigger a bogu

Re: [PATCH] Fix PR c++/69098 (bogus errors with static data member template)

On Wed, Feb 10, 2016 at 2:16 PM, Patrick Palka wrote: > tsubst_qualified_id() is currently not prepared to handle a SCOPED_REF > whose RHS is a variable template. r226642 made this deficiency more > obvious by marking all variable templates as dependent (thus forcing > them to be wrapped in a SCO

Warning location fix, PR c++/69733

This PR notes that in this warning: const.ii:5:25: warning: type qualifiers ignored on function return type [-Wignored-qualifiers] const double value() const {return val;} ^ we are pointing at the wrong qualifier. Below I'm attaching a patch that makes it point

Re: Warning location fix, PR c++/69733

On Wed, Feb 10, 2016 at 08:26:42PM +0100, Bernd Schmidt wrote: > This PR notes that in this warning: > const.ii:5:25: warning: type qualifiers ignored on function return type > [-Wignored-qualifiers] > const double value() const {return val;} > ^ > > we are pointin

Fix incomplete initialization of declspecs

I've noticed that build_null_declspecs fails to clear out all location information since it's missing a multiplication with the type size. The simplest fix seems to be to just clear the entire structure. Bootstrapping & testing now on x86_64-linux, ok if that succeeds? Depending on what we do

Re: Un-parallelized OpenACC kernels constructs with nvptx offloading: "avoid offloading"

On 02/10/2016 06:37 PM, Thomas Schwinge wrote: On Wed, 10 Feb 2016 17:37:30 +0100, Bernd Schmidt wrote: IIUC it's also disabling offloading for parallels rather than just kernels, which we previously said shouldn't happen. Ah, you're talking about mixed OpenACC parallel/kernels codes -- I und

[PATCH] PR driver/69265 and 69453: improved suggestions for various misspelled options

On Wed, 2016-02-10 at 17:25 +0100, Bernd Schmidt wrote: > On 02/09/2016 09:44 PM, David Malcolm wrote: > > This is a bug in a new feature, so it isn't a regression as such, > > but > > it's fairly visible, and I believe the fix is relatively low-risk > > (error-handling of typos of command-line opt

Re: Fix incomplete initialization of declspecs

On Wed, 10 Feb 2016, Bernd Schmidt wrote: > I've noticed that build_null_declspecs fails to clear out all location > information since it's missing a multiplication with the type size. The > simplest fix seems to be to just clear the entire structure. > > Bootstrapping & testing now on x86_64-lin

Re: [PATCH] Fix PR c++/69098 (bogus errors with static data member template)

On 02/10/2016 02:16 PM, Patrick Palka wrote: ...in tsubst_qualified_id() I just copied the way variable templates are handled in tsubst_copy_and_build [TEMPLATE_ID_EXPR]. Let's factor that code out into another function rather than copy it. Jason

Re: [PR69315] enable finish_function to recurse for constexpr functions

On Jan 26, 2016, Alexandre Oliva wrote: > We don't want finish_function to be called recursively from mark_used. > However, it's desirable and necessary to call itself recursively when > performing delayed folding, because that may have to instantiate and > evaluate constexpr template functions.

Re: [PATCH] Fix PR c++/69098 (bogus errors with static data member template)

On Wed, 10 Feb 2016, Jason Merrill wrote: On 02/10/2016 02:16 PM, Patrick Palka wrote: ...in tsubst_qualified_id() I just copied the way variable templates are handled in tsubst_copy_and_build [TEMPLATE_ID_EXPR]. Let's factor that code out into another function rather than copy it. Done. D

Re: [PATCH] Fix PR c++/69098 (bogus errors with static data member template)

OK. Jason

[PATCH] Fix unnecessary -Wmaybe-uninitialized false positive (PR target/65313)

Hi! During profiledbootstrap on ppc64 I've noticed a -Wmaybe-uninitialized warning in vect_schedule_slp_instance, when built with -fprofile-generate. While it is clearly a false positive, IMHO it is completely unnecessary to use here two variables, one uninitialized, another bool whether it is ini

[PATCH] Fix another ipa-split caused ICE (PR ipa/69241)

Hi! Markus has pointed out to a reduced testcase which still ICEs even with the PR69241 fix. In that case the function with TREE_ADDRESSABLE return type does not return at all (and -Wreturn-type properly diagnoses it). For that case the following patch just forces the lhs on the *.part.* call, so

[PATCH], PR 68404 patch #2 (disable power8/power9 fusion on PowerPC)

This patch disables -mcpu=power8/-mtune=power8 from setting -mpower8-fusion and -mcpu=power9/-mtune=power9 from setting -mpower9-fusion. I will look at the earlyclobber that Bernd Schmidt mentioned, but for now it may be safest to just disable it for GCC 6.0. I built it on a little endian power8

[C++ Patch] PR 68726 ("ice: tree check: expected tree_vec, have error_mark in comp_template_args_with_info, at cp/pt.c:7890")

Hi, turns out, this small ICE on invalid is actually a regression, I can only reproduce it in trunk. Anyway, the ICE occurs immediately when, during error recovery, comp_template_args gets an error_mark_node as second argument. Tested x86_64-linux. Thanks, Paolo. P.S. Second try: the first

Re: [PATCH], PR 68404 patch #2 (disable power8/power9 fusion on PowerPC)

On Wed, Feb 10, 2016 at 05:42:17PM -0500, Michael Meissner wrote: > This patch disables -mcpu=power8/-mtune=power8 from setting -mpower8-fusion > and > -mcpu=power9/-mtune=power9 from setting -mpower9-fusion. I will look at the > earlyclobber that Bernd Schmidt mentioned, but for now it may be sa

Re: AW: Wonly-top-basic-asm

Since no one expressed any objections, I have renamed the option from -Wonly-top-basic-asm to -Wbasic-asm-in-function. This more clearly conveys what the option does (give a warning if you find basic asm in a function). I believe the attached patch addresses all the other outstanding comments

Re: [C++ Patch] PR 68726

OK. Jason

[PATCH] PR plugins/69758: add params.list to PLUGIN_HEADERS

params.h is listed in PLUGIN_HEADERS. As of r227566 params.h #includes params.list, but the latter is not in PLUGIN_HEADERS, leading to compilation failure for plugins that include params.h e.g. for gcc-python-plugin: In file included from gcc-cfg.c:40:0: /install-dogfood/lib/gcc/x86_64-pc-l

Re: [PATCH] PR plugins/69758: add params.list to PLUGIN_HEADERS

On 02/11/2016 04:29 AM, David Malcolm wrote: gcc/ChangeLog: PR plugins/69758 * Makefile.in (PLUGIN_HEADERS): Add params.list. Ok. Bernd

Re: C++ PATCH for c++/10200 (DR 141, template name lookup after ->)

On 2016.02.10 at 10:30 -0500, Jason Merrill wrote: > After . or ->, when we see a name followed by < we look for a template name, > first in the scope of the object expression and then in the enclosing scope. > DR 141 clarified that when we look in the enclosing scope, we only consider > class temp

Re: TR29124 C++ Special Maths - Make pull functions into global namespace.

I’m seeing: /home/mrs/work1/gcc/libstdc++-v3/testsuite/special_functions/18_riemann_zeta/check_value.cc: In function 'void test(const testcase_riemann_zeta (&)[Num], Tp)': /home/mrs/work1/gcc/libstdc++-v3/testsuite/special_functions/18_riemann_zeta/check_value.cc:285:15: error: 'riemann_zeta' is

AW: AW: Wonly-top-basic-asm

On 11.2.2016, David Wohlferd wrote: > > Since no one expressed any objections, I have renamed the option from > -Wonly-top-basic-asm to -Wbasic-asm-in-function. This more clearly > conveys what the option does (give a warning if you find basic asm in a > function). > why not simply -Wbasic-asm ?

Re: [PATCH PR69052]Check if loop inv can be propagated into mem ref with additional addr expr canonicalization

On 02/09/2016 04:08 AM, Bin Cheng wrote: Hi, When counting cost for loop inv, GCC checks if a loop inv can be propagated into its use site (a memory reference). If it cannot be propagated, we increase its cost so that it's expensive enough to be hoisted out of loop. Currently we simply repla