[Bug tree-optimization/69196] [5/6/7 Regression] code size regression with jump threading at -O2

2016-05-02 Thread sebastian.hu...@embedded-brains.de
BILITY or FITNESS FOR A PARTICULAR PURPOSE. sparc-rtems4.12-gcc --version sparc-rtems4.12-gcc (GCC) 6.1.1 20160502 [gcc-6-branch revision cf53985:351726f:7429c1e9312a17e16f7d5be217ca79b82e56d01b] Copyright (C) 2016 Free Software Foundation, Inc. This is free software; see the source for copying cond

Re: Enabling -frename-registers?

2016-05-02 Thread Uros Bizjak
On Mon, May 2, 2016 at 3:20 PM, Bernd Schmidt wrote: > On 05/02/2016 01:57 PM, Uros Bizjak wrote: > >> With the referred testcase, the compare-debug failure trips on (debug_insn >> 101) > > > Ok, INDEX_REG_CLASS is NO_REGS on alpha, and apparently the contents of the > MEM

RE: option -mprfchw on 2 different Opteron cpus

2016-05-02 Thread Kumar, Venkataramanan
Hi > -Original Message- > From: NightStrike [mailto:nightstr...@gmail.com] > Sent: Monday, May 2, 2016 10:31 PM > To: Kumar, Venkataramanan > Cc: Uros Bizjak (ubiz...@gmail.com) ; > lopeziba...@gmail.com; Jan Hubicka ;

[Bug rtl-optimization/70890] [7 regression] r235660 miscompiles stage2 compiler on ia64

2016-05-02 Thread amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70890 Alan Modra changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at

[Bug middle-end/70920] New: if ((intptr_t)ptr == 0) doesn't get simplified to if (ptr == 0)

2016-05-02 Thread ppalka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70920 Bug ID: 70920 Summary: if ((intptr_t)ptr == 0) doesn't get simplified to if (ptr == 0) Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal

[Bug rtl-optimization/70919] New: wrong code at -O1 on x86_64-linux-gnu in 32-bit mode

2016-05-02 Thread su at cs dot ucdavis.edu
gcc version 7.0.0 20160502 (experimental) [trunk revision 235731] (GCC) $ $ gcc-trunk -m32 -O0 small.c; ./a.out $ gcc-trunk -m64 -O1 small.c; ./a.out $ $ gcc-trunk -m32 -O1 small.c $ ./a.out Aborted (core dumped) $ --- #pragma pack(1) struct S0 { int

[Bug c/70918] New: Internal compiler error: Illegal instruction

2016-05-02 Thread ljliang1990 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70918 Bug ID: 70918 Summary: Internal compiler error: Illegal instruction Product: gcc Version: 4.9.2 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c

Two small optimizations to vrp pass

2016-05-02 Thread Patrick Palka
This patch 1. changes the need_assert_for bitmap into an sbitmap since its size is known to be at most num_ssa_names 2. changes a use of FOR_EACH_IMM_USE_STMT to a use of FOR_EACH_IMM_USE_FAST since the uses are only being read, not modified Does this look OK to commit after bootstrap and

[Bug preprocessor/70917] New: out-of-date documentation for character set support and environment variables

2016-05-02 Thread wilson at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70917 Bug ID: 70917 Summary: out-of-date documentation for character set support and environment variables Product: gcc Version: 6.1.0 Status: UNCONFIRMED Severity:

Re: [gcc] Re: [PATCH] Fix spec-options.c test case

2016-05-02 Thread Kaz Kojima
Bernd Edlinger wrote: > No, I actually fixed it, thanks. That test was done without my patch: > > > LAST_UPDATED: Sun May 1 13:46:11 UTC 2016 (revision 235692) > > svn log -r235762 > > r235762

Re: [PATCH] Fix spec-options.c test case

2016-05-02 Thread Kaz Kojima
Oleg Endo wrote: > On Mon, 2016-05-02 at 16:13 +0200, Bernd Schmidt wrote: >> On 05/02/2016 03:43 PM, Bernd Edlinger wrote: >> > Yes, you are right. Only the original use-case seems to be >> > sh-superh-elf specific. But there are also spec strings >> > that are always

[Bug target/65951] [AArch64] Will not vectorize 64bit integer multiplication

2016-05-02 Thread wilson at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951 --- Comment #11 from Jim Wilson --- I've spent some time looking at solutions to this problem. One way to solve it is to simply add a mulv2di3 pattern to the aarch64 port. The presence of a multiply pattern means we will go through

[Bug libfortran/70684] [4.9/5/6/7 Regression] incorrect reading of values from file on Windows

2016-05-02 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70684 --- Comment #17 from Jerry DeLisle --- Author: jvdelisle Date: Tue May 3 00:51:30 2016 New Revision: 235801 URL: https://gcc.gnu.org/viewcvs?rev=235801=gcc=rev Log: 2016-05-02 Jerry DeLisle Backport from

[Bug target/65951] [AArch64] Will not vectorize 64bit integer multiplication

2016-05-02 Thread wilson at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951 --- Comment #10 from Jim Wilson --- Created attachment 38399 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38399=edit testcase unsigned long multiply by constant

[Bug target/65951] [AArch64] Will not vectorize 64bit integer multiplication

2016-05-02 Thread wilson at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951 --- Comment #8 from Jim Wilson --- Created attachment 38397 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38397=edit patch to use synth_mult to generate vector multiplies

[Bug target/65951] [AArch64] Will not vectorize 64bit integer multiplication

2016-05-02 Thread wilson at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951 --- Comment #9 from Jim Wilson --- Created attachment 38398 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38398=edit testcase signed long multiply by constant

[Bug target/65951] [AArch64] Will not vectorize 64bit integer multiplication

2016-05-02 Thread wilson at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65951 --- Comment #7 from Jim Wilson --- Created attachment 38396 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38396=edit patch to add a mulv2di3 pattern

[Bug target/49244] __sync or __atomic builtins will not emit 'lock bts/btr/btc'

2016-05-02 Thread bonzini at gnu dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49244 --- Comment #16 from Paolo Bonzini --- > This also suggests there's an error in the current x86_64 kernel > implementation > as the kernel bitops are supposed to operate on machine word-size locations, > so > it should be using BTSQ not BTSL

[Bug tree-optimization/70916] New: gcc ICE at -O3 on valid code on x86_64-linux-gnu in "tree_operand_check"

2016-05-02 Thread helloqirun at gmail dot com
-linux-gnu Configured with: ../gcc/configure --prefix=/home/absozero/trunk/root-gcc --enable-languages=c,c++ --disable-werror --enable-multilib Thread model: posix gcc version 7.0.0 20160502 (experimental) [trunk revision 235753] (GCC) $ gcc-trunk -O3 abc.c -c abc.c: In function ‘fn1’: abc.c:9:1

[Bug c/67793] Missing warning for C99 static array dimension mismatch

2016-05-02 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67793 Martin Sebor changed: What|Removed |Added Keywords||diagnostic Last reconfirmed|2015-10-01

Re: [PATCH] Fix spec-options.c test case

2016-05-02 Thread Bernd Edlinger
On 02.05.2016 23:28, Oleg Endo wrote: > On Mon, 2016-05-02 at 16:13 +0200, Bernd Schmidt wrote: >> On 05/02/2016 03:43 PM, Bernd Edlinger wrote: >>> Yes, you are right. Only the original use-case seems to be >>> sh-superh-elf specific. But there are also spec strings >>> that are always

[Bug c/45840] Enhance __builtin_object_size to return useful result when applied to T (*p)[N]

2016-05-02 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45840 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug middle-end/49905] Better sanity checking on sprintf src & dest to produce warning for dodgy code ?

2016-05-02 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49905 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: [PATCH #3], Fix _Complex when there are multiple FP types the same size

2016-05-02 Thread Bernd Schmidt
On 05/02/2016 11:10 PM, Michael Meissner wrote: So I would like to commit the following changes (assuming they bootstrap and have no regressions) instead. Are these patches ok for the trunk, and eventually the gcc 6.2 branch if they don't break other back ends? Ok for the non-rs6000 changes.

Re: [PATCH #3], Fix _Complex when there are multiple FP types the same size

2016-05-02 Thread Michael Meissner
The bootstrap and make check succeeded without problems. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797

[Bug target/70915] New: Improve loading 0/-1 in VSX registers on PowerPC

2016-05-02 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70915 Bug ID: 70915 Summary: Improve loading 0/-1 in VSX registers on PowerPC Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: minor Priority: P3 Component:

[Bug testsuite/70520] Incorrect child stack alignment in c-c++-common/asan/clone-test-1.c

2016-05-02 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70520 H.J. Lu changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/49244] __sync or __atomic builtins will not emit 'lock bts/btr/btc'

2016-05-02 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49244 --- Comment #15 from Jakub Jelinek --- Author: jakub Date: Mon May 2 22:12:54 2016 New Revision: 235792 URL: https://gcc.gnu.org/viewcvs?rev=235792=gcc=rev Log: PR target/49244 * gimple.c

Re: Patches to fix optimizer bug & C++ exceptions for GCC VAX backend

2016-05-02 Thread Maciej W. Rozycki
On Tue, 26 Apr 2016, Jeff Law wrote: > > So offhand I think you need an RTL instruction splitter to express this, > > and then avoid fetching 64 bits worth of data from memory -- for the sake > > of matching the indexed addressing mode -- where you only need 32 bits. > > At the hardware

[Bug gcov-profile/69004] Building t-engine on ARM fails during -fprofile-use stage

2016-05-02 Thread tulipawn at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69004 --- Comment #15 from PeteVine --- After deleting physfs profile data I got a little further (using gcc 6.1.0): ../src/luajit2/src/lj_gc.c: In function ‘lj_mem_newgco’: ../src/luajit2/src/lj_gc.c:816:10: error: corrupted value profile: ic

Re: [PATCH #3], Fix _Complex when there are multiple FP types the same size

2016-05-02 Thread Michael Meissner
On Mon, May 02, 2016 at 11:24:25PM +0200, FX wrote: > Hi Michael, > > Thanks for letting us know. > > Right now, the Fortran front-end uses the following real modes: > - those corresponding to {float,double,long_double}_type_node > - TFmode (if libquadmath support is enabled) > and then uses

[Bug testsuite/70520] Incorrect child stack alignment in c-c++-common/asan/clone-test-1.c

2016-05-02 Thread hjl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70520 --- Comment #5 from hjl at gcc dot gnu.org --- Author: hjl Date: Mon May 2 21:45:34 2016 New Revision: 235790 URL: https://gcc.gnu.org/viewcvs?rev=235790=gcc=rev Log: Backport r265322 from llvm upstream Since x86 psABIs require the function

Re: [PATCH] Optimize bit test and * atomics into lock; bt[src] (PR target/49244)

2016-05-02 Thread Jakub Jelinek
On Mon, May 02, 2016 at 11:01:02PM +0200, Uros Bizjak wrote: > > +(define_expand "atomic_bit_test_and_set" > > + [(match_operand:SWI248 0 "register_operand") > > + (match_operand:SWI248 1 "memory_operand") > > + (match_operand:SWI248 2 "nonmemory_operand") > > + (match_operand:SI 3

Re: [PATCH] PR testsuite/70520: Backport r265322 from llvm upstream

2016-05-02 Thread Jakub Jelinek
On Mon, May 02, 2016 at 02:11:49PM -0700, H.J. Lu wrote: > Since x86 psABIs require the function incoming stack must align at 16 > bytes, child process stack passed to clone should be aligned at 16 > bytes. > > Tested on Linux/x86-64. OK for trunk? > > > H.J. > PR testsuite/70520 >

Re: [PATCH] Fix spec-options.c test case

2016-05-02 Thread Oleg Endo
On Mon, 2016-05-02 at 16:13 +0200, Bernd Schmidt wrote: > On 05/02/2016 03:43 PM, Bernd Edlinger wrote: > > Yes, you are right. Only the original use-case seems to be > > sh-superh-elf specific. But there are also spec strings > > that are always available. I think adding -DFOO to > >

Re: [PATCH #3], Fix _Complex when there are multiple FP types the same size

2016-05-02 Thread FX
Hi Michael, Thanks for letting us know. Right now, the Fortran front-end uses the following real modes: - those corresponding to {float,double,long_double}_type_node - TFmode (if libquadmath support is enabled) and then uses the corresponding complex modes. So, I guess the question in your

Re: Is MODES_TIEABLE_P transitive?

2016-05-02 Thread Michael Meissner
On Mon, Apr 25, 2016 at 11:04:01AM -0600, Jeff Law wrote: > On 04/21/2016 01:53 PM, Michael Meissner wrote: > >As I start to allow integer modes into vector registers, I need to revisit > >MODES_TIEABLE_P. I'm wondering if MODES_TIEABLE_P is transitive? > I don't recall a need for it to be

[PATCH] PR testsuite/70520: Backport r265322 from llvm upstream

2016-05-02 Thread H.J. Lu
Since x86 psABIs require the function incoming stack must align at 16 bytes, child process stack passed to clone should be aligned at 16 bytes. Tested on Linux/x86-64. OK for trunk? H.J. PR testsuite/70520 * c-c++-common/asan/clone-test-1.c (main): Align child process

Re: [PATCH #3], Fix _Complex when there are multiple FP types the same size

2016-05-02 Thread Michael Meissner
On Mon, May 02, 2016 at 12:54:43PM +0200, Bernd Schmidt wrote: > On 04/30/2016 06:00 PM, Segher Boessenkool wrote: > >On Fri, Apr 29, 2016 at 04:51:27PM -0400, Michael Meissner wrote: > >>2016-04-29 Michael Meissner > >> > >>* config/rs6000/rs6000.c

[Bug target/65317] [SH] Shifts used instead of and with const_int

2016-05-02 Thread olegendo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65317 --- Comment #5 from Oleg Endo --- (In reply to Segher Boessenkool from comment #4) > > What are the original instructions it tried to combine here? (insn 7 4 8 2 (parallel [ (set (reg:SI 168 [ r ]) (lshiftrt:SI

Re: [PATCH] Optimize bit test and * atomics into lock; bt[src] (PR target/49244)

2016-05-02 Thread Uros Bizjak
On Mon, May 2, 2016 at 6:36 PM, Jakub Jelinek wrote: > Hi! > > This patch adds pattern recognition (see attached testcase on what it e.g. > can handle) of the i?86/x86_64 lock; bt[src] operations. > It is too late to do this during or after RTL expansion, so it is done late >

Re: An abridged "Writing C" for the gcc web pages

2016-05-02 Thread Jeff Law
On 05/02/2016 02:40 PM, Carlos O'Donell wrote: However, in the end, I think that yet-another-document is not the solution we want. What we actually need is a program that just formats your source according to the GNU Coding Style and that way you can always tell your users "Run indent" and be

Re: An abridged "Writing C" for the gcc web pages

2016-05-02 Thread Carlos O'Donell
On 04/22/2016 12:21 PM, Bernd Schmidt wrote: > (Apologies if you get this twice, the mailing list didn't like the > html attachment in the first attempt). > > We frequently get malformatted patches, and it's been brought to my > attention that some people don't even make the effort to read the

[Bug ada/70900] [6 regression] a library-level class-wide subprogram is not generated into object file

2016-05-02 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70900 Eric Botcazou changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ada/70900] [6 regression] a library-level class-wide subprogram is not generated into object file

2016-05-02 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70900 --- Comment #3 from Eric Botcazou --- Author: ebotcazou Date: Mon May 2 20:33:47 2016 New Revision: 235789 URL: https://gcc.gnu.org/viewcvs?rev=235789=gcc=rev Log: PR ada/70900 Backport from mainline 2016-04-18 Ed

[Bug c/70910] error declaring a type in clause-1 of a for statement

2016-05-02 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70910 --- Comment #2 from Martin Sebor --- Thanks for the pointer. That seems like a rather gratuitous restriction, and an unnecessary deviation from the C++ rule where the feature originated. I'm also not sure that the committee's response to the

[Bug rtl-optimization/70890] [7 regression] r235660 miscompiles stage2 compiler on ia64

2016-05-02 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70890 Bill Seurer changed: What|Removed |Added CC||seurer at linux dot vnet.ibm.com ---

[Bug libstdc++/67375] abi::__cxa_demangle crashes demangling a lambda

2016-05-02 Thread j.v.dijk at tue dot nl
from Jan van Dijk --- Has this been fixed in the meantime? I tried to reproduce this on an x86_64 box (OpenSuSE Tumbleweed) with both g++ (SUSE Linux) 5.3.1 20160301 [gcc-5-branch revision 233849] g++ (GCC) 7.0.0 20160502 (experimental) Compilation with default options and running the test code

[Bug c/70911] Typo in "warning: initialized field with side-effects overwritten"

2016-05-02 Thread ch3root at openwall dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70911 --- Comment #2 from Alexander Cherepanov --- I think it's misleading. IMHO "overwritten" would describe the situation when the first initializer is evaluated, the resulting value is written to the field but then the second initializer

Re: r235766 incomplete?

2016-05-02 Thread Jan Hubicka
> On Mon, 2016-05-02 at 11:50 -0600, Martin Sebor wrote: > > Hi Jan, > > > > I just noticed the compilation errors in the attached file with > > the latest trunk. It seems as though your recent patch below may > > be incomplete: > > > >commit 46e5dccc6f188bd0fd5af4e9778f547ab63c9cae > >

Re: [PING][PATCH] New plugin event when evaluating a constexpr call

2016-05-02 Thread Andres Tiraboschi
2016-04-26 10:41 GMT-03:00 Andres Tiraboschi : > Hi, thanks for answering, > > 2016-04-25 16:21 GMT-03:00 Jason Merrill : >> Let's create a constexpr.h rather than expose constexpr internals to all of >> the front end. Really, I'd prefer

Re: [PATCH] Improve detection of constant conditions during jump threading

2016-05-02 Thread H.J. Lu
On Sat, Apr 30, 2016 at 2:28 PM, Patrick Palka wrote: > `On Fri, Apr 29, 2016 at 3:15 PM, Jeff Law wrote: >> On 04/19/2016 11:50 AM, Patrick Palka wrote: >> >>> 1. This patch introduces a "regression" in gcc.dg/tree-ssa/ssa-thread-11.c >>> in that we no

Re: [PATCH, GCC 5] PR 70613, -fabi-version docs don't match implementation

2016-05-02 Thread Jim Wilson
On Mon, Apr 25, 2016 at 11:47 AM, Bernd Schmidt wrote: Here is a patch to correct the -fabi-version docs on the GCC 5 branch. >>> https://gcc.gnu.org/ml/gcc-patches/2016-04/msg00480.html ping^3 I put an explanation of the patch history for gcc-5 in the PR

[Bug c++/62314] Fix-it Hints

2016-05-02 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62314 --- Comment #7 from David Malcolm --- Author: dmalcolm Date: Mon May 2 19:09:30 2016 New Revision: 235785 URL: https://gcc.gnu.org/viewcvs?rev=235785=gcc=rev Log: PR c++/62314: C++: add fixit hint to misspelled member names When we emit a

Re: [PATCH] Allow arg promotion in gimple_builtin_call_types_compatible_p (PR target/49244)

2016-05-02 Thread Richard Biener
On May 2, 2016 6:29:55 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >Most of the builtins don't pass arguments in char/short types, >except for some sync/atomic builtins, some sanitizer builtins and >TM builtins. >On targets where targetm.calls.promote_prototypes returns true (e.g.

[Bug fortran/70914] New: ICE in gimplify_var_or_parm_decl, at gimplify.c:1851 (and endless compilation)

2016-05-02 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70914 Bug ID: 70914 Summary: ICE in gimplify_var_or_parm_decl, at gimplify.c:1851 (and endless compilation) Product: gcc Version: 6.1.0 Status: UNCONFIRMED Severity:

[Bug fortran/70913] ICE in gfc_encode_character, at fortran/target-memory.c:227

2016-05-02 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70913 --- Comment #1 from Gerhard Steinmetz --- For completeness, these variants compile without errors : $ cat z2.f90 program p type t character :: c end type type t2 character :: q =

[Bug fortran/70913] New: ICE in gfc_encode_character, at fortran/target-memory.c:227

2016-05-02 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70913 Bug ID: 70913 Summary: ICE in gfc_encode_character, at fortran/target-memory.c:227 Product: gcc Version: 6.1.0 Status: UNCONFIRMED Severity: normal

[Bug fortran/70854] ICE in gfc_process_block_locals, at fortran/trans-decl.c:6447

2016-05-02 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70854 --- Comment #2 from Gerhard Steinmetz --- As a side note, above examples are more or less a follow-up of pr67885. --- As tested recently, some more examples from gfortran's testsuite seem to be affected

[Bug fortran/60561] ICE in gimplify_var_or_parm_decl, at gimplify.c:1721 for gfortran.dg/associate_1.f03

2016-05-02 Thread gerhard.steinmetz.fort...@t-online.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60561 Gerhard Steinmetz changed: What|Removed |Added CC|

[PATCH, i386]: Merge some more x87 and SSE patterns

2016-05-02 Thread Uros Bizjak
2016-05-02 Uros Bizjak * config/i386/predicates.md (register_ssemem_operand): New predicate. * config/i386/i386.md (*cmpi): Merge from *cmpi_mixed and *cmpi_i387. Disable unsupported alternatives using "enabled" attribute. Use register_ssemem_operand

determining reassociation width

2016-05-02 Thread Aaron Sawdey
So, my first cut at the function to select reassociation width for power was modeled after what I saw i386 and aarch64 doing, which is to return something based on the number of that kind of op we can do at the same time: static int rs6000_reassociation_width (unsigned int opc, enum machine_mode

[Bug c++/62314] Fix-it Hints

2016-05-02 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62314 --- Comment #6 from David Malcolm --- Author: dmalcolm Date: Mon May 2 18:36:40 2016 New Revision: 235777 URL: https://gcc.gnu.org/viewcvs?rev=235777=gcc=rev Log: PR c++/62314: add fixit hint for missing "template <> " in explicit

Re: r235766 incomplete?

2016-05-02 Thread David Malcolm
On Mon, 2016-05-02 at 11:50 -0600, Martin Sebor wrote: > Hi Jan, > > I just noticed the compilation errors in the attached file with > the latest trunk. It seems as though your recent patch below may > be incomplete: > >commit 46e5dccc6f188bd0fd5af4e9778f547ab63c9cae >Author: hubicka

Re: [OpenACC] minor code cleanup

2016-05-02 Thread Jakub Jelinek
On Mon, May 02, 2016 at 02:16:25PM -0400, Nathan Sidwell wrote: > While working on some more loop partitioning improvements, I noticed some > unnecessary checking. By construction an openacc loop must have at least > one head/tail marker, so we should assert that earlier when lowering the > loop

[OpenACC] minor code cleanup

2016-05-02 Thread Nathan Sidwell
While working on some more loop partitioning improvements, I noticed some unnecessary checking. By construction an openacc loop must have at least one head/tail marker, so we should assert that earlier when lowering the loop (in the common compiler) and then rely on it later when processing

[SPARC] Support for --with-{cpu,tune}-{32,64} in sparc*-* targets

2016-05-02 Thread Jose E. Marchesi
Hi people. This patch adds support for the --with-{cpu,tune}-{32,64} configure options to sparc*-* targets. This allows to separately select cpus and tune options for -m32 and -m64 modes in multilib compilers. Tested in sparc64-*-* and sparcv9-*-* targets. 2016-04-28 Jose E. Marchesi

r235766 incomplete?

2016-05-02 Thread Martin Sebor
Hi Jan, I just noticed the compilation errors in the attached file with the latest trunk. It seems as though your recent patch below may be incomplete: commit 46e5dccc6f188bd0fd5af4e9778f547ab63c9cae Author: hubicka Date: Mon May 2 16:55:56

Re: Inline across -ffast-math boundary

2016-05-02 Thread Jan Hubicka
> On Mon, May 02, 2016 at 07:01:38PM +0200, Jan Hubicka wrote: > > This patch impleemnts the suggested check for presence of FP parameters. > > We can play with special casing the moves incrementally. > > This patch has broken the build: > > /home/marek/src/gcc/gcc/ipa-inline.c: In function

Re: (R5900) Implementing Vector Support

2016-05-02 Thread Richard Henderson
On 04/29/2016 07:54 AM, Liu Woon Yung wrote: I've done something like that, but GCC still doesn't select the pattern to use: (define_insn "vec_cmp" Because you've used the wrong name. The patterns are: OPTAB_CD(vec_cmp_optab, "vec_cmp$a$b") OPTAB_CD(vec_cmpu_optab, "vec_cmpu$a$b") I see

Re: [PATCH] Better location info for "incomplete type" error msg (PR c/70756)

2016-05-02 Thread Jason Merrill
On 05/02/2016 12:41 PM, Marek Polacek wrote: On Fri, Apr 29, 2016 at 04:04:13PM -0400, Jason Merrill wrote: On 04/28/2016 11:59 AM, Marek Polacek wrote: 3) for the C++ FE I used a macro so that I don't have to change all the cxx_incomplete_type_error calls now, How about an inline

[Bug c++/70498] Libiberty Demangler segfaults (3)

2016-05-02 Thread bernds at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70498 Bernd Schmidt changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c/70911] Typo in "warning: initialized field with side-effects overwritten"

2016-05-02 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70911 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment

Re: Inline across -ffast-math boundary

2016-05-02 Thread Marek Polacek
On Mon, May 02, 2016 at 07:01:38PM +0200, Jan Hubicka wrote: > This patch impleemnts the suggested check for presence of FP parameters. > We can play with special casing the moves incrementally. This patch has broken the build: /home/marek/src/gcc/gcc/ipa-inline.c: In function ‘bool

[Bug c/70910] error declaring a type in clause-1 of a for statement

2016-05-02 Thread jsm28 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70910 Joseph S. Myers changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug tree-optimization/70912] New: reassociation width needs to be aware of FMA, width of expression, and other architectural details

2016-05-02 Thread acsawdey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70912 Bug ID: 70912 Summary: reassociation width needs to be aware of FMA, width of expression, and other architectural details Product: gcc Version: 7.0 Status: UNCONFIRMED

[Bug c++/70498] Libiberty Demangler segfaults (3)

2016-05-02 Thread bernds at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70498 --- Comment #3 from Bernd Schmidt --- Author: bernds Date: Mon May 2 17:06:40 2016 New Revision: 235767 URL: https://gcc.gnu.org/viewcvs?rev=235767=gcc=rev Log: Demangler integer overflow fixes from Marcel Böhme. PR c++/70498

Re: Fix for PR70498 in Libiberty Demangler

2016-05-02 Thread Bernd Schmidt
On 05/01/2016 06:24 PM, Marcel Böhme wrote: Please attach it (text/plain) instead. Done. That still seemed to be inlined, but I managed to apply this version. Committed. Bernd

Re: option -mprfchw on 2 different Opteron cpus

2016-05-02 Thread NightStrike
On Mon, May 2, 2016 at 5:55 AM, Kumar, Venkataramanan wrote: >> If I compile on a k8 Opteron 248 with -march=native, I do not see -mprfchw >> listed in the options in -fverbose-asm. In the assembly, I see this: >> >> prefetcht0 (%rax) # ivtmp.1160 >>

Re: Inline across -ffast-math boundary

2016-05-02 Thread Jan Hubicka
> On Thu, 21 Apr 2016, Jan Hubicka wrote: > > > Hi, > > this patch implements the long promised logic to inline across -ffast-math > > boundary when eitehr caller or callee has no fp operations in it. This is > > needed to resolve code quality regression on Firefox with LTO where > >

[Bug c/70911] New: Typo in "warning: initialized field with side-effects overwritten"

2016-05-02 Thread ch3root at openwall dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70911 Bug ID: 70911 Summary: Typo in "warning: initialized field with side-effects overwritten" Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal

[Bug target/65317] [SH] Shifts used instead of and with const_int

2016-05-02 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65317 Segher Boessenkool changed: What|Removed |Added CC||segher at gcc dot gnu.org ---

[Bug c/70910] New: error declaring a type in clause-1 of a for statement

2016-05-02 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70910 Bug ID: 70910 Summary: error declaring a type in clause-1 of a for statement Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3

[Bug rtl-optimization/70467] Useless "and [esp],-1" emitted on AND with uint64_t variable

2016-05-02 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70467 --- Comment #11 from Jakub Jelinek --- Author: jakub Date: Mon May 2 16:46:10 2016 New Revision: 235765 URL: https://gcc.gnu.org/viewcvs?rev=235765=gcc=rev Log: PR rtl-optimization/70467 * cse.c (cse_insn): Handle no-op MEM

Re: [PATCH] Better location info for "incomplete type" error msg (PR c/70756)

2016-05-02 Thread Marek Polacek
On Fri, Apr 29, 2016 at 04:04:13PM -0400, Jason Merrill wrote: > On 04/28/2016 11:59 AM, Marek Polacek wrote: > > 3) for the C++ FE I used a macro so that I don't have to change all the > > cxx_incomplete_type_error calls now, > > How about an inline overload, instead? I realized the macro

Re: [PATCH] Improve add/sub TImode double word splitters (PR rtl-optimization/70467)

2016-05-02 Thread Bernd Schmidt
2016-05-02 Jakub Jelinek PR rtl-optimization/70467 * cse.c (cse_insn): Handle no-op MEM moves after folding. * gcc.target/i386/pr70467-1.c: New test. I seem to have a memory of acking this before. Certainly looks OK. Bernd

[PATCH] Optimize bit test and * atomics into lock; bt[src] (PR target/49244)

2016-05-02 Thread Jakub Jelinek
Hi! This patch adds pattern recognition (see attached testcase on what it e.g. can handle) of the i?86/x86_64 lock; bt[src] operations. It is too late to do this during or after RTL expansion, so it is done late during gimple, by recognizing these sequences in the fold builtins pass, turning

[PATCH] Allow arg promotion in gimple_builtin_call_types_compatible_p (PR target/49244)

2016-05-02 Thread Jakub Jelinek
Hi! Most of the builtins don't pass arguments in char/short types, except for some sync/atomic builtins, some sanitizer builtins and TM builtins. On targets where targetm.calls.promote_prototypes returns true (e.g. always on x86_64/i686), unfortunately this means that gimple_call_builtin_p often

[PATCH] Improve add/sub TImode double word splitters (PR rtl-optimization/70467)

2016-05-02 Thread Jakub Jelinek
On Fri, Apr 01, 2016 at 08:29:17PM +0200, Uros Bizjak wrote: > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for stage1 > > (while the previous patch looks simple enough that I'd like to see it in > > 6.x, this one IMHO can wait). > > Yes, please. This is not a regression. So, I'm

Re: [PATCH] Clean up tests where a later dg-do completely overrides another.

2016-05-02 Thread Dominik Vogt
On Mon, May 02, 2016 at 09:29:50AM -0600, Jeff Law wrote: > On 04/29/2016 05:56 PM, Dominik Vogt wrote: > > ... > >Maybe a comment should be added to the test case > > > > /* If this test is *run* (not just compiled) and therefore fails > > on non sh*-targets, this is because of a bug older

[committed] Fix internal-fn handling in ipa-pure-const.c (PR rtl-optimization/70467)

2016-05-02 Thread Jakub Jelinek
Hi! During testing of my PR70467 patch I've run into execute/va-arg-13.c miscompilation, caused by ipa-pure-const.c in ipa mode saying a function using VA_ARG internal function is pure - it might not be, e.g. on x86_64 where va_list is [1] array of struct and is passed to this "pure" function, it

[Bug c/70756] Wrong column number shown for "error: invalid use of flexible array member"

2016-05-02 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70756 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment #3

[Bug rtl-optimization/70467] Useless "and [esp],-1" emitted on AND with uint64_t variable

2016-05-02 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70467 --- Comment #10 from Jakub Jelinek --- Author: jakub Date: Mon May 2 16:17:02 2016 New Revision: 235764 URL: https://gcc.gnu.org/viewcvs?rev=235764=gcc=rev Log: PR rtl-optimization/70467 * ipa-pure-const.c (check_call): Handle

[Bug libstdc++/70101] Allocator-extended priority_queue constructors are badly broken

2016-05-02 Thread rs2740 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70101 --- Comment #1 from TC --- Test case for everything except the first: #include #include #include #include #include struct Cmp : std::less { explicit Cmp(int) {}; }; Cmp comp(1); std::allocator alloc; std::vector vec; using PQ =

Re: [PATCH] Re-use cc1-checksum.c for stage-final

2016-05-02 Thread Jeff Law
On 04/29/2016 05:36 AM, Richard Biener wrote: On Thu, 28 Apr 2016, Jeff Law wrote: On 04/28/2016 02:49 AM, Richard Biener wrote: The following prototype patch re-uses cc1-checksum.c from the previous stage when compiling stage-final. This eventually allows to compare cc1 from the last two

Re: [PATCH][CilkPlus] Merge libcilkrts from upstream

2016-05-02 Thread Jeff Law
On 04/29/2016 05:36 AM, Ilya Verbin wrote: Hi! This patch brings the latest libcilkrts from upstream. Regtested on i686-linux and x86_64-linux. Abidiff: Functions changes summary: 0 Removed, 1 Changed (16 filtered out), 2 Added functions Variables changes summary: 0 Removed, 0 Changed (1

Re: [patch] cleanups for vtable-verify

2016-05-02 Thread Jeff Law
On 05/01/2016 07:34 AM, Steven Bosscher wrote: Hello, This patch is random cleanups on the vtable-verify code. OK for trunk? Ciao! Steven gcc/ * vtable-verify.h (verify_vtbl_ptr_fndecl): Add GTY markers. (num_vtable_map_nodes): Remove extern declaration.

Re: [libvtv, build] Don't install libvtv without --enable-vtable-verify

2016-05-02 Thread Jeff Law
On 05/02/2016 07:33 AM, Rainer Orth wrote: When installing gcc 6.1.0 on Solaris 12, installation failed in libvtv: libtool: install: /usr/gnu/bin/install -c .libs/libvtv.lai /var/gcc/gcc-6.1.0/12-gcc-gas/install/vol/gcc-6/lib/amd64/libvtv.la libtool: install: /usr/gnu/bin/install -c

Re: [PATCH] Clean up tests where a later dg-do completely overrides another.

2016-05-02 Thread Jeff Law
On 04/29/2016 05:56 PM, Dominik Vogt wrote: Yeah, sorry, I really should have mentioned this but forgot about it. It's a bug in DejaGnu. When it encounters a conditional dg-do and the condition does not match, it *still* replaces the do-action of a prior dg-do with the current one. With

Fix for PR70909 in Libiberty Demangler (4)

2016-05-02 Thread Marcel Böhme
Hi, This fixes several stack overflows due to infinite recursion in d_print_comp (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70909). The method d_print_comp in cp-demangle.c recursively constructs the d_print_info dpi from the demangle_component dc. The method d_print_comp_inner traverses

  1   2   3   >