[Bug tree-optimization/71831] __builtin_object_size poor results with no optimization

2016-09-01 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71831 --- Comment #9 from Martin Sebor --- Author: msebor Date: Fri Sep 2 02:14:50 2016 New Revision: 239953 URL: https://gcc.gnu.org/viewcvs?rev=239953=gcc=rev Log: PR tree-optimization/71831 - __builtin_object_size poor results with no

[Bug tree-optimization/71831] __builtin_object_size poor results with no optimization

2016-09-01 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71831 Martin Sebor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/77448] Incorrect code generation for calling stack object's method

2016-09-01 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77448 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[Bug libstdc++/75402] std::condition_variable::wait_until wakes up constantly

2016-09-01 Thread nfitzgerald at mozilla dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=75402 Nick Fitzgerald [:fitzgen] [⏰PDT; UTC-7] changed: What|Removed |Added Status|WAITING |RESOLVED

[Bug c++/77448] New: Incorrect code generation for calling stack object's method

2016-09-01 Thread nfitzgerald at mozilla dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77448 Bug ID: 77448 Summary: Incorrect code generation for calling stack object's method Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal

gcc-6-20160901 is now available

2016-09-01 Thread gccadmin
Snapshot gcc-6-20160901 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/6-20160901/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 6 SVN branch with the following options: svn://gcc.gnu.org/svn/gcc/branches/gcc-6

[Bug libfortran/77393] [7 Regression] Revision r237735 changed the behavior of F0.0

2016-09-01 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77393 --- Comment #7 from Dominique d'Humieres --- > Works for me here! Are you sure that you have tested print "(f8.0)", huge(1.0) print "(f18.0)", huge(1.0_8) print "(f20.0)", huge(1.0_10) print "(f40.0)", huge(1.0_16) end ?

[Bug c++/77442] error: ‘free’ is not a member of ‘std’ std::free(ptr);

2016-09-01 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77442 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[PATCH 2/4] rs6000: Rename 74 -> CR6_REGNO

2016-09-01 Thread Segher Boessenkool
2016-09-01 Segher Boessenkool * config/rs6000/altivec.md: Use CR6_REGNO instead of 74 throughout. * config/rs6000/vector.md: Ditto. * config/rs6000/vsx.md: Ditto. --- gcc/config/rs6000/altivec.md | 30 +++---

[PATCH 4/4] rs6000: Rename 110 -> VSCR_REGNO

2016-09-01 Thread Segher Boessenkool
2016-09-01 Segher Boessenkool * config/rs6000/altivec.md: Use VSCR_REGNO instead of 110 throughout. --- gcc/config/rs6000/altivec.md | 37 +++-- 1 file changed, 19 insertions(+), 18 deletions(-) diff --git

[PATCH 3/4] rs6000: Rename 109 -> VRSAVE_REGNO

2016-09-01 Thread Segher Boessenkool
2016-09-01 Segher Boessenkool * config/rs6000/altivec.md: Use VRSAVE_REGNO instead of 109 throughout. --- gcc/config/rs6000/altivec.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gcc/config/rs6000/altivec.md

[PATCH 1/4] Hack: non-symbolic numeric constant warning

2016-09-01 Thread Segher Boessenkool
This patch prints a warning if a register number in the machine description patterns is non-symbolic, to catch places that should use a constant from a define_constants instead (e.g., LR_REGNO). I hardcoded this to not warn for regno < 32, i.e. the GPRs on PowerPC. That of course is not

[Bug libfortran/77393] [7 Regression] Revision r237735 changed the behavior of F0.0

2016-09-01 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77393 --- Comment #6 from Jerry DeLisle --- The update to the test case should correct any platform specific KIND issues. May need to upsize the bufers a little for Dominique's issue. I set the buffers tight so we could catch different behaviors on

[Bug rtl-optimization/77416] [7 Regression] LRA rematerializing use of CA reg across function call

2016-09-01 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77416 --- Comment #6 from Peter Bergner --- Looking closer, it does seem remat is trying to handle these hard coded registers via its hard_regs field. One thing that strikes me as I look at that code, it checks for reg->type being == OP_IN or !=

[Bug libfortran/77393] [7 Regression] Revision r237735 changed the behavior of F0.0

2016-09-01 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77393 --- Comment #5 from Jerry DeLisle --- Author: jvdelisle Date: Thu Sep 1 21:17:42 2016 New Revision: 239945 URL: https://gcc.gnu.org/viewcvs?rev=239945=gcc=rev Log: 2016-09-01 Jerry DeLisle PR

[Bug c++/77434] warn about suspicious precedence of ternary operator (?:)

2016-09-01 Thread bernd.edlinger at hotmail dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77434 --- Comment #7 from Bernd Edlinger --- (In reply to jos...@codesourcery.com from comment #6) > On Thu, 1 Sep 2016, bernd.edlinger at hotmail dot de wrote: > > > + warning_at (location, 0, > > + "?: expression using

[Bug c++/77434] warn about suspicious precedence of ternary operator (?:)

2016-09-01 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77434 --- Comment #6 from joseph at codesourcery dot com --- On Thu, 1 Sep 2016, bernd.edlinger at hotmail dot de wrote: > + warning_at (location, 0, > + "?: expression using integer constants in boolean > context"); This

[Bug c++/77427] [6/7 Regression] ice when canonical types differ for identical types

2016-09-01 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77427 --- Comment #6 from Tom de Vries --- (In reply to Tom de Vries from comment #5) > Hmm, PR52830 looks similar. This more generic patch fixes PR52830 as well: ... diff --git a/gcc/tree.c b/gcc/tree.c index 33e6f97..f5c1f6a 100644 --- a/gcc/tree.c

[Bug c++/77434] warn about suspicious precedence of ternary operator (?:)

2016-09-01 Thread bernd.edlinger at hotmail dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77434 --- Comment #5 from Bernd Edlinger --- This is an idea for a warning that does not focus on parentheses: Here we had: a ? c1 : c2; but in a context where a boolean is requested. It is always suspicious, when c1, and c2 are integer constants

[Bug libfortran/77393] [7 Regression] Revision r237735 changed the behavior of F0.0

2016-09-01 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77393 --- Comment #4 from Jerry DeLisle --- (In reply to Dominique d'Humieres from comment #3) > At revision r239908 the second test in comment 0 still gives > > > ** > > Program received signal SIGSEGV: Segmentation fault -

Re: [patch, libgfortran] PR77393 [7 Regression] Revision r237735 changed the behavior of F0.0

2016-09-01 Thread Jerry DeLisle
.0)") -huge(1.0_10) >> if (len(trim(str)).lt.4935) error stop "FAILED AT 9" >> write(str, "(f0.10)") -huge(1.0_16) >> if (len(trim(str)).lt.4945) error stop "FAILED AT 11" >> end program testbigf0 > > FAIL: gfortran.dg/fmt_f0_2.

[PATCH, fortran testsuite]: Correct the calls to c functions in gfortran.dg/c_by_val_1.f

2016-09-01 Thread Uros Bizjak
Hello! Functions c_to_c__ and c_to_c8__ in the supplemental testcase c file are prototyped as: extern void c_to_c__ (complex float*, complex float, complex float*, complex float**); extern void c_to_c8__ (complex double*, complex double, complex double*, complex double**); so, we have to call

Re: [PATCH, docs] invoke.texi: random copy-editing

2016-09-01 Thread Gerald Pfeifer
On Wed, 29 Aug 2012, Sandra Loosemore wrote: > * doc/invoke.texi: Fix numerous typos and punctuation/grammatical > errors throughout the file. Re-word some awkward sentences and > paragraphs. I noticed you changed return-value and return-type to their variants without a dash.

[Bug rtl-optimization/77416] [7 Regression] LRA rematerializing use of CA reg across function call

2016-09-01 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77416 --- Comment #5 from Peter Bergner --- So this is a latent bug in LRA's rematerialization code. For this test case, we have insn: Basic BlocK 6: ... (insn 179 178 92 6 (parallel [ (set (reg/v:SI 175 [ pD.2425 ])

[Bug libgcc/77447] /usr/arm-linux-gnueabihf is not in xgcc include path

2016-09-01 Thread malithyapa at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77447 malithyapa at gmail dot com changed: What|Removed |Added Status|WAITING |RESOLVED

[Bug libgcc/77447] /usr/arm-linux-gnueabihf is not in xgcc include path

2016-09-01 Thread malithyapa at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77447 --- Comment #3 from malithyapa at gmail dot com --- Yes! sorry, didn't think it'd matter, i even saw that in the original configure line. Thanks!

[Bug fortran/72832] [6/7 Regression] [OOP] ALLOCATE with SOURCE fails to allocate requested dimensions

2016-09-01 Thread vehre at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72832 vehre at gcc dot gnu.org changed: What|Removed |Added Priority|P3 |P4

[Bug libgcc/77447] /usr/arm-linux-gnueabihf is not in xgcc include path

2016-09-01 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77447 --- Comment #2 from Andrew Pinski --- I suspect you did not use --with-float=hard

[Bug libgcc/77447] /usr/arm-linux-gnueabihf is not in xgcc include path

2016-09-01 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77447 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[Bug libgcc/77447] New: /usr/arm-linux-gnueabihf is not in xgcc include path

2016-09-01 Thread malithyapa at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77447 Bug ID: 77447 Summary: /usr/arm-linux-gnueabihf is not in xgcc include path Product: gcc Version: 4.9.4 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug fortran/72832] [6/7 Regression] [OOP] ALLOCATE with SOURCE fails to allocate requested dimensions

2016-09-01 Thread vehre at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72832 vehre at gcc dot gnu.org changed: What|Removed |Added Priority|P4 |P3

Re: stray warning from gcc's cpp

2016-09-01 Thread Gerald Pfeifer
Hi David, I found this older report (including patches for two approaches even!) that it seems did not see any response despite the nice analysis (and the patch). Mind having a look? Gerald PS: If you prefer, I can put this into Bugzilla, too. On Fri, 28 Mar 2014, Andriy Gapon wrote: > on

[Bug c++/77427] [6/7 Regression] ice when canonical types differ for identical types

2016-09-01 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77427 --- Comment #5 from Tom de Vries --- Hmm, PR52830 looks similar.

Re: [PATCH][msp430] Don't output __interrupt_vector sections for weak functions

2016-09-01 Thread Joe Seymour
On 29/08/2016 22:09, DJ Delorie wrote: > > Which results in a more user-obvious case, ignoring the interrupt > attribute or ignoring the weak attribute? I would think that we never > want to compile and link successfully if we can't do what the user > wants, and omitting an interrupt handler

Re: PR35503 - warn for restrict pointer

2016-09-01 Thread Martin Sebor
The attached version passes bootstrap+test on ppc64le-linux-gnu. Given that it only looks if parameters are restrict qualified and not how they're used inside the callee, this can have false positives as in above test-cases. Should the warning be put in Wextra rather than Wall (I have left it in

Re: Implement -Wimplicit-fallthrough (version 8)

2016-09-01 Thread Joseph Myers
On Thu, 1 Sep 2016, Marek Polacek wrote: > Yet another version. Changes from version 7: > > * no more fix-it hints untils we resolve some of the issues this patch > has discovered; instead, I simply used an inform call, > * I've fixed a bogus -Wdeclaration-after-statement warning in the C FE,

Re: [PATCH, testsuite]: Use dg-add-options ieee in some _FloatN tests

2016-09-01 Thread Joseph Myers
On Thu, 1 Sep 2016, Uros Bizjak wrote: > Hello! > > Attached patch adds dg-add-options ieee in _FloatN tests where > advanced IEEE features are used (e.g. NaN, infinity, subnormals). > > 2016-09-01 Uros Bizjak > > * gcc.dg/torture/float128-builtin.c,

Re: Implement C _FloatN, _FloatNx types [version 6]

2016-09-01 Thread Joseph Myers
On Thu, 1 Sep 2016, Szabolcs Nagy wrote: > it seems that the FLT_EVAL_METHOD change in TS 18661 was a > bad decision: existing code have to be modified to conform > to the new semantics. this situation could have been avoided > by introducing new macros or only changing behaviour when > the user

Re: [patch, libstdc++] std::shuffle: Generate two swap positions at a time if possible

2016-09-01 Thread Jonathan Wakely
On 01/09/16 17:31 +0200, Eelis van der Weegen wrote: On 2016-09-01 17:14, Jonathan Wakely wrote: On 31/08/16 13:45 +0100, Jonathan Wakely wrote: On 03/05/16 16:42 +0200, Eelis van der Weegen wrote: Ah, thanks, I forgot to re-attach when I sent to include the libstdc++ list. On 2016-05-03

Re: [patch, libstdc++] std::shuffle: Generate two swap positions at a time if possible

2016-09-01 Thread Jonathan Wakely
On 01/09/16 17:27 +0200, Marc Glisse wrote: On Thu, 1 Sep 2016, Jonathan Wakely wrote: + const __uc_type __comp_range = __swap_range * (__swap_range + 1); If __swap_range is 3, then __comp_range is 10 and ??? Bah :-) Thanks. I guess I read the code correctly the other day at

Re: [patch, libstdc++] std::shuffle: Generate two swap positions at a time if possible

2016-09-01 Thread Eelis van der Weegen
On 2016-09-01 17:14, Jonathan Wakely wrote: On 31/08/16 13:45 +0100, Jonathan Wakely wrote: On 03/05/16 16:42 +0200, Eelis van der Weegen wrote: Ah, thanks, I forgot to re-attach when I sent to include the libstdc++ list. On 2016-05-03 14:38, Jonathan Wakely wrote: ENOPATCH On 1 May 2016 at

Re: [patch, libstdc++] std::shuffle: Generate two swap positions at a time if possible

2016-09-01 Thread Marc Glisse
On Thu, 1 Sep 2016, Jonathan Wakely wrote: + const __uc_type __comp_range = __swap_range * (__swap_range + 1); If __swap_range is 3, then __comp_range is 10 and ??? -- Marc Glisse

Re: [PATCH 2/3] rs6000: Use LR_REGNO instead of constant 65

2016-09-01 Thread Segher Boessenkool
On Thu, Sep 01, 2016 at 09:50:09AM +0100, Iain Sandoe wrote: > > I left it in *restore_world because Iain will remove it there soon. > > Here is the patch to fix up Darwin, > I guess it’s both Darwin-local and reasonably obvious now, but OK? Yes, okay for trunk, thanks! Segher > 2016-09-01

Re: [patch, libstdc++] std::shuffle: Generate two swap positions at a time if possible

2016-09-01 Thread Jonathan Wakely
On 31/08/16 13:45 +0100, Jonathan Wakely wrote: On 03/05/16 16:42 +0200, Eelis van der Weegen wrote: Ah, thanks, I forgot to re-attach when I sent to include the libstdc++ list. On 2016-05-03 14:38, Jonathan Wakely wrote: ENOPATCH On 1 May 2016 at 15:21, Eelis wrote:

[Bug target/72827] [7 Regression] gnat bootstrap broken on powerpc64le-linux-gnu

2016-09-01 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72827 Eric Botcazou changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: Proposal: readable and writable attributes on variables

2016-09-01 Thread Martin Sebor
On 08/31/2016 04:56 AM, Jens Bauer wrote: Hi Martin (and everyone else on the gcc list). I appreciate your input and kind reply to my proposal. :) On Tue, 30 Aug 2016 20:44:35 -0600, Martin Sebor wrote: This sounds reasonable and useful to me but to be fully integrated into the language,

[Bug c/7652] -Wswitch-break : Warn if a switch case falls through

2016-09-01 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=7652 --- Comment #60 from Marek Polacek --- Author: mpolacek Date: Thu Sep 1 14:59:50 2016 New Revision: 239939 URL: https://gcc.gnu.org/viewcvs?rev=239939=gcc=rev Log: PR c/7652 gcc/c-family/ * c-common.c

Re: Implement -Wimplicit-fallthrough (version 8): add gcc_fallthrough()

2016-09-01 Thread Jakub Jelinek
On Thu, Sep 01, 2016 at 04:51:37PM +0200, Marek Polacek wrote: > I thought I'd keep it since it conveys something more than a pure "falls > through" comments, but I changed it anyway. Here's the pure comment > changes patch which I hope can be committed right away. > > 2016-09-01 Marek Polacek

Re: Implement -Wimplicit-fallthrough (version 8): add gcc_fallthrough()

2016-09-01 Thread Marek Polacek
On Thu, Sep 01, 2016 at 04:27:01PM +0200, Jakub Jelinek wrote: > On Thu, Sep 01, 2016 at 03:42:12PM +0200, Marek Polacek wrote: > > --- gcc/gcc/c-family/c-common.c > > +++ gcc/gcc/c-family/c-common.c > > @@ -11590,6 +11590,7 @@ resolve_overloaded_builtin (location_t loc, tree > > function, > >

[Bug target/72827] [7 Regression] gnat bootstrap broken on powerpc64le-linux-gnu

2016-09-01 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72827 --- Comment #26 from Bill Schmidt --- The original patch direction was impossible to make work, as with a base register of r31 (frame pointer) and an offset register of r0 (not allowed as a base register), there is no legitimate way to update

[Bug target/72827] [7 Regression] gnat bootstrap broken on powerpc64le-linux-gnu

2016-09-01 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72827 --- Comment #25 from Bill Schmidt --- Author: wschmidt Date: Thu Sep 1 14:43:55 2016 New Revision: 239938 URL: https://gcc.gnu.org/viewcvs?rev=239938=gcc=rev Log: 2016-09-01 Bill Schmidt Michael

Re: [PATCH v2, rs6000] Fix PR72827 (ada bootstrap failure)

2016-09-01 Thread Bill Schmidt
On Sep 1, 2016, at 9:32 AM, Segher Boessenkool wrote: > > On Wed, Aug 31, 2016 at 09:15:32PM -0500, Bill Schmidt wrote: >> This patch (suggested by Michael Meissner) instead prevents the problem >> by disallowing reg+reg addressing for TImode, allowing D-form

Re: Proposal: readable and writable attributes on variables

2016-09-01 Thread Jens Bauer
Hi Florian. On Thu, 1 Sep 2016 15:21:27 +0200, Florian Weimer wrote: >> So I'd like to propose the following two attributes, which is 'off' >> by default (eg. read access + write access): >> __attribute__((not_readable)) >> __attribute__((not_writable)) > > You can already do this: > > “ > int

[Bug c++/77446] New: Suspicious "non-constant condition for static assertion" error

2016-09-01 Thread TonyELewis at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77446 Bug ID: 77446 Summary: Suspicious "non-constant condition for static assertion" error Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal

[Bug target/71607] [5/6/7 Regression] [ARM] ice due to forbidden enabled attribute dependency on instruction operands

2016-09-01 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71607 Thomas Preud'homme changed: What|Removed |Added Status|ASSIGNED|NEW --- Comment #6 from Thomas

[Bug target/71607] [5/6/7 Regression] [ARM] ice due to forbidden enabled attribute dependency on instruction operands

2016-09-01 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71607 --- Comment #5 from Thomas Preud'homme --- (In reply to Christophe Monat from comment #4) > (In reply to Thomas Preud'homme from comment #3) > > Thomas, Hi Christophe, > > I am seeing that the assignee name has been reset : does it mean that

Re: [PATCH v2, rs6000] Fix PR72827 (ada bootstrap failure)

2016-09-01 Thread Segher Boessenkool
On Wed, Aug 31, 2016 at 09:15:32PM -0500, Bill Schmidt wrote: > This patch (suggested by Michael Meissner) instead prevents the problem > by disallowing reg+reg addressing for TImode, allowing D-form addressing > to be used for the separate stores of the GPRs. This is not an ideal > permanent

Re: Implement -Wimplicit-fallthrough (version 8): add gcc_fallthrough()

2016-09-01 Thread Jakub Jelinek
On Thu, Sep 01, 2016 at 03:42:12PM +0200, Marek Polacek wrote: > --- gcc/gcc/c-family/c-common.c > +++ gcc/gcc/c-family/c-common.c > @@ -11590,6 +11590,7 @@ resolve_overloaded_builtin (location_t loc, tree > function, > gcc_unreachable (); > } > /* Fallthrough to the normal

Ping Re: Make max_align_t respect _Float128

2016-09-01 Thread Joseph Myers
Ping. This patch is pending review (with either __i386__ or __SIZEOF_FLOAT128__, although __i386__ is safest in that it minimizes the cases where there is any ABI change at all even in the size of the type). -- Joseph S. Myers

[Bug target/71607] [5/6/7 Regression] [ARM] ice due to forbidden enabled attribute dependency on instruction operands

2016-09-01 Thread christophe.monat at st dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71607 --- Comment #4 from Christophe Monat --- (In reply to Thomas Preud'homme from comment #3) Thomas, I am seeing that the assignee name has been reset : does it mean that you definitely disengage from looking at this problem ? If this is the

[Bug c++/77443] Empty initializer on huge object array member slow down the compilation dramatically with -O1

2016-09-01 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77443 Richard Biener changed: What|Removed |Added Keywords||compile-time-hog CC|

[Bug tree-optimization/77445] [7 Regression] Performance drop after r239219 on coremark test

2016-09-01 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77445 Richard Biener changed: What|Removed |Added Keywords||missed-optimization

[Bug c/65467] [libgomp] sorry, unimplemented: '_Atomic' with OpenMP

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65467 --- Comment #12 from Jakub Jelinek --- Actually, firstprivate on _Atomic vars in target construct could be implemented just by forcing it into a temporary with non-_Atomic qualified type on the host side (i.e. atomically loading it),

[Bug middle-end/77436] [5/6 Regression] Incorrect constant result for summing loop inserted

2016-09-01 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77436 Richard Biener changed: What|Removed |Added Known to work||7.0 Summary|[5/6/7

[Bug c++/77434] warn about suspicious precedence of ternary operator (?:)

2016-09-01 Thread bernd.edlinger at hotmail dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77434 Bernd Edlinger changed: What|Removed |Added CC||bernd.edlinger at hotmail dot de ---

Implement -Wimplicit-fallthrough (version 8): add gcc_fallthrough()

2016-09-01 Thread Marek Polacek
Tobias pointed out that I need to add a new gcc_fallthrough into trans-io.c, so done. Bootstrapped/regtested on x86_64-linux and ppc64-linux. 2016-08-31 Marek Polacek PR c/7652 * Makefile.in (insn-attrtab.o-warn, insn-dfatab.o-warn,

Implement -Wimplicit-fallthrough (version 8)

2016-09-01 Thread Marek Polacek
Yet another version. Changes from version 7: * no more fix-it hints untils we resolve some of the issues this patch has discovered; instead, I simply used an inform call, * I've fixed a bogus -Wdeclaration-after-statement warning in the C FE, new test added, * I've addressed Jason's comments

Re: [PATCH][AArch64 - v2] Simplify eh_return implementation

2016-09-01 Thread Ramana Radhakrishnan
On 10/08/16 17:26, Wilco Dijkstra wrote: > I noticed it would still be a good idea to add an extra barrier in the epilog > as the > scheduler doesn't appear to handle aliases of frame accesses properly. > > This patch simplifies the handling of the EH return value. We force the use > of the

[Bug middle-end/77436] [5/6/7 Regression] Incorrect constant result for summing loop inserted

2016-09-01 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77436 --- Comment #6 from Richard Biener --- Author: rguenth Date: Thu Sep 1 13:38:25 2016 New Revision: 239937 URL: https://gcc.gnu.org/viewcvs?rev=239937=gcc=rev Log: 2016-09-01 Richard Biener PR middle-end/77436

Re: [patch] Fix segfault in param_change_prob

2016-09-01 Thread Richard Biener
On Thu, Sep 1, 2016 at 3:17 PM, Eric Botcazou wrote: >> So I think it's better to apply get_base_address here rather than only >> stripping VIEW_CONVERT_EXRPs. >> (beware of it returning NULL_TREE for WITH_SIZE_EXPRs, thus maybe >> strip those first as well) > > Something

[Bug rtl-optimization/77416] [7 Regression] LRA rematerializing use of CA reg across function call

2016-09-01 Thread pthaugen at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77416 --- Comment #4 from Pat Haugen --- Didn't really think of this before, but the call isn't necessarily the problem, addze sets the CA bit too. So it's invalid to remataterialize the addze after an addze.

[PATCH] Fix PR77436

2016-09-01 Thread Richard Biener
The following fixes the wide-int conversion of tree_fold_binomial. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2016-09-01 Richard Biener PR middle-end/77436 * tree-chrec.c (tree_fold_binomial): Use widest_int, properly

Re: Proposal: readable and writable attributes on variables

2016-09-01 Thread Florian Weimer
On 08/30/2016 02:22 PM, Jens Bauer wrote: Hi all. I know it's possible to declare a variable 'read-only' by using 'const'. When working with microcontrollers (small ICs, which often requires you to write your code at driver-level), you need to be able to declare a structure member

Re: [patch] Fix segfault in param_change_prob

2016-09-01 Thread Eric Botcazou
> So I think it's better to apply get_base_address here rather than only > stripping VIEW_CONVERT_EXRPs. > (beware of it returning NULL_TREE for WITH_SIZE_EXPRs, thus maybe > strip those first as well) Something like this? The testsuite is still clean with it. * ipa-inline-analysis.c

[Bug c++/25814] Request for warning for parser ambiguity of function declarations and variable declarations with initializations

2016-09-01 Thread egall at gwmail dot gwu.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=25814 Eric Gallager changed: What|Removed |Added CC||egall at gwmail dot gwu.edu --- Comment

[Bug c++/77427] [6/7 Regression] ice when canonical types differ for identical types

2016-09-01 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77427 --- Comment #4 from Tom de Vries --- This fixes the ICE: ... diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index 4531647..f13790f 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -10551,6 +10551,8 @@

[Bug web/69601] current/ redirect is off by at least a day

2016-09-01 Thread egall at gwmail dot gwu.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69601 --- Comment #6 from Eric Gallager --- Seems to be fixed for September.

[arm-embedded] [PATCH, ARM 7/7] Enable atomics for ARMv8-M Mainline

2016-09-01 Thread Thomas Preudhomme
Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Best regards, Thomas Forwarded Message Subject: Re: [PATCH, ARM 7/7] Enable atomics for ARMv8-M Mainline Date: Thu, 14 Jul 2016 17:34:44 +0100 From: Thomas Preudhomme

[Bug tree-optimization/77445] [7 Regression] Performance drop after r239219 on coremark test

2016-09-01 Thread ysrumyan at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77445 --- Comment #1 from Yuri Rumyantsev --- Created attachment 39535 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39535=edit test-case to reproduce It is sufficient to compile it with -Ofast option.

[Bug tree-optimization/77445] New: [7 Regression] Performance drop after r239219 on coremark test

2016-09-01 Thread ysrumyan at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77445 Bug ID: 77445 Summary: [7 Regression] Performance drop after r239219 on coremark test Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH][AArch64 - v2] Simplify eh_return implementation

2016-09-01 Thread Wilco Dijkstra
Ping I noticed it would still be a good idea to add an extra barrier in the epilog as the scheduler doesn't appear to handle aliases of frame accesses properly. This patch simplifies the handling of the EH return value.  We force the use of the frame pointer so the return location is always

[Bug c++/77427] [6/7 Regression] ice when canonical types differ for identical types

2016-09-01 Thread vries at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77427 Tom de Vries changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |vries at gcc dot gnu.org ---

Re: [PATCH, PR70955] Tag {ms,sysv}_va_list_type_node with {ms,sysv}_abi attribute

2016-09-01 Thread Markus Trippelsdorf
On 2016.08.26 at 17:33 +0200, Tom de Vries wrote: > On 26/08/16 10:40, Richard Biener wrote: > >I suppose > >you could get away with using sth entirely private to the backend, > >like "sysv abi valist", also not ever user-creatable. Does this > >side-step the FE issue? > > It does. > > Added

Re: [PATCH, docs] invoke.texi: random copy-editing

2016-09-01 Thread Gerald Pfeifer
Hi Sandra, On Wed, 29 Aug 2012, Sandra Loosemore wrote: > I've had this largish pile of random copy-edits to invoke.texi left over > from my previous passes through that file earlier this year. that was an amazing amount of changes; I admire your patience and thoroughness! > I'll wait a few

[PTX] cbranch expanders

2016-09-01 Thread Nathan Sidwell
I noticed the conditional branch expanders unnecessarily forced alll ops to be registers. Of course later optimization fixes that up, but why create the problem in the first place? committed to trunk. nathan

[committed] Backports to 6.x branch

2016-09-01 Thread Jakub Jelinek
Hi! I've committed following backports of some trunk fixes, after bootstrapping/regtesting them on x86_64-linux and i686-linux. Jakub 2016-09-01 Jakub Jelinek Backported from mainline 2016-08-16 Jakub Jelinek PR

Re: [patch] Fix segfault in param_change_prob

2016-09-01 Thread Richard Biener
On Wed, Aug 31, 2016 at 8:46 PM, Eric Botcazou wrote: > Hi, > > the attached Ada testcase triggers a segfault in param_change_prob because the > parameter is VIEW_CONVERT_EXPR (or it fools the logic and walk_aliased_vdefs is called on a NULL

[Bug tree-optimization/77352] ICE: verify_ssa failed

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77352 --- Comment #4 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:48:54 2016 New Revision: 239933 URL: https://gcc.gnu.org/viewcvs?rev=239933=gcc=rev Log: Backported from mainline 2016-08-31 Jakub Jelinek

[Bug fortran/77374] [6/7 Regression] ICE in resolve_omp_atomic, at fortran/openmp.c:3949

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77374 --- Comment #5 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:48:02 2016 New Revision: 239932 URL: https://gcc.gnu.org/viewcvs?rev=239932=gcc=rev Log: Backported from mainline 2016-08-31 Jakub Jelinek

[Bug tree-optimization/72866] Compile time hog w/ -O3 (-Ofast)

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72866 --- Comment #6 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:46:57 2016 New Revision: 239931 URL: https://gcc.gnu.org/viewcvs?rev=239931=gcc=rev Log: Backported from mainline 2016-08-30 Jakub Jelinek

[Bug other/77421] Bugs found in GCC with the help of PVS-Studio

2016-09-01 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77421 --- Comment #12 from Jonathan Wakely --- (In reply to Richard Biener from comment #11) > (In reply to Jonathan Wakely from comment #5) > > cand_value_at in tree-ssa-loop-ivopts.c does: > > > > tree type = TREE_TYPE (iv->base); > > tree

[Bug c++/77363] [5/6 Regression] Missing debug information in DWARF

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77363 --- Comment #8 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:43:15 2016 New Revision: 239930 URL: https://gcc.gnu.org/viewcvs?rev=239930=gcc=rev Log: Backported from mainline 2016-08-30 Jakub Jelinek

[Bug middle-end/77377] [7 Regression] c-c++-common/pr59037.c ICEs with -fpic -msse on i686

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77377 --- Comment #5 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:42:22 2016 New Revision: 239929 URL: https://gcc.gnu.org/viewcvs?rev=239929=gcc=rev Log: Backported from mainline 2016-08-30 Jakub Jelinek

[Bug tree-optimization/77444] New: Bogus assignments in cand_value_at

2016-09-01 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77444 Bug ID: 77444 Summary: Bogus assignments in cand_value_at Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug fortran/71014] associate statement inside omp parallel do appears to disable default private attribute for inner loop indices

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71014 --- Comment #12 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:41:34 2016 New Revision: 239928 URL: https://gcc.gnu.org/viewcvs?rev=239928=gcc=rev Log: Backported from mainline 2016-08-19 Jakub Jelinek

[Bug fortran/72744] ICE in verify_ssa, at tree-ssa.c:1039

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72744 --- Comment #4 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:38:57 2016 New Revision: 239927 URL: https://gcc.gnu.org/viewcvs?rev=239927=gcc=rev Log: Backported from mainline 2016-08-19 Jakub Jelinek

[Bug fortran/69281] gfortran ICE on temporary array in function call with -fstack-arrays -fopenmp

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69281 --- Comment #4 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:38:16 2016 New Revision: 239926 URL: https://gcc.gnu.org/viewcvs?rev=239926=gcc=rev Log: Backported from mainline 2016-08-19 Jakub Jelinek

[Bug middle-end/77259] [6/7 Regression] ICE in emit_move_insn since r232167

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77259 --- Comment #7 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:36:50 2016 New Revision: 239924 URL: https://gcc.gnu.org/viewcvs?rev=239924=gcc=rev Log: Backported from mainline 2016-08-17 Jakub Jelinek

[Bug target/71910] ICE on valid OpenMP code

2016-09-01 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71910 --- Comment #7 from Jakub Jelinek --- Author: jakub Date: Thu Sep 1 11:29:55 2016 New Revision: 239922 URL: https://gcc.gnu.org/viewcvs?rev=239922=gcc=rev Log: Backported from mainline 2016-08-16 Jakub Jelinek

[Bug rtl-optimization/77425] Pointer test follows dereference in sched-int.h

2016-09-01 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77425 --- Comment #3 from Jonathan Wakely --- (In reply to Manuel López-Ibáñez from comment #1) > There is also this example: > > odr_type > get_odr_type (tree type, bool insert) > { > > odr_types[val->id] = 0; > gcc_assert

  1   2   >