Getting Wales Moving 2019

2018-08-15 Thread Getting Wales Moving Conference
GETTING WALES MOVING DELIVERING ON THE WELL-BEING OF WALES 27TH FEBRUARY 2019 HILTON HOTEL - CARDIFF Following on from the hugely successful inaugural Getting Wales Moving Conference - Delivering on the Well-being for Future Generations Act, we are delighted to announce the second conference

Re: Why did Intel change his static branch prediction mechanism over these years?

2018-08-15 Thread 2016 quekong
Hi, Florian. >>>There is a movement to replace table-driven stack unwinding with a >>>conditional branch after most function calls... I am sorry I don't understand its meaning, forgive my poor knowledge of dynamic prediction, I am reading relative materials these days, could you explain a little

Re: Why did Intel change his static branch prediction mechanism over these years?

2018-08-15 Thread Florian Weimer
On 08/14/2018 03:36 PM, 2016 quekong wrote: And Intel seems don't want to talk about it any more, because the latest material I found within Intel Document was written about ten years ago. Since branch prediction is nowadays a security feature, I doubt you will see detailed public

Re: [GSOC] LTO dump tool project - feedback

2018-08-15 Thread Martin Liška
Hi. After last update of the branch, there's a feedback that will be needed before we can adept to have it merged into trunk: 1) there's patch for lto-dump proper install: diff --git a/gcc/lto/Make-lang.in b/gcc/lto/Make-lang.in index e474f85ebc6..e9d2659025c 100644 --- a/gcc/lto/Make-lang.in

Re: Why did Intel change his static branch prediction mechanism over these years?

2018-08-15 Thread Florian Weimer
On 08/15/2018 01:16 PM, 2016 quekong wrote: Hi, Florian. There is a movement to replace table-driven stack unwinding with a conditional branch after most function calls... I am sorry I don't understand its meaning, forgive my poor knowledge of dynamic prediction, I am reading relative

Re: Help with fixinclude fix for PR86599

2018-08-15 Thread David Edelsohn
You must manually insert the additional test lines into the appropriate file in fixincludes/tests/base and then retest. Please see step (6) in fixincludes/README Thanks, David On Wed, Aug 15, 2018 at 3:46 PM Albert Chin wrote: > Hi. I've come up with a fixincl fix for PR86599 but "make

Re: Help with fixinclude fix for PR86599

2018-08-15 Thread Albert Chin
On Wed, Aug 15, 2018 at 03:52:33PM -0400, David Edelsohn wrote: > You must manually insert the additional test lines into the appropriate > file in fixincludes/tests/base and then retest. > > Please see step (6) in fixincludes/README Ok, thanks. fixincludes/tests/res/stdlib.h has: #if defined(

gcc-6-20180815 is now available

2018-08-15 Thread gccadmin
Snapshot gcc-6-20180815 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/6-20180815/ 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: Help with fixinclude fix for PR86599

2018-08-15 Thread David Edelsohn
The change should not unintentionally affect existing tests. The new test should be exactly the text that "make check" produces, not what you think it should look like and not like another, neighboring test. If the test doesn't look as you expect, you need to examine your test text. You also can

Re: Trying to convert to LRA, running into an ICE (infinite reload loop)

2018-08-15 Thread Paul Koning
> On Aug 15, 2018, at 1:01 AM, Jeff Law wrote: > > On 08/14/2018 01:32 PM, Paul Koning wrote: >> I'm trying to convert the pdp11 target to use LRA. >> >> A lot of it "just works". But one of the test suite files fails in a way >> that I can't figure out at all. I'm hoping for some help or

Help with fixinclude fix for PR86599

2018-08-15 Thread Albert Chin
Hi. I've come up with a fixincl fix for PR86599 but "make check" in the fixincludes directory does not work and I do not know why. I know just enough fixincl-fu to produce a patch for this issue. I am trying to convert a chunk of code in stdlib on HP-UX 11.31/PA: # ifndef _LONG_DOUBLE #

[Bug target/86856] Format warnings building all-gcc

2018-08-15 Thread jon_y at users dot sourceforge.net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86856 --- Comment #10 from jon_y --- I noticed there are some casting from gcall to gimple, not being familiar with gcc internals, is it the correct way to suppress the warnings?

[Bug target/86777] Bfin port needs updating for CVE-2017-5753

2018-08-15 Thread rearnsha at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86777 --- Comment #2 from Richard Earnshaw --- I don't think you could do that through the API provided by this patch set; but it's not really appropriate for this case. I'm not familiar with the bfin architecture so cannot comment on what the best

[Bug target/84908] retpoline weirdness: 7.3.0-1 and 4.8.5-16: with -fPIC: R_X86_64_PC32 against undefined symbol `__x86_indirect_thunk_rax'

2018-08-15 Thread jason.vas.dias at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84908 --- Comment #14 from Jason Vas Dias --- RE: Comment #13: > You said that Andi Kleen had a comment. Can you point me to it? Here is a quote, from LKML message : Subject: Re: [PATCH v4.16-rc5 2/2] x86/vdso: \ VDSO should handle

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-15 Thread Martin Liška
On 07/26/2018 11:19 PM, Joseph Myers wrote: > On Thu, 19 Jul 2018, Martin Liška wrote: > >> I must admit that was my intention :) In my eyes it makes it more consistent >> and >> it gives consumers feedback about usage of an option that does nothing. >> For x86_64 there's list of options that

RE: [PATCH] arm: Generate correct const_ints (PR86640)

2018-08-15 Thread Tamar Christina
Hi All, I'd like to ask for permissions to backport of this patch to GCC 8? Thanks, Tamar > -Original Message- > From: Kyrill Tkachov > Sent: Tuesday, July 31, 2018 12:55 > To: Segher Boessenkool > Cc: gcc-patches@gcc.gnu.org; tnfch...@gcc.gnu.org > Subject: Re: [PATCH] arm: Generate

[PATCH v2] S/390: Remove literal pool chunkification loop

2018-08-15 Thread Ilya Leoshkevich
Since there is no branch splitting anymore, the loop is no longer necessary: pool chunkification can be done in one step. gcc/ChangeLog: 2018-08-13 Ilya Leoshkevich * config/s390/s390.c (s390_reorg): remove loop --- gcc/config/s390/s390.c | 68

[PATCH] Set start_location to 0 if we ran out of line map space

2018-08-15 Thread H.J. Lu
With profiledbootstrap and --with-build-config=bootstrap-lto, linemap_add may create a macro map when we run out of line map space. This patch changes start_location to UNKNOWN_LOCATION (0) in this case. Tested with profiledbootstrap and --with-build-config=bootstrap-lto on Linux/x86-64.

[Bug tree-optimization/86925] [9 Regression] ICE in get_predictor_value, at predict.c:2551

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86925 Martin Liška changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug tree-optimization/86925] [9 Regression] ICE in get_predictor_value, at predict.c:2551

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86925 --- Comment #4 from Martin Liška --- Author: marxin Date: Wed Aug 15 08:55:15 2018 New Revision: 263552 URL: https://gcc.gnu.org/viewcvs?rev=263552=gcc=rev Log: Fix merging of 2 predictors (PR tree-optimization/86925). 2018-08-15 Martin Liska

[Patch wwwdocs] Document arm-8-branch

2018-08-15 Thread Ramana Radhakrishnan
As $subject. Ok ? regards Ramana ? htdocs/svn.html.~1.223.~ Index: htdocs/svn.html === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v retrieving revision 1.223 diff -a -u -p -r1.223 svn.html --- htdocs/svn.html 18 Jul 2018

[Bug c++/70693] valgrind error in get_visual_column

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70693 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org --- Comment #7

[Bug lto/86412] lto1: internal compiler error: in lto_symtab_prevailing_virtual_decl, at lto/lto-symtab.c

2018-08-15 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86412 David Binderman changed: What|Removed |Added CC||dcb314 at hotmail dot com --- Comment

Re: [PATCH][GCC][AArch64] Limit movmem copies to TImode copies.

2018-08-15 Thread Tamar Christina
Hi All, I'm updating the patch with the suggested changes and also fixing a bug with a boundary condition. On AArch64 we have integer modes larger than TImode, and while we can generate moves for these they're not as efficient. So instead make sure we limit the maximum we can copy to TImode.

[PATCH][GCC][mid-end] Fix DSE big-endian subreg crash

2018-08-15 Thread Tamar Christina
Hi All, This patch fixes an ICE that would happen when extract_low_bits is called with modes for which you can't extract a valid subreg. e.g. taking a 32 bytes subreg from a 48 byte mode. The ICE happens because convert_modes which eventually calls simplify_gen_subreg does not expect the

[arm-8-branch] Backport spectre v1 patches

2018-08-15 Thread Ramana Radhakrishnan
Hi, Backport spectre mitigation patches into the arm-8-branch , these appeared to be clean backports back to the branch without any conflicts. Backported svn revisions 263169 263170 263171 263172 263173 263174 263175 263176 263180 263196 263197 263209 263253 263254 263255 263256 263258 263259

Re: [PATCH] arm: Generate correct const_ints (PR86640)

2018-08-15 Thread Kyrill Tkachov
On 15/08/18 09:21, Tamar Christina wrote: Hi All, I'd like to ask for permissions to backport of this patch to GCC 8? Ok. Please bundle the testcase in the backport as well. Thanks, Kyrill Thanks, Tamar -Original Message- From: Kyrill Tkachov Sent: Tuesday, July 31, 2018 12:55

[Bug target/81685] [7/8/9 Regression] FAIL: g++.dg/debug/dwarf2/inline-ns-2.C -std=gnu++* (internal compiler error) on darwin

2018-08-15 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81685 --- Comment #6 from Iain Sandoe --- Author: iains Date: Wed Aug 15 09:19:39 2018 New Revision: 263553 URL: https://gcc.gnu.org/viewcvs?rev=263553=gcc=rev Log: Update Darwin section names for DWARF5 gcc/ 2018-08-15 Iain Sandoe PR

[Bug fortran/83196] ICE in gfc_build_compare_string, at fortran/trans-expr.c:3609 (and others)

2018-08-15 Thread arjen.markus895 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83196 Arjen Markus changed: What|Removed |Added CC||arjen.markus895 at gmail dot com ---

Re: [PATCH] PR libstdc++/86846 Alternative to pointer-width atomics

2018-08-15 Thread Jonathan Wakely
On 15/08/18 10:40 +0100, Szabolcs Nagy wrote: On 14/08/18 14:13, Jonathan Wakely wrote: Define a class using std::mutex for when std::atomic cannot be used to implement the default memory resource. When std::mutex constructor is not constexpr the constant_init trick won't work, so just define

[Bug sanitizer/86962] [9 Regression] ICE in sanitize_rewrite_addressable_params, at sanopt.c:1173 with nested functions

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86962 Martin Liška changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug sanitizer/86962] [9 Regression] ICE in sanitize_rewrite_addressable_params, at sanopt.c:1173 with nested functions

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86962 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: [PATCH] PR libstdc++/86846 Alternative to pointer-width atomics

2018-08-15 Thread Szabolcs Nagy
On 14/08/18 14:13, Jonathan Wakely wrote: Define a class using std::mutex for when std::atomic cannot be used to implement the default memory resource. When std::mutex constructor is not constexpr the constant_init trick won't work, so just define a global and use init_priority for it. The

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-15 Thread Martin Liška
On 08/14/2018 06:02 PM, Martin Sebor wrote: > On 08/14/2018 03:06 AM, Martin Liška wrote: >> Hi. >> >> The patch adds more usages of the new macro. I hope it improves >> readability of code. > > I think it does :)  I see that most invocations of it in your > patch are with BUILT_IN_NORMAL as the

[Bug bootstrap/86872] [9 Regression] LTO bootstrap failed with profiledbootstrap

2018-08-15 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86872 --- Comment #5 from H.J. Lu --- A patch is posted at https://gcc.gnu.org/ml/gcc-patches/2018-08/msg00866.html

[Bug sanitizer/86962] New: [9 Regression] ICE in sanitize_rewrite_addressable_params, at sanopt.c:1173 with nested functions

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86962 Bug ID: 86962 Summary: [9 Regression] ICE in sanitize_rewrite_addressable_params, at sanopt.c:1173 with nested functions Product: gcc Version: 9.0

[Bug fortran/83196] ICE in gfc_build_compare_string, at fortran/trans-expr.c:3609 (and others)

2018-08-15 Thread arjen.markus895 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83196 --- Comment #5 from Arjen Markus --- (In reply to Arjen Markus from comment #4) > Created attachment 44543 [details] > Sample module exhibiting the problem > > ICE at line 11329 - any reduction seems to make the problem go away (but > also the

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-15 Thread James Greenhalgh
On Tue, Aug 14, 2018 at 09:34:08PM -0500, Martin Sebor wrote: > On 08/14/2018 09:24 AM, Martin Sebor wrote: > > On 08/14/2018 09:08 AM, Martin Sebor wrote: > >> On 08/14/2018 07:27 AM, James Greenhalgh wrote: > >>> On Wed, Aug 08, 2018 at 07:17:07PM -0500, Martin Sebor wrote: > On 08/08/2018

[Bug c/19315] document undocumented extension that allows code where variable is not emitted in the asm

2018-08-15 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19315 --- Comment #10 from Iain Sandoe --- Author: iains Date: Wed Aug 15 11:45:44 2018 New Revision: 263556 URL: https://gcc.gnu.org/viewcvs?rev=263556=gcc=rev Log: Don't make unsized objects into extern. 2018-08-15 Iain Sandoe gcc/c:

[Bug libstdc++/86963] std::tuple incorrectly assigned

2018-08-15 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86963 --- Comment #2 from Jonathan Wakely --- I think we want a similar solution as for PR 86751 i.e. r263185

[Bug libstdc++/86963] std::tuple incorrectly assigned

2018-08-15 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86963 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug fortran/80931] ICE on move_alloc in gimplify_expr, at gimplify.c:11335

2018-08-15 Thread arjen.markus895 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80931 Arjen Markus changed: What|Removed |Added CC||arjen.markus895 at gmail dot com ---

Re: [aarch64}: added variable issue rate feature for falkor

2018-08-15 Thread Kai Tietz
Hello Kyrill, thanks for your comments. 2018-08-14 16:50 GMT+02:00 Kyrill Tkachov : > Hi Kai, > > > On 13/08/18 17:48, Kai Tietz wrote: >> >> I repost updated patch containing ChangeLog entry. >> >> Regards, >> Kai > > > I think I understand what this patch does, please correct me if I'm wrong.

[Bug libstdc++/86963] New: std::tuple incorrectly assigned

2018-08-15 Thread denin at mail dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86963 Bug ID: 86963 Summary: std::tuple incorrectly assigned Product: gcc Version: 8.1.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: libstdc++

Re: [PATCH v2] S/390: Remove literal pool chunkification loop

2018-08-15 Thread Andreas Krebbel
On 08/15/2018 12:50 PM, Ilya Leoshkevich wrote: > gcc/ChangeLog: > > 2018-08-13 Ilya Leoshkevich > > * config/s390/s390.c (s390_reorg): remove loop Applied. Thanks! Andreas

[RFC][debug] Add -greadable-dwarf

2018-08-15 Thread Tom de Vries
Hi, This patch adds option -greadable-dwarf. In absence of an DW_AT_comment attribute, it sets the DW_AT_name attribute of dies that otherwise do not get that attribute, to make it easier to figure out what the die is describing. The option exports the names of artificial variables: ... DIE

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Bernd Edlinger
On 08/15/18 07:12, Jeff Law wrote: > On 08/14/2018 08:08 AM, Martin Sebor wrote: >> On 08/14/2018 04:25 AM, Bernd Edlinger wrote: >>> Hi, >>> >>> this patch is a follow-up to my patch here: >>> https://gcc.gnu.org/ml/gcc-patches/2018-07/msg01800.html >> >> As I said multiple times, this patch is

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-15 Thread Richard Biener
On August 15, 2018 12:28:55 PM GMT+02:00, James Greenhalgh wrote: >On Tue, Aug 14, 2018 at 09:34:08PM -0500, Martin Sebor wrote: >> On 08/14/2018 09:24 AM, Martin Sebor wrote: >> > On 08/14/2018 09:08 AM, Martin Sebor wrote: >> >> On 08/14/2018 07:27 AM, James Greenhalgh wrote: >> >>> On Wed,

Re: Stack alignment on Darwin (PR78444)

2018-08-15 Thread H.J. Lu
On Wed, Aug 15, 2018 at 8:41 AM, Iain Sandoe wrote: > Hi HJ, > > I am trying to track down a misalignment of the stack on Darwin (pr78444). > > In r163971 you applied this: > > --- gcc/config/i386/darwin.h(revision 163970) > +++ gcc/config/i386/darwin.h(revision 163971) > @@ -79,7 +79,9

[PATCH, testsuite] Fix PR78544 for Darwin.

2018-08-15 Thread Iain Sandoe
Hi, The fails on Darwin are because the section naming convention is different. The patch adds Darwin-specific section attributes and a corresponding target-specific scan-assembler clause. OK for trunk? affected branches (7, 8)? thanks Iain 2018-08-15 Iain Sandoe gcc/testsuite:

Re: [PATCH][GCC][Arm] Fix subreg crash in different way by enabling the FP16 pattern unconditionally.

2018-08-15 Thread Kyrill Tkachov
Hi Tamar, On 26/07/18 12:01, Tamar Christina wrote: Hi Thomas, > -Original Message- > From: Thomas Preudhomme > Sent: Thursday, July 26, 2018 09:29 > To: Tamar Christina > Cc: gcc-patches@gcc.gnu.org; nd ; Ramana Radhakrishnan > ; Richard Earnshaw > ; ni...@redhat.com; Kyrylo Tkachov

[Bug libgcc/86512] Incorrect sub result for float subnormal inputs in armv7(with -msoft-float).

2018-08-15 Thread wilco at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86512 Wilco changed: What|Removed |Added CC||wilco at gcc dot gnu.org --- Comment #3 from

Stack alignment on Darwin (PR78444)

2018-08-15 Thread Iain Sandoe
Hi HJ, I am trying to track down a misalignment of the stack on Darwin (pr78444). In r163971 you applied this: --- gcc/config/i386/darwin.h(revision 163970) +++ gcc/config/i386/darwin.h(revision 163971) @@ -79,7 +79,9 @@ Failure to ensure this will lead to a crash in the system

Re: [PATCH 0/6] improve handling of char arrays with missing nul (PR 86552, 86711, 86714)

2018-08-15 Thread Jeff Law
On 08/15/2018 08:47 AM, Martin Sebor wrote: > On 08/15/2018 12:02 AM, Jeff Law wrote: >> On 08/13/2018 03:23 PM, Martin Sebor wrote: >>> To make reviewing the changes easier I've split up the patch >>> into a series: >> [ ... ] >> I'm about done for the night and thus won't get into the series

[Bug gcov-profile/86957] gcc should warn about missing profiles for a compilation unit or a new function with -fprofile-use

2018-08-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86957 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[PATCH,testsuite, Darwin] Skip tests with pthread_barrier.

2018-08-15 Thread Iain Sandoe
Hi, Darwin is stated to be SUSv6 compliant and, at that revision, pthread_barrier is optional. It is not implemented on any version at least up to Darwin18. This skips the tests currently attempted which use pthread_barrier. OK for trunk? thanks Iain gcc/testsuite/ *

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Jeff Law
On 08/15/2018 08:03 AM, Bernd Edlinger wrote: > > But must admit, that I have sometimes the impression of being ignored, > for instance what I wrote in response to Martin's patch here: > https://gcc.gnu.org/ml/gcc-patches/2018-08/msg00184.html > ... and already earlier here: >

[PATCH, testsuite] Fix gcc.dg/memcmp-1.c for targets using __USER_LABEL_PREFIX__

2018-08-15 Thread Iain Sandoe
Hi, The test fails on Darwin (and presumably on other _U_L_P_ targets) because the asm-specified symbol isn’t found at link time. OK for trunk? thanks Iain gcc/testsuite: * gcc.dg/memcmp-1.c (lib_memcmp): Apply __USER_LABEL_PREFIX__. (lib_strncmp): Likewise. diff --git

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Jeff Law
On 08/14/2018 04:25 AM, Bernd Edlinger wrote: > Hi, > > this patch is a follow-up to my patch here: > https://gcc.gnu.org/ml/gcc-patches/2018-07/msg01800.html > > Since most calls of c_strlen and get_range_strlen expect > a string length in bytes of a zero-terminated string, there is > a need

[PATCH, testsuite] Stringify __USER_LABEL_PREFIX__ for pr81923.c

2018-08-15 Thread Iain Sandoe
Hi, This test has always failed on Darwin (and presumably the other targets with user __USER_LABEL_PREFIX__ != “” ) since it needs to be stringified. OK for trunk? affected branches? thanks Iain 2018-08-15 Iain Sandoe gcc/testsuite: * gcc.dg/asan/pr81923.c: Stringify

Re: [PATCH, testsuite] Stringify __USER_LABEL_PREFIX__ for pr81923.c

2018-08-15 Thread Richard Biener
On August 15, 2018 4:15:42 PM GMT+02:00, Iain Sandoe wrote: >Hi, > >This test has always failed on Darwin (and presumably the other targets >with user __USER_LABEL_PREFIX__ != “” ) since it needs to be >stringified. > >OK for trunk? OK. >affected branches? Yes. If you tested on x86_64-linux

[Bug tree-optimization/71625] missing strlen optimization on different array initialization style

2018-08-15 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71625 --- Comment #18 from Martin Sebor --- Author: msebor Date: Wed Aug 15 15:25:46 2018 New Revision: 263561 URL: https://gcc.gnu.org/viewcvs?rev=263561=gcc=rev Log: PR tree-optimization/71625 - missing strlen optimization on different array

[Bug rtl-optimization/85412] [8/9 Regression] ICE in put_TImodes, at sel-sched.c:7191

2018-08-15 Thread slyfox at inbox dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85412 Sergei Trofimovich changed: What|Removed |Added CC||slyfox at inbox dot ru --- Comment

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-15 Thread Martin Sebor
On 08/15/2018 04:28 AM, James Greenhalgh wrote: On Tue, Aug 14, 2018 at 09:34:08PM -0500, Martin Sebor wrote: On 08/14/2018 09:24 AM, Martin Sebor wrote: On 08/14/2018 09:08 AM, Martin Sebor wrote: On 08/14/2018 07:27 AM, James Greenhalgh wrote: On Wed, Aug 08, 2018 at 07:17:07PM -0500,

[Bug debug/86964] New: Too many debug symbols included, especially for extern globals

2018-08-15 Thread znerol at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86964 Bug ID: 86964 Summary: Too many debug symbols included, especially for extern globals Product: gcc Version: 6.3.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH 0/6] improve handling of char arrays with missing nul (PR 86552, 86711, 86714)

2018-08-15 Thread Martin Sebor
On 08/15/2018 12:02 AM, Jeff Law wrote: On 08/13/2018 03:23 PM, Martin Sebor wrote: To make reviewing the changes easier I've split up the patch into a series: [ ... ] I'm about done for the night and thus won't get into the series (and as you know Bernd has a competing patch in this space).

Re: [RFC PATCH, i386]: Deprecate -mmitigate-rop

2018-08-15 Thread Uros Bizjak
On Wed, Aug 15, 2018 at 5:56 AM, Jeff Law wrote: > On 08/10/2018 05:42 AM, Uros Bizjak wrote: >> This option is fairly ineffective, and in the light of CET, nobody >> seems interested to improve it. Deprecate the option, so it won't lure >> developers to the land of false security. >> >>

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-15 Thread Jeff Law
On 08/15/2018 03:02 PM, Martin Sebor wrote: > On 08/15/2018 06:07 AM, Joseph Myers wrote: >> On Tue, 14 Aug 2018, Martin Sebor wrote: >> This is with Bison 3.0.4, should the version used to produce intl/plural.c prove relevant. >>> >>> Can you send me the translation unit and the

[Bug debug/86941] ICE in i386/winnt.c:1258 in i386_pe_seh_unwind_emit

2018-08-15 Thread nightstrike at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86941 nightstrike changed: What|Removed |Added CC||uros at gcc dot gnu.org See

[Bug translation/79997] simple-ssa-sprintf i18n: wrong plural form in maybe_warn

2018-08-15 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79997 Martin Sebor changed: What|Removed |Added Status|NEW |RESOLVED Known to work|

[Bug c++/86966] ICE (Segmentation fault) for an explicit specialization of a member class template

2018-08-15 Thread v.reshetnikov at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86966 --- Comment #3 from Vladimir Reshetnikov --- Might be related to Bug 86960 and Bug 86918.

[Bug fortran/86945] BUG with optimisation of select case statement in gfortran v8.x

2018-08-15 Thread anlauf at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86945 --- Comment #4 from Harald Anlauf --- (In reply to Harald Anlauf from comment #3) > Self contained alternative testcase: > With -Og, -O1 and higher: > > id= 1 > ierr1, OK =0 T > ierr2, OK =1 F The magic

[Bug c++/86942] A trailing-return-type is allowed when the return type is not 'auto' for using declarations

2018-08-15 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86942 Marek Polacek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-15 Thread Martin Sebor
On 08/15/2018 06:07 AM, Joseph Myers wrote: On Tue, 14 Aug 2018, Martin Sebor wrote: This is with Bison 3.0.4, should the version used to produce intl/plural.c prove relevant. Can you send me the translation unit and the options it was compiled with that triggered the errors? I've attached

Re: C++ PATCH for c++/67012, diagnose decltype(auto) f() -> int;

2018-08-15 Thread Marek Polacek
On Wed, Aug 15, 2018 at 02:45:02PM -0400, Marek Polacek wrote: > I think that [dcl.fct] says that a function with trailing-return-type should > have plain auto as the return type, so this test should not compile. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? Looking at 86942, I think

[Bug rtl-optimization/85412] [8/9 Regression] ICE in put_TImodes, at sel-sched.c:7191

2018-08-15 Thread slyfox at inbox dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85412 Sergei Trofimovich changed: What|Removed |Added CC||abel at ispras dot ru --- Comment

[PATCH] Fix experimental::pmr typedefs and add tests

2018-08-15 Thread Jonathan Wakely
The typedefs in and don't need to be in the __cxx11 namespace, because they are only aliases and so will have the same mangled name as the underlying types. * include/experimental/regex: Remove begin/end macros for namespace. * include/experimental/string: Likewise. *

Re: [PATCH, OpenACC] Add support for gang local storage allocation in shared memory

2018-08-15 Thread Bernhard Reutner-Fischer
On 15 August 2018 18:46:37 CEST, Julian Brown wrote: >On Mon, 13 Aug 2018 12:06:21 -0700 >Cesar Philippidis wrote: atttribute has more t than strictly necessary. Don't like signed integer levels where they should be some unsigned. Also don't like single switch cases instead of if. And

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-15 Thread Jeff Law
On 08/15/2018 03:02 PM, Martin Sebor wrote: > On 08/15/2018 06:07 AM, Joseph Myers wrote: >> On Tue, 14 Aug 2018, Martin Sebor wrote: >> This is with Bison 3.0.4, should the version used to produce intl/plural.c prove relevant. >>> >>> Can you send me the translation unit and the

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Bernd Edlinger
On 08/15/18 18:25, Jeff Law wrote: > On 08/14/2018 04:25 AM, Bernd Edlinger wrote: >> Hi, >> >> this patch is a follow-up to my patch here: >> https://gcc.gnu.org/ml/gcc-patches/2018-07/msg01800.html >> >> Since most calls of c_strlen and get_range_strlen expect >> a string length in bytes of a

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Martin Sebor
Standard procedure in this kind of situation where we have two patches that are handling the same issue is to bake them off and choose one based on the technical merits. To that end... It would be helpful if you could compare/contrast your patch to Bernd's. ie, are there cases that will be

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-15 Thread Joseph Myers
On Wed, 15 Aug 2018, Martin Liška wrote: > Ok, so you have very similar opinion as Jakub. Thus I'm sending new > version that preserves status quo, it only does: This is removing RejectNegative from some Deprecated options. Won't that result in the -fno-* variants of those options starting to

[Bug c++/86966] ICE (Segmentation fault) for an explicit specialization of a member class template

2018-08-15 Thread v.reshetnikov at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86966 --- Comment #2 from Vladimir Reshetnikov --- I believe the code is valid. We explicitly specialize the member class template X of S for S<> (i.e. the parameter pack T is empty). T is expanded into a list of zero non-type template parameters of

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Jeff Law
Ignore this prior message from me I'm off-base here... On 08/15/2018 12:13 PM, Jeff Law wrote: > On 08/15/2018 10:38 AM, Bernd Edlinger wrote: >> On 08/15/18 18:25, Jeff Law wrote: >>> On 08/14/2018 04:25 AM, Bernd Edlinger wrote: Hi, this patch is a follow-up to my patch

[PATCH][GCC][DOCS] Remove rtl.texi references to old RTX code class names

2018-08-15 Thread Matthew Malcomson
Remove rtl.texi references to old RTX code class names The old names were changed in svn revision svn+ssh://gcc.gnu.org/svn/gcc/trunk@78824 138bc75d-0d04-0410-961f-82ee72b054a4 but the documentation was not fully updated at the time. Documentation checked after full build (both .info and .pdf)

[PATCH] CPP Macro predicates

2018-08-15 Thread Nathan Sidwell
I've been working on a cleanup of the preprocessor's macro machinery, this is the first patch of that series. 1) Introduce a set of inline predicates for detecting if an identifier is a macro of some kind. The implementation of those predicates will change, but the users will not need

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Jeff Law
On 08/15/2018 10:38 AM, Bernd Edlinger wrote: > On 08/15/18 18:25, Jeff Law wrote: >> On 08/14/2018 04:25 AM, Bernd Edlinger wrote: >>> Hi, >>> >>> this patch is a follow-up to my patch here: >>> https://gcc.gnu.org/ml/gcc-patches/2018-07/msg01800.html >>> >>> Since most calls of c_strlen and

Re: [PATCH] Add a character size parameter to c_strlen/get_range_strlen

2018-08-15 Thread Bernd Edlinger
On 08/15/18 20:13, Jeff Law wrote: > On 08/15/2018 10:38 AM, Bernd Edlinger wrote: >> On 08/15/18 18:25, Jeff Law wrote: >>> On 08/14/2018 04:25 AM, Bernd Edlinger wrote: Hi, this patch is a follow-up to my patch here: https://gcc.gnu.org/ml/gcc-patches/2018-07/msg01800.html

[Bug c++/86942] A trailing-return-type is allowed when the return type is not 'auto' for using declarations

2018-08-15 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86942 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment

Re: [PATCH, testsuite] Fix PR78544 for Darwin.

2018-08-15 Thread Iain Sandoe
> On 15 Aug 2018, at 20:07, Joseph Myers wrote: > > On Wed, 15 Aug 2018, Iain Sandoe wrote: > >> One of my Grand Ideas (i.e. things that might never happen because of >> time) would be to split out the Well Known Sections used by GCC into >> file/files with .defs for the different flavours.

[Bug testsuite/86519] [9 Regression] New test case gcc.dg/strcmpopt_6.c fails with its introduction in r262636

2018-08-15 Thread qinzhao at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86519 --- Comment #17 from qinzhao at gcc dot gnu.org --- the patch has been committed as: https://gcc.gnu.org/viewcvs/gcc?view=revision=263563

Re: [PATCH, testsuite] Fix gcc.dg/memcmp-1.c for targets using __USER_LABEL_PREFIX__

2018-08-15 Thread Jeff Law
On 08/15/2018 10:05 AM, Iain Sandoe wrote: > Hi, > > The test fails on Darwin (and presumably on other _U_L_P_ targets) because > the asm-specified symbol isn’t found at link time. > > OK for trunk? > > thanks > Iain > > gcc/testsuite: > > * gcc.dg/memcmp-1.c (lib_memcmp): Apply

lightweight class to wide int ranges in VRP and friends

2018-08-15 Thread Aldy Hernandez
I kept seeing the same patterns over and over in all this re-factoring: 1. extract value_range constants into pairs of wide ints 2. mimic symbolics as [-MIN, +MAX] (most of the time :)) 3. perform some wide int operation on the wide int range 4. convert back to a value range So I've decided

Re: [PATCH,testsuite, Darwin] Skip tests with pthread_barrier.

2018-08-15 Thread Mike Stump
On Aug 15, 2018, at 7:22 AM, Iain Sandoe wrote: > Darwin is stated to be SUSv6 compliant and, at that revision, pthread_barrier > is optional. > It is not implemented on any version at least up to Darwin18. > > This skips the tests currently attempted which use pthread_barrier. > > OK for

C++ PATCH for c++/67012, diagnose decltype(auto) f() -> int;

2018-08-15 Thread Marek Polacek
I think that [dcl.fct] says that a function with trailing-return-type should have plain auto as the return type, so this test should not compile. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2018-08-15 Marek Polacek PR c++/67012 * decl.c (grokdeclarator): Disallow

Re: [PATCH][Middle-end]patch for fixing PR 86519

2018-08-15 Thread Qing Zhao
> On Aug 14, 2018, at 11:25 PM, Jeff Law wrote: > > On 08/14/2018 08:57 AM, Qing Zhao wrote: >> Hi, >> >> PR 86519:New test case gcc.dg/strcmpopt_6.c fails with its introduction in >> r262636. >> >> gcc/ChangeLog: >> >> +2018-08-14 Qing Zhao >> + >> + PR testsuite/86519 >> +

[Bug c++/67012] decltype(auto) with trailing return type

2018-08-15 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67012 Marek Polacek changed: What|Removed |Added Status|NEW |ASSIGNED CC|

[Bug c++/86966] New: ICE (Segmentation fault) for an explicit specialization of a member class template

2018-08-15 Thread v.reshetnikov at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86966 Bug ID: 86966 Summary: ICE (Segmentation fault) for an explicit specialization of a member class template Product: gcc Version: 9.0 Status: UNCONFIRMED

Re: [aarch64}: added variable issue rate feature for falkor

2018-08-15 Thread Steve Ellcey
On Wed, 2018-08-15 at 12:45 +0200, Kai Tietz wrote: >  > > I believe the scheduler provides hooks specifically for storing > > backend-specific scheduling state so we should > > avoid creating such static variables in aarch64.c. Can you use the > > TARGET_SCHED_*_SCHED_CONTEXT family of hooks

Re: [PATCH, testsuite] Fix PR78544 for Darwin.

2018-08-15 Thread Iain Sandoe
> On 15 Aug 2018, at 19:39, Mike Stump wrote: > > On Aug 15, 2018, at 6:20 AM, Iain Sandoe wrote: >> >> The fails on Darwin are because the section naming convention is different. >> >> The patch adds Darwin-specific section attributes and a corresponding >> target-specific scan-assembler

  1   2   >