Re: Power 8 in-core crypto not working as expected

2017-09-06 Thread R0b0t1
On Wed, Sep 6, 2017 at 11:37 PM, Jeffrey Walton wrote: > Hi Everyone, > > I'm on gcc rather than gcc-help because we need to talk with some GCC > devs who can help take this further. > > I have implementation for AES on Power 8 using GCC's built-ins. Its > available for

Re: [llvm-dev] DragonEgg for GCC v8.x and LLVM v6.x is just able to work

2017-09-06 Thread Leslie Zhai
Dear Chris, Thanks for your kind response! The motivating of this work: 1. Clang can not build Linux https://bugs.llvm.org/show_bug.cgi?id=22830 and LLVMLinux patch was not be maintained? http://llvm.linuxfoundation.org/index.php/Main_Page 2. Clang analyzer Frontend can not static analysis

[Bug c++/82110] Concept for default constructing works with new T, not with new T[1]

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82110 Eric Gallager changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug middle-end/82123] [7 regression] spurious -Wformat-overflow warning for converted vars

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82123 Eric Gallager changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Power 8 in-core crypto not working as expected

2017-09-06 Thread Jeffrey Walton
Hi Everyone, I'm on gcc rather than gcc-help because we need to talk with some GCC devs who can help take this further. I have implementation for AES on Power 8 using GCC's built-ins. Its available for inspection and download at https://github.com/noloader/AES-Power8. The problem is, it does not

Re: [llvm-dev] DragonEgg for GCC v8.x and LLVM v6.x is just able to work

2017-09-06 Thread Chris Lattner
> On Sep 4, 2017, at 8:13 PM, Leslie Zhai via llvm-dev > wrote: > > Hi LLVM and GCC developers, > > LLVM China http://www.llvm.org.cn forked DragonEgg > https://github.com/LLVM-China/dragonegg because: > > * Some subprojects are impractical or uninteresting to

[Bug c/54113] -Wmissing-prototypes produces false alarms for C99 inline functions

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54113 Eric Gallager changed: What|Removed |Added CC||egallager at gcc dot gnu.org ---

[Bug middle-end/81318] [8 regression] ICE in to_reg_br_prob_base, at profile-count.h:189

2017-09-06 Thread daniel.black at au dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81318 Daniel Black changed: What|Removed |Added CC||daniel.black at au dot ibm.com ---

[Bug c++/70029] [8 Regression] ICE with C++11 and -flto

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70029 --- Comment #13 from Jason Merrill --- I believe r241831 fixed the actual problem that verify_type was catching. This still fails because free_lang_data_in_type clears TYPE_LANG_FLAG_4 on 't' but not 'ct'. Should something have added

[Bug c++/70029] [8 Regression] ICE with C++11 and -flto

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70029 Jason Merrill changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug c++/81640] [8 Regression] ICE in lookup_fnfields_slot_nolazy w/ -Wshadow=compatible-local

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81640 Jason Merrill changed: What|Removed |Added Status|NEW |RESOLVED CC|

[Bug c++/82053] [8 Regression] ICE on invalid code

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82053 Jason Merrill changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/82053] [8 Regression] ICE on invalid code

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82053 --- Comment #2 from Jason Merrill --- Author: jason Date: Thu Sep 7 01:02:46 2017 New Revision: 251826 URL: https://gcc.gnu.org/viewcvs?rev=251826=gcc=rev Log: PR c++/82053 - ICE with default argument in lambda in template *

C++ PATCH for c++/82053, ICE with default argument in lambda in template

2017-09-06 Thread Jason Merrill
When we regenerate a lambda, the resulting op() doesn't have any template information, so we can't delay instantiating default arguments like we do for a normal template function. I believe this is also the direction of the core working group for default arguments in local extern function

Re: [PATCH 1/3] improve detection of attribute conflicts (PR 81544)

2017-09-06 Thread Joseph Myers
On Thu, 17 Aug 2017, Martin Sebor wrote: > +/* Check LAST_DECL and NODE of the same symbol for attributes that are > + recorded in EXCL to be mutually exclusive with ATTRNAME, diagnose > + them, and return true if any have been found. NODE can be a DECL > + or a TYPE. */ > + > +static

[Bug c++/82125] New: Suboptimal error message for range-based for

2017-09-06 Thread rs2740 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82125 Bug ID: 82125 Summary: Suboptimal error message for range-based for Product: gcc Version: 8.0 Status: UNCONFIRMED Keywords: diagnostic Severity: normal

Re: [PATCH] correct documentation of attribute ifunc (PR 81882)

2017-09-06 Thread Joseph Myers
This patch is OK with the spacing in the function prototype fixed as noted to follow normal GNU standards. -- Joseph S. Myers jos...@codesourcery.com

gcc-6-20170906 is now available

2017-09-06 Thread gccadmin
Snapshot gcc-6-20170906 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/6-20170906/ 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

Re: [PATCH, rs6000] Add builtins to convert from float/double to int/long using current rounding mode

2017-09-06 Thread Carl Love
On Wed, 2017-09-06 at 16:13 -0500, Pat Haugen wrote: > On 09/06/2017 11:24 AM, Carl Love wrote: > > + "fctiw %1,%1; mfvsrd %0,%1; extsw %0,%0" > > + [(set_attr "type" "integer") > > + (set_attr "length" "4")]) > > Should be type "three" and length "12". > > -Pat Pat: Yes, that is wrong in

Re: [RFA] [PATCH 4/4] Ignore reads of "dead" memory locations in DSE

2017-09-06 Thread Jeff Law
Another old patch getting resurrected... On 01/04/2017 06:50 AM, Richard Biener wrote: > On Thu, Dec 22, 2016 at 7:26 AM, Jeff Law wrote: >> This is the final patch in the kit to improve our DSE implementation. >> >> It's based on a observation by Richi. Namely that a read

Re: Add support to trace comparison instructions and switch statements

2017-09-06 Thread Jakub Jelinek
On Wed, Sep 06, 2017 at 10:08:01PM +0200, David Edelsohn wrote: > This change broke bootstrap on AIX because sancov.c now references a > macro that is defined as a function on AIX. sancov.c needs to include > tm_p.h to pull in the target-dependent prototypes. The following > patch works for me.

Re: [PATCH, rs6000] Add builtins to convert from float/double to int/long using current rounding mode

2017-09-06 Thread Pat Haugen
On 09/06/2017 11:24 AM, Carl Love wrote: > + "fctiw %1,%1; mfvsrd %0,%1; extsw %0,%0" > + [(set_attr "type" "integer") > + (set_attr "length" "4")]) Should be type "three" and length "12". -Pat

RFC: Representation of runtime offsets and sizes

2017-09-06 Thread Richard Sandiford
The next main step in the SVE submission is to add support for offsets and sizes that are a runtime invariant rather than a compile time constant. This is an RFC about our approach for doing that. It's an update of https://gcc.gnu.org/ml/gcc/2016-11/msg00031.html (which covered more topics than

Re: Add support to trace comparison instructions and switch statements

2017-09-06 Thread David Edelsohn
This change broke bootstrap on AIX because sancov.c now references a macro that is defined as a function on AIX. sancov.c needs to include tm_p.h to pull in the target-dependent prototypes. The following patch works for me. Is this okay? * sancov.c: Include tm_p.h. Index: sancov.c

[Bug c++/69953] [5/6 Regression] Using lto causes gtkmm/gparted and gtkmm/inkscape compile to fail

2017-09-06 Thread dilfridge at gentoo dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69953 --- Comment #37 from Andreas K. Huettel --- Created attachment 42140 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42140=edit gparted build log Here's the build log from my Gentoo colleague. If you need more, please tell me precisely

[Bug c++/82070] [8 Regression] inaccessible within this context in lambda rejects valid

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82070 Jason Merrill changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug libgomp/82124] FAIL: libgomp.c++/pr69393.C (test for excess errors)

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82124 --- Comment #2 from Eric Gallager --- Created attachment 42139 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42139=edit Full testresults diff (attaching the full diff between the 2 test logs to remind myself to open other bugs for the

Re: [PATCH 1/1] sparc: support for -mmisalign in the SPARC M8

2017-09-06 Thread Qing Zhao
Just a followup on this patch. We did some run-time performance testing internally on this set of change on sparc M8 machine with -mmisalign and -mno-misalign based on the latest upstream gcc for CPU2017 C/C++ SPEED run: ***without -O, -mmisalign slowdown the run-time performance about 4% on

[Bug libgomp/82124] FAIL: libgomp.c++/pr69393.C (test for excess errors)

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82124 --- Comment #1 from Eric Gallager --- Created attachment 42138 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42138=edit bzipped libgomp.log Oops, the log was too big to attach on its own; trying again after compressing it...

[Bug libgomp/82124] New: FAIL: libgomp.c++/pr69393.C (test for excess errors)

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82124 Bug ID: 82124 Summary: FAIL: libgomp.c++/pr69393.C (test for excess errors) Product: gcc Version: 8.0 Status: UNCONFIRMED Keywords: openmp Severity: normal

[Bug c++/82070] [8 Regression] inaccessible within this context in lambda rejects valid

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82070 --- Comment #1 from Jason Merrill --- Author: jason Date: Wed Sep 6 19:36:48 2017 New Revision: 251819 URL: https://gcc.gnu.org/viewcvs?rev=251819=gcc=rev Log: PR c++/82070 - error with nested lambda capture * pt.c

[Bug c++/82080] ICE: Segmentation fault

2017-09-06 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82080 Eric Gallager changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: dejagnu version update?

2017-09-06 Thread David Edelsohn
On Wed, Sep 6, 2017 at 8:48 PM, Jonathan Wakely wrote: > On 25 August 2017 at 14:55, David Edelsohn wrote: >> On Fri, Aug 25, 2017 at 9:50 AM, Rainer Orth >> wrote: >>> Hi H.J., >>> On Fri, Aug 25, 2017 at 6:32 AM, David Edelsohn

Re: [PATCH, rs6000] Add support for vec_xst_len_r() and vec_xl_len_r() builtins

2017-09-06 Thread Segher Boessenkool
Hi Carl, On Wed, Sep 06, 2017 at 08:22:03AM -0700, Carl Love wrote: > (define_insn "*stxvl"): add missing argument to the sldi instruction. s/add/Add/ . This one-liner fix is approved right now, please commit it as a separate patch. > +(define_insn "addi_neg16" > + [(set

[Bug libstdc++/79433] __has_include() is true but #include gives #error when -std=old

2017-09-06 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79433 Jonathan Wakely changed: What|Removed |Added Status|NEW |ASSIGNED

C++ PATCH for c++/82070, error with nested lambda capture

2017-09-06 Thread Jason Merrill
I was expecting that references to capture proxies would be resolved in the reconstructed lambda by normal name lookup, but that doesn't work in decltype, and processing the nested lambda really wants to find the new capture proxy, not the captured variable. Tested x86_64-pc-linux-gnu, applying

[Bug tree-optimization/81987] [8 Regression] ICE in verify_ssa with -O3 -march=skylake-avx512

2017-09-06 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81987 Bill Schmidt changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug tree-optimization/81987] [8 Regression] ICE in verify_ssa with -O3 -march=skylake-avx512

2017-09-06 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81987 --- Comment #11 from Bill Schmidt --- Author: wschmidt Date: Wed Sep 6 18:48:50 2017 New Revision: 251817 URL: https://gcc.gnu.org/viewcvs?rev=251817=gcc=rev Log: [gcc] 2017-09-06 Bill Schmidt Backport

Re: dejagnu version update?

2017-09-06 Thread Jonathan Wakely
On 25 August 2017 at 14:55, David Edelsohn wrote: > On Fri, Aug 25, 2017 at 9:50 AM, Rainer Orth > wrote: >> Hi H.J., >> >>> On Fri, Aug 25, 2017 at 6:32 AM, David Edelsohn wrote: On Fri, Aug 25, 2017 at 9:24 AM, H.J. Lu

Re: Redundant sign-extension instructions on RISC-V

2017-09-06 Thread Jeff Law
On 09/06/2017 11:17 AM, Richard Henderson wrote: > On 09/06/2017 09:53 AM, Jeff Law wrote: >>> I think the easiest solution to this is for combine to notice when IOR has >>> operands with non-zero-bits that do not overlap, convert the operation to >>> ADD. >>> That allows the final two insns to

[Bug tree-optimization/81987] [8 Regression] ICE in verify_ssa with -O3 -march=skylake-avx512

2017-09-06 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81987 --- Comment #10 from Bill Schmidt --- Author: wschmidt Date: Wed Sep 6 18:44:51 2017 New Revision: 251816 URL: https://gcc.gnu.org/viewcvs?rev=251816=gcc=rev Log: [gcc] 2017-09-06 Bill Schmidt Backport

[Bug tree-optimization/81987] [8 Regression] ICE in verify_ssa with -O3 -march=skylake-avx512

2017-09-06 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81987 --- Comment #9 from Bill Schmidt --- Author: wschmidt Date: Wed Sep 6 18:42:56 2017 New Revision: 251815 URL: https://gcc.gnu.org/viewcvs?rev=251815=gcc=rev Log: [gcc] 2017-09-06 Bill Schmidt Backport

Re: [Patch, fortran] Parameterized Derived Types

2017-09-06 Thread Damian Rouson
  Thanks for your tireless efforts on this, Paul! I look forward to trying this out after it hits the trunk. Your phrase “last unimplemented F2003” feature bolsters my suspicion that it might be ok to switch the features listed as “Partial” on the Fortran wiki to “Yes." I suppose the

[Bug c++/82070] [8 Regression] inaccessible within this context in lambda rejects valid

2017-09-06 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82070 Jason Merrill changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

Re: Redundant sign-extension instructions on RISC-V

2017-09-06 Thread Richard Henderson
On 09/06/2017 10:17 AM, Richard Henderson wrote: >> Yea. I'd also expect zero/nonzero bits tracking in combine to catch >> this. Shouldn't the sign bit be known to be zero after the shift which >> makes the extension redundant regardless of the SUBREG_PROMOTED flag? > You're right, this should

[Bug libstdc++/77726] On MinGW targets, user-defined `operator delete(void *)` is not called if the sized-deallocation version is not provided in C++14 mode when libstdc++ is linked dynamically

2017-09-06 Thread lh_mouse at 126 dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77726 --- Comment #2 from Liu Hao --- It isn't weired if you know how a Dynamic-Link Library (DLL) on Windows works different than a Shared Object (SO) on Linux. Windows does not have a dynamic linker such as `ld.so` on Linux. Symbols in a DLL are

Re: [Patch, fortran] Parameterized Derived Types

2017-09-06 Thread Janus Weil
Hi Paul, thanks for your patch! It's really great to finally see PDTs come to gfortran. You're a hero, man ;) Also: Sorry about the silence. It's certainly not due to lack of interest, but rather lack of time (day job and private life taking up all of mine at the moment). In my current

Re: [PATCH] [Aarch64] Optimize subtract in shift counts

2017-09-06 Thread Richard Sandiford
Richard Sandiford writes: > Richard Sandiford writes: >> Michael Collison writes: >>> Richard, >>> >>> The problem with this approach for Aarch64 is that >>> TARGET_SHIFT_TRUNCATION_MASK is based on

Re: Redundant sign-extension instructions on RISC-V

2017-09-06 Thread Richard Henderson
On 09/06/2017 09:53 AM, Jeff Law wrote: >> I think the easiest solution to this is for combine to notice when IOR has >> operands with non-zero-bits that do not overlap, convert the operation to >> ADD. >> That allows the final two insns to fold to "addw" and the compiler need do no >> further

Re: [PATCH], Enable -mfloat128 by default on PowerPC VSX systems

2017-09-06 Thread Segher Boessenkool
On Wed, Sep 06, 2017 at 01:48:38AM -0400, Michael Meissner wrote: > Here is a respin of the patch to enable -mfloat128 on PowerPC Linux systems > now > that the libquadmath patch has been applied. I rebased the patches against > the > top of the trunk on Tuesday (subversion id 251609). > > I

Re: [PATCH] [Aarch64] Optimize subtract in shift counts

2017-09-06 Thread Richard Sandiford
Richard Sandiford writes: > Michael Collison writes: >> Richard, >> >> The problem with this approach for Aarch64 is that >> TARGET_SHIFT_TRUNCATION_MASK is based on SHIFT_COUNT_TRUNCATED which is >> normally 0 as it based on the

Re: Redundant sign-extension instructions on RISC-V

2017-09-06 Thread Richard Henderson
On 08/30/2017 02:43 AM, Michael Clark wrote: > POINTERS_EXTEND_UNSIGNED -1 (which is true) is defined on some targets. I > assume they sign-extend but the meaning has been overloaded. Just for your edification, this is for e.g. ia64's "addp4" instruction and it is not a normal extension. A

Re: [PATCH] [Aarch64] Optimize subtract in shift counts

2017-09-06 Thread Richard Sandiford
Michael Collison writes: > Richard, > > The problem with this approach for Aarch64 is that > TARGET_SHIFT_TRUNCATION_MASK is based on SHIFT_COUNT_TRUNCATED which is > normally 0 as it based on the TARGET_SIMD flag. Maybe I'm wrong, but that seems like a missed

Re: Redundant sign-extension instructions on RISC-V

2017-09-06 Thread Jeff Law
On 09/06/2017 10:43 AM, Richard Henderson wrote: > On 08/29/2017 05:36 PM, Michael Clark wrote: >> We’re investigating an issue with redundant sign-extension instructions >> being emitted with the riscv backend. Firstly I would like to state that >> riscv is possibly a unique backend with

Re: [PATCH] Fix rs6000 sysv4 -fPIC hot/cold partitioning handling (PR target/81979)

2017-09-06 Thread Segher Boessenkool
On Wed, Sep 06, 2017 at 06:26:10PM +0200, Jakub Jelinek wrote: > > Maybe this "switch to the other section" thing should be abstracted out? > > Messing with in_cold_section_p is a bit dirty. > > But it reflects the reality, and is what final.c and varasm.c also do. Yes, but those aren't target

Re: Redundant sign-extension instructions on RISC-V

2017-09-06 Thread Richard Henderson
On 08/29/2017 05:36 PM, Michael Clark wrote: > We’re investigating an issue with redundant sign-extension instructions being > emitted with the riscv backend. Firstly I would like to state that riscv is > possibly a unique backend with respect to its canonical sign-extended > register form due

[committed][Testsuite] PR78468 - add alloca alignment test

2017-09-06 Thread Wilco Dijkstra
Add an alignment test to check that aligned alloca's really do get correctly aligned. Some targets may not ensure SP is always a multiple of STACK_BOUNDARY (particularly with outgoing arguments), which means aligned alloca does not get correctly aligned. This can be fixed either by aligning the

[Bug middle-end/78468] [8 regression] libgomp.c/reduction-10.c and many more FAIL

2017-09-06 Thread wilco at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468 --- Comment #43 from Wilco --- Author: wilco Date: Wed Sep 6 16:34:54 2017 New Revision: 251811 URL: https://gcc.gnu.org/viewcvs?rev=251811=gcc=rev Log: PR78468 - add alloca alignment test Add an alignment test to check that aligned alloca's

Re: [PATCH] Fix rs6000 sysv4 -fPIC hot/cold partitioning handling (PR target/81979)

2017-09-06 Thread Jakub Jelinek
On Wed, Sep 06, 2017 at 11:10:07AM -0500, Segher Boessenkool wrote: > >for (insn = get_insns (); insn; insn = NEXT_INSN (insn)) > > { > > > if (INSN_P (insn)) > > @@ -25270,10 +25273,14 @@ uses_TOC (void) > > sub = XEXP (sub, 0); > > if (GET_CODE (sub) ==

[PATCH, rs6000] Add builtins to convert from float/double to int/long using current rounding mode

2017-09-06 Thread Carl Love
GCC Maintainers: The following patch adds support for a couple of requested builtins that convert from float/double to int / long using the current rounding mode. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE). Please let me know if the following patch is acceptable.

[Bug libstdc++/81413] overloaded new and delete might not be called when dynamically linked with libstdc++

2017-09-06 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81413 Jonathan Wakely changed: What|Removed |Added CC||drizt at land dot ru --- Comment #2

[Bug libstdc++/82122] Overloaded operator new/delete in MinGW is not calling from .dlls

2017-09-06 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82122 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug libstdc++/77726] On MinGW targets, user-defined `operator delete(void *)` is not called if the sized-deallocation version is not provided in C++14 mode when libstdc++ is linked dynamically

2017-09-06 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77726 Jonathan Wakely changed: What|Removed |Added Severity|major |normal

[Bug libstdc++/77726] On MinGW targets, user-defined `operator delete(void *)` is not called if the sized-deallocation version is not provided in C++14 mode when libstdc++ is linked dynamically

2017-09-06 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77726 Jonathan Wakely changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

[Bug c++/82067] G++ has an internal compiler error in possible_polymorphic_call_targets, at ipa-devirt.c:1557

2017-09-06 Thread jupitercuso4 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82067 --- Comment #5 from jupitercuso4 at gmail dot com --- $ g++ -std=c++11 -O3 --save-temps test.i test.cpp: In constructor 'xtsc_component::xtsc_queue_pin::xtsc_queue_pin(sc_core::sc_module_name, const xtsc_component::xtsc_queue_pin_parms&)':

Re: [PATCH] Fix rs6000 sysv4 -fPIC hot/cold partitioning handling (PR target/81979)

2017-09-06 Thread Segher Boessenkool
Hi, On Tue, Sep 05, 2017 at 11:27:25PM +0200, Jakub Jelinek wrote: > On powerpc with sysv4 -fPIC we emit something like > .LCL0: > .long .LCTOC1-.LCF0 > before we start emitting the function, and in the prologue we emit > .LCF0: > and some code. This fails to assemble if the prologue is

RE: [PATCH][compare-elim] Merge zero-comparisons with normal ops

2017-09-06 Thread Michael Collison
Patch updated with all relevant comments and suggestions. Bootstrapped and tested on arm-none-linux-gnueabihf, and aarch64-none-linux-gnu and x86_64. Ok for trunk? 2017-08-05 Kyrylo Tkachov Michael Collison *

[Bug middle-end/82123] New: [7 regression] spurious -Wformat-overflow warning for converted vars

2017-09-06 Thread arnd at linaro dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82123 Bug ID: 82123 Summary: [7 regression] spurious -Wformat-overflow warning for converted vars Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal

RE: [PATCH] [Aarch64] Optimize subtract in shift counts

2017-09-06 Thread Michael Collison
Richard, The problem with this approach for Aarch64 is that TARGET_SHIFT_TRUNCATION_MASK is based on SHIFT_COUNT_TRUNCATED which is normally 0 as it based on the TARGET_SIMD flag. -Original Message- From: Richard Sandiford [mailto:richard.sandif...@linaro.org] Sent: Wednesday,

Re: [PATCH] Fix ICE in categorize_decl_for_section with TLS decl (PR middle-end/82095)

2017-09-06 Thread Jakub Jelinek
On Wed, Sep 06, 2017 at 09:29:25AM -0600, Jeff Law wrote: > > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for > > trunk? > > > > 2017-09-05 Jakub Jelinek > > > > PR middle-end/82095 > > * varasm.c (categorize_decl_for_section): Use

[C++ PATCH] Merge fn and non-fn lookup interface

2017-09-06 Thread Nathan Sidwell
This patch merges the lookup of function and non-function member lookup into get_class_binding_direct. lookup_field_1 becomes an internal detail. We grow a tri-valued argument to get_class_binding_direct: <0 -- caller wants functions =0 -- caller wants whatever is bound >0 -- caller wants

Re: [PATCH, ARM] correctly encode the CC reg data flow

2017-09-06 Thread Kyrill Tkachov
On 06/09/17 14:17, Bernd Edlinger wrote: On 09/06/17 14:51, Richard Earnshaw (lists) wrote: On 06/09/17 13:44, Bernd Edlinger wrote: On 09/04/17 21:54, Bernd Edlinger wrote: Hi Kyrill, Thanks for your review! On 09/04/17 15:55, Kyrill Tkachov wrote: Hi Bernd, On 18/01/17 15:36, Bernd

Re: [PATCH] Fix ICE in categorize_decl_for_section with TLS decl (PR middle-end/82095)

2017-09-06 Thread Jeff Law
On 09/05/2017 03:16 PM, Jakub Jelinek wrote: > Hi! > > If a DECL_THREAD_LOCAL_P decl has NULL DECL_INITIAL and > -fzero-initialized-in-bss (the default), we ICE starting with > r251602, which changed bss_initializer_p: > + /* Do not put constants into the .bss section, they belong in a readonly

[Bug fortran/82121] [7/8 Regression] Unclassifiable statement during compilation when assigning to a Character array in a derived type contained in a ASSOCIATE statement

2017-09-06 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82121 Dominique d'Humieres changed: What|Removed |Added Priority|P3 |P4

[PATCH, config.gcc] fix case filter for powerpc-*-vxworkspe

2017-09-06 Thread Olivier Hainque
To match on vxworks*spe so it applies to VxWorks 7 as well. Committing to mainline after verifying for e500v2-wrs-vxworks7 that we now include config/powerpcspe/vxworks.h instead of config/rs6000/vxworks.h. Olivier 2017-09-06 Olivier Hainque * config.gcc

[PATCH, rs6000] Add support for vec_xst_len_r() and vec_xl_len_r() builtins

2017-09-06 Thread Carl Love
GCC Maintainers: The following patch adds support for the vec_xst_len_r() and vec_xl_len_r() Powerr 9 builtins. The patch has been run on powerpc64le-unknown-linux-gnu (Power 9 LE). No regressions were found but it does seem to "fix" a couple of existing tests. 136a137 > FAIL: TestCgoCallbackGC

[arm-embedded] [PATCH 3/3, GCC/ARM] Add support for ARM Cortex-R52 processor

2017-09-06 Thread Thomas Preudhomme
Hi, We have decided to apply the following patch to the embedded-7-branch to enable Arm Cortex-R52 support. *** gcc/ChangeLog.arm *** 2017-09-04 Thomas Preud'homme Backport from mainline 2017-07-14 Thomas Preud'homme

[arm-embedded] [PATCH, GCC/ARM] Rewire -mfpu=fp-armv8 as VFPv5 + D32 + DP

2017-09-06 Thread Thomas Preudhomme
Hi, We have decided to apply the following patch to the embedded-7-branch to enable ARMv8-R support. ChangeLog entry is as follows: *** gcc/ChangeLog.arm *** 2017-09-04 Thomas Preud'homme Backport from mainline 2017-07-14 Thomas Preud'homme

[arm-embedded] [PATCH 2/3, GCC/ARM] Add support for ARMv8-R architecture

2017-09-06 Thread Thomas Preudhomme
Hi, We have decided to apply the following patch to the embedded-7-branch to enable ARMv8-R support. ChangeLog entry is as follows: *** gcc/ChangeLog.arm *** 2017-09-04 Thomas Preud'homme Backport from mainline 2017-07-06 Thomas Preud'homme

[arm-embedded] [PATCH 1/3, GCC/ARM, ping] Add MIDR info for ARM Cortex-R7 and Cortex-R8

2017-09-06 Thread Thomas Preudhomme
Hi, We have decided to apply the following patch to the embedded-7-branch as a dependency patch to enable ARMv8-R support. ChangeLog entry is as follows: *** gcc/ChangeLog.arm *** 2017-09-04 Thomas Preud'homme Backport from mainline 2017-07-04

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

2017-09-06 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #4 from Jakub Jelinek --- Author: jakub Date: Wed Sep 6 15:10:28 2017 New Revision: 251806 URL: https://gcc.gnu.org/viewcvs?rev=251806=gcc=rev Log: PR testsuite/82120 * gcc.dg/tree-ssa/pr81588.c: Don't run on

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

2017-09-06 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #3 from Thomas Preud'homme --- (In reply to Jakub Jelinek from comment #2) > This is a total mess. I've copied a boilerplate from other tests that > require branch cost of 2. > I guess > 2017-09-06 Jakub Jelinek

Re: Add support to trace comparison instructions and switch statements

2017-09-06 Thread Jakub Jelinek
On Wed, Sep 06, 2017 at 04:37:18PM +0200, Jakub Jelinek wrote: > Ok. Please make sure those entrypoints make it into the various example > __sanitier_cov_trace* fuzzer implementations though, so that people using > -fsanitize-coverage=trace-cmp in GCC will not need to hack stuff themselves. > At

Re: Add support to trace comparison instructions and switch statements

2017-09-06 Thread Jakub Jelinek
On Wed, Sep 06, 2017 at 07:47:29PM +0800, 吴潍浠(此彼) wrote: > Hi Jakub > I compiled libjpeg-turbo and libdng_sdk with options "-g -O3 -Wall > -fsanitize-coverage=trace-pc,trace-cmp -fsanitize=address". > And run my fuzzer with pc and cmp feedbacks for hours. It works fine. > About

[Bug c++/78269] FAIL: FAIL: g++.dg/cpp1z/noexcept-type11.C and FAIL: g++.dg/cpp1z/noexcept-type9.C

2017-09-06 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78269 --- Comment #10 from Thomas Preud'homme --- (In reply to Thomas Preud'homme from comment #9) > (In reply to Paolo Carlini from comment #8) > > Confirmed that 7.1.0 is fine. > > It is indeed. As can be seen in my comment #4, this was fixed

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

2017-09-06 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 --- Comment #2 from Jakub Jelinek --- This is a total mess. I've copied a boilerplate from other tests that require branch cost of 2. I guess 2017-09-06 Jakub Jelinek PR testsuite/82120 *

[Bug c++/78269] FAIL: FAIL: g++.dg/cpp1z/noexcept-type11.C and FAIL: g++.dg/cpp1z/noexcept-type9.C

2017-09-06 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78269 --- Comment #9 from Thomas Preud'homme --- (In reply to Paolo Carlini from comment #8) > Confirmed that 7.1.0 is fine. It is indeed. As can be seen in my comment #4, this was fixed somewhere before 14th of November 2016, well before GCC 7

[Bug c++/78269] FAIL: FAIL: g++.dg/cpp1z/noexcept-type11.C and FAIL: g++.dg/cpp1z/noexcept-type9.C

2017-09-06 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78269 Paolo Carlini changed: What|Removed |Added Status|NEW |RESOLVED

[Bug target/82108] [7 Regression] Wrong vectorized code generated for x86_64

2017-09-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82108 Richard Biener changed: What|Removed |Added Priority|P3 |P2 Known to work|

[Bug target/82112] internal compiler error: in fold_convert_loc, at fold-const.c:2262

2017-09-06 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82112 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug testsuite/82120] FAIL: gcc.dg/tree-ssa/pr81588.c

2017-09-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 Richard Biener changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment

[Bug libstdc++/82122] New: Overloaded operator new/delete in MinGW is not calling from .dlls

2017-09-06 Thread drizt at land dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82122 Bug ID: 82122 Summary: Overloaded operator new/delete in MinGW is not calling from .dlls Product: gcc Version: 7.2.0 Status: UNCONFIRMED Severity: normal

[Bug fortran/82121] New: Unclassifiable statement during compilation when assigning to a Character array in a derived type contained in a ASSOCIATE statement

2017-09-06 Thread iain.miller at ecmwf dot int
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82121 Bug ID: 82121 Summary: Unclassifiable statement during compilation when assigning to a Character array in a derived type contained in a ASSOCIATE statement Product: gcc

[PATCH 2/2] [arm] Improve error checking in parsecpu.awk

2017-09-06 Thread Richard Earnshaw (lists)
This patch adds a bit more error checking to parsecpu.awk to ensure that statements are not missing arguments or have excess arguments beyond those permitted. It also slightly improves the handling of errors so that we terminate properly if parsing fails and be as helpful as we can while in the

[Bug tree-optimization/81588] [7/8 Regression] Wrong code at -O2

2017-09-06 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81588 --- Comment #10 from Christophe Lyon --- (In reply to Christophe Lyon from comment #9) > I've noticed that the new testcase (gcc.dg/tree-ssa/pr81588.c) fails on the > gcc-7 branch (r251446) on arm-linux-gnueabihf --with-cpu=cortex-a5 >

[arm] auto-generate arm-isa.h from CPU descriptions

2017-09-06 Thread Richard Earnshaw (lists)
This patch autogenerates arm-isa.h from new entries in arm-cpus.in. This has the primary advantage that it makes the description file more self-contained, but it also solves the 'array dimensioning' problem that Tamar recently encountered. It adds two new constructs to arm-cpus.in: features and

[Bug testsuite/82120] New: FAIL: gcc.dg/tree-ssa/pr81588.c

2017-09-06 Thread thopre01 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82120 Bug ID: 82120 Summary: FAIL: gcc.dg/tree-ssa/pr81588.c Product: gcc Version: 7.2.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: testsuite

[C++ PATCH] method vec

2017-09-06 Thread Nathan Sidwell
This preparatory patch fixes up a couple of places where a non-function could start appearing in the METHOD_VEC. The warn_hidden change looks bigger than necessary, because of indentation change. I noticed check_classfn could check a template mismatch earlier, and avoid doing some work.

[Ada] Wrong code on assignment of conditional expression to a mutable obkect

2017-09-06 Thread Arnaud Charlet
This patch fixes an error in an assignmen statement to an entity of a mutable type (variable or in-out parameter) when the righ-hand side of the assignment is a conditioal expression, some of whose alternatives are aggregates. Prior to this patch, not all components of the mutable object were

[C++ PATCH] class FIELD_VEC initialization

2017-09-06 Thread Nathan Sidwell
Here's some cleanup of the SORTED_FIELDS vector initialization. Some function renaming, to be more specific. The functionality change is a minor bug in late enums. We only add them to the field vec, if there's already a field vec. But of course, their addition could have cause the class's

  1   2   >