[Bug c++/81668] LTO ODR warnings are not helpful

2017-08-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81668 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment #4

[Bug c++/81718] New: g++ segfault when creating type alias

2017-08-04 Thread adrienstalain at hotmail dot fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81718 Bug ID: 81718 Summary: g++ segfault when creating type alias Product: gcc Version: 7.1.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++

Re: [PATCH 2/3] Matching tokens: C parts (v2)

2017-08-04 Thread David Malcolm
On Thu, 2017-08-03 at 11:34 -0600, Jeff Law wrote: > On 08/01/2017 02:21 PM, David Malcolm wrote: > > Changed in v2: > > > > * Renamed template argument to traits_t; eliminated subclasses, > > just > > using traits struct. > > * Moved enum constants into struct bodies (string constants can't >

Re: [Patch] Testsuite fixes for failures caused by patch for PR 80925 - loop peeling and alignment

2017-08-04 Thread Rainer Orth
Richard Biener writes: > On Fri, Jul 28, 2017 at 8:22 PM, Steve Ellcey wrote: >> On Fri, 2017-07-28 at 09:47 +0200, Richard Biener wrote: >>> On Fri, Jul 28, 2017 at 12:16 AM, Steve Ellcey wrote: >>> > >>> > Any comments from

Re: void function declared attribute const

2017-08-04 Thread Jonathan Wakely
On 02/08/17 15:41 -0600, Martin Sebor wrote: Hi Honza, While testing improvements to GCC attribute handling I came across the warning below: In file included from /ssd/src/gcc/81544/libstdc++-v3/src/c++98/mt_allocator.cc:31:0:

Re: [PATCH] PR c++/80287 add new testcase

2017-08-04 Thread Yvan Roux
On 13 July 2017 at 14:02, Yvan Roux wrote: > Hi, > > as discussed in the PR, this patch adds a new reduced testcase which > doesn't rely on c++17 features, this is a prereq to the backport of > the fix into GCC 6 branch which is impacted by this issue. > > Validated on x86,

Re: [RFC][PATCH] Do refactoring of attribute functions and move them to attribs.[hc].

2017-08-04 Thread Martin Liška
On 07/14/2017 09:23 AM, Jeff Law wrote: > On 07/13/2017 07:51 AM, Martin Liška wrote: >> Hi. >> >> It's request for comment where I mechanically moved attribute-related >> function to attribs.[hc]. >> >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >> >> Thoughts? >

Re: [Libgomp, Fortran] Fix canadian cross build

2017-08-04 Thread Yvan Roux
On 11 July 2017 at 12:25, Yvan Roux wrote: > On 3 July 2017 at 11:21, Yvan Roux wrote: >> On 23 June 2017 at 15:44, Yvan Roux wrote: >>> Hello, >>> >>> Fortran parts of libgomp (omp_lib.mod, openacc.mod, etc...) are >>> missing

Re: [PATCH][AArch64][GCC 6] PR target/79041: Correct -mpc-relative-literal-loads logic in aarch64_classify_symbol

2017-08-04 Thread Yvan Roux
On 11 July 2017 at 12:26, Yvan Roux wrote: > On 3 July 2017 at 12:48, Yvan Roux wrote: >> On 27 June 2017 at 13:14, Yvan Roux wrote: >>> Hi Wilco >>> >>> On 27 June 2017 at 12:53, Wilco Dijkstra wrote:

Re: gotools patch committed: Test runtime, misc/cgo/{test,testcarchive}

2017-08-04 Thread Ian Lance Taylor
On Fri, Aug 4, 2017 at 3:54 AM, Uros Bizjak wrote: > > The problem was following: > > runtime.Breakpoint comprises only call to __builtin_trap (), which in > case of alpha maps to "call_pall 0x81". Since __builtin_trap () is a > noreturn function, no other instructions are

Re: [PATCH v2][RFC] Canonize names of attributes.

2017-08-04 Thread Martin Liška
On 08/02/2017 01:25 PM, Joseph Myers wrote: > On Thu, 13 Jul 2017, Martin Liška wrote: > >> +/* For a given IDENTIFIER_NODE, strip leading and trailing '_' characters >> + so that we have a canonical form of attribute names. */ >> + >> +static inline tree >> +canonicalize_attr_name (tree

[Bug c++/81717] [c++ concepts] - simple "define function if not defined" code doesn't compile, no errors generated

2017-08-04 Thread jordok at wp dot pl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81717 --- Comment #1 from Przemysław Czechowski --- Created attachment 41926 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41926=edit Generated with g++ -save-temps ...

[Bug c++/81717] New: [c++ concepts] - simple "define function if not defined" code doesn't compile, no errors generated

2017-08-04 Thread jordok at wp dot pl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81717 Bug ID: 81717 Summary: [c++ concepts] - simple "define function if not defined" code doesn't compile, no errors generated Product: gcc Version: unknown Status:

[Bug c++/81716] Bogus -Wlto warning with forward-declared pointers

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81716 Richard Biener changed: What|Removed |Added Keywords||diagnostic CC|

Re: [PING^4][PATCH v2] Generate reproducible output independently of the build-path

2017-08-04 Thread Jakub Jelinek
On Fri, Aug 04, 2017 at 08:32:33AM -0400, Matthias Klose wrote: > >> GCC already supports a similar environment variable SOURCE_DATE_EPOCH, > >> which was accepted about 2 years ago in a patch written by one of our GSoC > >> students. We are not planning any more environment variables like this,

help with PR78809 - inline strcmp for small constant strings

2017-08-04 Thread Prathamesh Kulkarni
Hi, I was having a look at PR78809. For the test-case: int t1(const char *s) { return __builtin_strcmp (s, "a"); } for aarch64, trunk with -O2 generates: t1: adrpx1, .LANCHOR0 add x1, x1, :lo12:.LANCHOR0 b strcmp For i386, it seems strcmp is expanded inline

[Bug middle-end/81698] expand_case uses wrong edge as default edge

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81698 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: RFC: C extension to support variable-length vector types

2017-08-04 Thread Richard Biener
On Fri, Aug 4, 2017 at 12:46 PM, Alexander Monakov wrote: > On Thu, 3 Aug 2017, Richard Biener wrote: >> Btw., I did this once to represent constrained expressions on >> multi-dimensional arrays in SSA form. There control (aka loop) structure was >> also implicit. Google for

Re: broken link on this page https://gcc.gnu.org/gcc-7/changes.html for link to "Profile Mode" page:

2017-08-04 Thread Jonathan Wakely
On 2 August 2017 at 13:55, Sergei Kurenkov wrote: > Link on this page https://gcc.gnu.org/gcc-7/changes.html for "Profile Mode": > > * The libstdc++ Profile Mode has been deprecated and will be removed > in a future version. > > gives: > > Not Found > > The requested URL >

Re: [PATCH] Simplify pow with constant

2017-08-04 Thread Richard Biener
On Fri, Aug 4, 2017 at 2:26 PM, Alexander Monakov wrote: > On Fri, 4 Aug 2017, Wilco Dijkstra wrote: >> This patch simplifies pow (C, x) into exp (x * C1), where C1 = log (C). > > I don't think you can do that for non-positive C. Hmm, the question is also how this interacts

[PATCH][AArch64] Remove aarch64_frame_pointer_required

2017-08-04 Thread Wilco Dijkstra
To implement -fomit-leaf-frame-pointer, there are 2 places where we need to check whether we have to use a frame chain (since register allocation may allocate LR in a leaf function that omits the frame pointer, but if LR is spilled we must emit a frame chain). To simplify this do not force

Re: [PING^4][PATCH v2] Generate reproducible output independently of the build-path

2017-08-04 Thread Matthias Klose
On 03.08.2017 12:05, Jeff Law wrote: > On 08/02/2017 08:06 PM, Ximin Luo wrote: >> Jeff Law: >>> On 07/21/2017 10:15 AM, Ximin Luo wrote: (Please keep me on CC, I am not subscribed) Proposal This patch series adds a new environment variable

[Bug go/81617] mksigtab.sh fails to resolve NSIG with glibc 2.26

2017-08-04 Thread ian at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81617 --- Comment #5 from ian at gcc dot gnu.org --- Author: ian Date: Fri Aug 4 12:29:55 2017 New Revision: 250872 URL: https://gcc.gnu.org/viewcvs?rev=250872=gcc=rev Log: PR go/81617 libgo: change mksigtab to recognize glibc 2.26 NSIG

Re: libgo patch committed: Fix signal counting for glibc 2.26

2017-08-04 Thread Ian Lance Taylor
On Fri, Aug 4, 2017 at 12:22 AM, Richard Biener wrote: > On Thu, Aug 3, 2017 at 8:11 PM, Ian Lance Taylor wrote: >> This patch to libgo changes the mksigtab script to recognize the glibc >> 2.26 NSIG expression. Bootstrapped and ran Go testsuite on

Re: [PATCH] Simplify pow with constant

2017-08-04 Thread Alexander Monakov
On Fri, 4 Aug 2017, Wilco Dijkstra wrote: > This patch simplifies pow (C, x) into exp (x * C1), where C1 = log (C). I don't think you can do that for non-positive C. > Do this only for fast-math as accuracy is reduced. This is much faster > since pow is more complex than exp - with a current

[PATCH][AArch64] Introduce emit_frame_chain

2017-08-04 Thread Wilco Dijkstra
The current frame code combines the separate concepts of a frame chain (saving old FP,LR in a record and pointing new FP to it) and a frame pointer used to access locals. Add emit_frame_chain to the aarch64_frame descriptor and use it in the prolog and epilog code. For now just initialize it as

Re: [PATCH][1/2] Early LTO debug, simple-object part

2017-08-04 Thread Richard Biener
On Fri, 28 Jul 2017, Jason Merrill wrote: > On 07/28/2017 05:55 PM, Jason Merrill wrote: > > On Fri, Jul 28, 2017 at 8:54 AM, Richard Biener wrote: > > > On Tue, 4 Jul 2017, Richard Biener wrote: > > > > > > > On Tue, 20 Jun 2017, Richard Biener wrote: > > > > > > > > > On

[PATCH][AArch64] Use gen_frame_mem for callee-saves

2017-08-04 Thread Wilco Dijkstra
The frame code uses a mixture of gen_rtx_MEM and gen_frame_mem for callee-saves. Callee-saves never alias with local variables, so using gen_frame_mem is correct. Minor cleanup - no change in code. OK for commit? ChangeLog: 2017-08-04 Wilco Dijkstra gcc/ *

Re: [PATCH][2/2] early LTO debug, main part

2017-08-04 Thread Richard Biener
On Thu, 3 Aug 2017, Jason Merrill wrote: > On Thu, Aug 3, 2017 at 6:51 AM, Richard Biener wrote: > > On Wed, 2 Aug 2017, Jason Merrill wrote: > > > >> On Wed, Aug 2, 2017 at 6:35 AM, Richard Biener wrote: > >> > On Wed, 2 Aug 2017, Jason Merrill wrote: > >>

[patch, gcc doc + fortran] Make -Ofast honor -fmax-stack-var-size

2017-08-04 Thread Thomas Koenig
Hello world, the attached patch makes -Ofast honor -fmax-stack-var-size, and adjusts the documentation in the gcc and fortran directories accordingly. This is done to alleviate PR 68829, to make it possible to run -Ofast with less stack usage. I have also taken the opportunity to make it a bit

[Bug c/81687] Compiler drops label in OpenMP region

2017-08-04 Thread protze at itc dot rwth-aachen.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81687 --- Comment #5 from Joachim Protze --- Jakub, thank you for the quick solution! I successfully applied your patch to the sources from 7.1 release tarball. The two test cases you added are built and linked successfully. I can also successfully

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

2017-08-04 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81318 --- Comment #17 from David Binderman --- (In reply to Bill Schmidt from comment #16) > Check out the code from http://gcc.gnu.org/svn/gcc/branches/gcc-7-branch to > see if the problem exists there. From what I can see from this discussion > it

[Bug middle-end/81695] [5/6/7 Regression] internal compiler error: in size_binop_loc, at fold-const.c:1768

2017-08-04 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81695 Marek Polacek changed: What|Removed |Added Summary|[5/6/7/8 Regression]|[5/6/7 Regression] internal

[Bug middle-end/81695] [5/6/7/8 Regression] internal compiler error: in size_binop_loc, at fold-const.c:1768

2017-08-04 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81695 --- Comment #7 from Marek Polacek --- Author: mpolacek Date: Fri Aug 4 11:28:04 2017 New Revision: 250871 URL: https://gcc.gnu.org/viewcvs?rev=250871=gcc=rev Log: PR middle-end/81695 * fold-const.c (fold_indirect_ref_1): For

[PATCH] Simplify pow with constant

2017-08-04 Thread Wilco Dijkstra
This patch simplifies pow (C, x) into exp (x * C1), where C1 = log (C). Do this only for fast-math as accuracy is reduced. This is much faster since pow is more complex than exp - with a current GLIBC the speedup is more than 7 times for this transformation. ChangeLog: 2017-08-04 Wilco Dijkstra

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #7 from rguenther at suse dot de --- On Fri, 4 Aug 2017, jakub at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 > > --- Comment #6 from Jakub Jelinek --- > (In reply to rguent...@suse.de from comment

Re: [PATCH][RFC] Make expansion of balanced binary trees of switches on tree level.

2017-08-04 Thread Martin Liška
On 08/03/2017 03:27 PM, Steven Bosscher wrote: > On Thu, Aug 3, 2017 at 2:56 PM, Richard Biener wrote: >> I think the main reason for not doing it early is the benefit is small >> (unless it is GIMPLE optimizations triggering) > > Agree. > >> and we can't get rid of >> switches completely

Re: [PATCH][RFC] Make expansion of balanced binary trees of switches on tree level.

2017-08-04 Thread Martin Liška
On 08/03/2017 02:52 PM, Steven Bosscher wrote: > On Wed, Aug 2, 2017 at 1:20 PM, Martin Liška wrote: >> Hello. >> >> After some discussions with Honza, I've decided to convert current code in >> stmt.c that >> is responsible for switch expansion. More precisely, I would like to convert >> the

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #6 from Jakub Jelinek --- (In reply to rguent...@suse.de from comment #5) > True, but it removes the ability to avoid the vectorized variant with > using the __builtin_ variant ;) Do we ever want to generate less efficient code when

[Bug c++/81716] New: Bogus -Wlto warning with forward-declared pointers

2017-08-04 Thread sgunderson at bigfoot dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81716 Bug ID: 81716 Summary: Bogus -Wlto warning with forward-declared pointers Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #5 from rguenther at suse dot de --- On Fri, 4 Aug 2017, jakub at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 > > --- Comment #4 from Jakub Jelinek --- > The C/C++ FE change would be something like:

Re: [GCC][PATCH][mid-end][Version 3] Optimize x * copysign (1.0, y) [Patch (1/2)]

2017-08-04 Thread Richard Biener
On Thu, 3 Aug 2017, Tamar Christina wrote: > Hi All, > > this patch implements a optimization rewriting > > x * copysign (1.0, y) > > to: > > x ^ (y & (1 << sign_bit_position)) > > > This is only done when not honoring signaling NaNs. > This transormation is done at ssa mult widening time

Re: gotools patch committed: Test runtime, misc/cgo/{test,testcarchive}

2017-08-04 Thread Uros Bizjak
On Wed, Jul 26, 2017 at 10:26 PM, Ian Lance Taylor wrote: > On Sat, Jul 22, 2017 at 11:08 AM, Uros Bizjak wrote: >>> This patch to the gotools Makefile adds tests to `make check`. We now >>> test the runtime package using the newly built go tool, and test

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #4 from Jakub Jelinek --- The C/C++ FE change would be something like: --- gcc/tree.c.jj 2017-07-29 09:48:40.0 +0200 +++ gcc/tree.c 2017-08-04 12:06:35.636072718 +0200 @@ -5022,8 +5022,8 @@ attribute_value_equal

Re: RFC: C extension to support variable-length vector types

2017-08-04 Thread Alexander Monakov
On Thu, 3 Aug 2017, Richard Biener wrote: > Btw., I did this once to represent constrained expressions on > multi-dimensional arrays in SSA form. There control (aka loop) structure was > also implicit. Google for 'middle-end array expressions'. The C interface > was builtins and VLAs. The

[Bug tree-optimization/81136] [8 Regression] ICE: in vect_update_misalignment_for_peel, at tree-vect-data-refs.c:910

2017-08-04 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81136 --- Comment #4 from rsandifo at gcc dot gnu.org --- Author: rsandifo Date: Fri Aug 4 10:42:53 2017 New Revision: 250870 URL: https://gcc.gnu.org/viewcvs?rev=250870=gcc=rev Log: Pool alignment information for common bases This patch is a

[Bug hsa/81713] BIT_FIELD_REF produced by Brig FE do not pass new verification

2017-08-04 Thread pekka.jaaskelainen at parmance dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81713 --- Comment #2 from Pekka Jääskeläinen --- https://github.com/linehill/gccbrig/commit/feab8a56be8cbe3b95f4dd121e7db4306f75655e.patch I will commit this a bit later unless you wish to fix it sooner.

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #2 from Jakub Jelinek --- Created attachment 41923 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41923=edit gcc8-pr81706.patch The reason for why cmath does this is that C++ wants to have sin overloads. And the problem is

[Bug middle-end/81705] [8 Regression] UBSAN: yet another false positive

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 --- Comment #6 from Richard Biener --- Author: rguenth Date: Fri Aug 4 10:33:39 2017 New Revision: 250866 URL: https://gcc.gnu.org/viewcvs?rev=250866=gcc=rev Log: 2017-08-04 Richard Biener PR middle-end/81705

[Bug middle-end/81705] [8 Regression] UBSAN: yet another false positive

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 Richard Biener changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[PATCH] Fix PR81705

2017-08-04 Thread Richard Biener
The following fixes PR81705, a simple omission in my last association patch. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2017-08-04 Richard Biener PR middle-end/81705 * fold-const.c (fold_binary_loc): Properly restrict

[Bug sanitizer/81715] New: asan-stack=1 redzone allocation is too inflexible

2017-08-04 Thread arnd at linaro dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81715 Bug ID: 81715 Summary: asan-stack=1 redzone allocation is too inflexible Product: gcc Version: 7.1.1 Status: UNCONFIRMED Severity: normal Priority: P3

RE: [PATCH] i386: Rewrite check for AVX512 features

2017-08-04 Thread Peryt, Sebastian
> -Original Message- > From: Uros Bizjak [mailto:ubiz...@gmail.com] > Sent: Sunday, July 30, 2017 11:02 AM > To: H.J. Lu > Cc: gcc-patches@gcc.gnu.org; Koval, Julia ; Peryt, > Sebastian > Subject: Re: [PATCH] i386:

Re: [WIP] Possible Bug in vect_bb_slp_scalar_cost?

2017-08-04 Thread Richard Biener
On Fri, Aug 4, 2017 at 12:08 PM, Dominik Inführ wrote: > Hi, > > vect_bb_slp_scalar_cost computes the scalar cost of a SLP node. If there are > non-scalar uses of a definition, the costs for it and its operands (children) > are ignored. The vector LIFE is

Re: [PATCH][GCC] Simplification of 1U << (31 - x)

2017-08-04 Thread Richard Biener
On Tue, Aug 1, 2017 at 11:14 AM, Sudi Das wrote: > > > > > Sorry about the delayed response but looking at the above discussion, should > I conclude that this is a valid tree simplification? Yes, I think so. Jakub requested code to undo this at RTL expansion based on target

[Bug c++/81668] LTO ODR warnings are not helpful

2017-08-04 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81668 Markus Trippelsdorf changed: What|Removed |Added CC||trippels at gcc dot gnu.org ---

Re: [PATCH] Verify edge probability consistency in verify_flow_info

2017-08-04 Thread Tom de Vries
On 08/04/2017 11:15 AM, Jan Hubicka wrote: OK for trunk if bootstrap and reg-test on x86_64 succeeds? Yea, but I'd like to see ongoing work towards full checking. I have full checking in my tree for some time. At x86-64 bootstrap there is one remaining offender in simd_clone_adjust which was

[WIP] Possible Bug in vect_bb_slp_scalar_cost?

2017-08-04 Thread Dominik Inführ
Hi, vect_bb_slp_scalar_cost computes the scalar cost of a SLP node. If there are non-scalar uses of a definition, the costs for it and its operands (children) are ignored. The vector LIFE is used to keep track of this and an element is set to true, such that the def and its children are

Re: Overwhelmed by GCC frustration

2017-08-04 Thread Richard Earnshaw
On 04/08/17 10:38, Claudiu Zissulescu wrote: > Maybe better is to use the updated CsIbe repo from github > https://github.com/szeged/csibe. I use it for ARC to track the code > size. > Thanks for the link Claudiu. Personally I'll probably stick with the existing code as I now have size data for

[Bug c++/81668] LTO ODR warnings are not helpful

2017-08-04 Thread sgunderson at bigfoot dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81668 --- Comment #2 from sgunderson at bigfoot dot com --- Running with -fno-diagnostics-show-caret does not help any: ../include/violite.h:288:8: warning: type ‘struct st_vio’ violates the C++ One Definition Rule [-Wodr] ../include/violite.h:288:0:

Re: [PATCH] Fold (int *) + 4 to a[1] using offset_int (PR middle-end/81695)

2017-08-04 Thread Marek Polacek
On Fri, Aug 04, 2017 at 11:08:49AM +0200, Richard Biener wrote: > On Fri, Aug 4, 2017 at 10:38 AM, Marek Polacek wrote: > > We were crashing because size_binop_loc got operands of different types: > > sizetype and ssizetype. Richi suggested performing the computation in > >

[C++ Patch Ping] Re: [C++ Patch] PR 79790 ("[C++17] ICE class template argument deduction")

2017-08-04 Thread Paolo Carlini
Hi, On 14/07/2017 19:51, Nathan Sidwell wrote: On 07/14/2017 01:32 PM, Paolo Carlini wrote: While working on the bug I also noticed that we can simplify a bit the code generating the implicit deduction guides: if I'm not mistaken, when we pass types as first argument of build_deduction_guide

Re: Overwhelmed by GCC frustration

2017-08-04 Thread Claudiu Zissulescu
Maybe better is to use the updated CsIbe repo from github https://github.com/szeged/csibe. I use it for ARC to track the code size. Cheers, Claudiu On Fri, Aug 4, 2017 at 11:12 AM, Richard Earnshaw wrote: > On 03/08/17 13:11, Steven Bosscher wrote: >> On Mon, Jul

Re: Handle data dependence relations with different bases

2017-08-04 Thread Richard Biener
On Fri, Aug 4, 2017 at 11:28 AM, Richard Sandiford wrote: > Richard Biener writes: >> On Thu, Jul 27, 2017 at 2:19 PM, Richard Sandiford >> wrote: >>> Richard Sandiford writes:

[Bug c++/81702] [7/8 Regression] ICE in gimple_get_virt_method_for_vtable

2017-08-04 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81702 Martin Jambor changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: Handle data dependence relations with different bases

2017-08-04 Thread Richard Sandiford
Richard Biener writes: > On Thu, Jul 27, 2017 at 2:19 PM, Richard Sandiford > wrote: >> Richard Sandiford writes: >>> Eric Botcazou writes: [Sorry for missing the previous

Re: [PATCH] Verify edge probability consistency in verify_flow_info

2017-08-04 Thread Jan Hubicka
> > > > III. > > > > I've written this patch to check for the missing probability more > > consistently. I'm not certain if we can require that the probability > > should always be set, so I'm just requiring that if it is set on one > > outgoing edge, it needs to be set on all outgoing edges. >

Re: Overwhelmed by GCC frustration

2017-08-04 Thread Richard Earnshaw
On 03/08/17 13:11, Steven Bosscher wrote: > On Mon, Jul 31, 2017 at 6:49 PM, Joel Sherrill wrote: > >> >> Long ago, there was a code size regression tester for at least >> ARM. Is that still around? > > There used to be autotesters from CSiBE. Something still appears to > exist

Re: [PATCHv5][PR 57371] Remove useless floating point casts in comparisons

2017-08-04 Thread Richard Biener
On Sun, Jul 30, 2017 at 9:25 AM, Yury Gribov wrote: > Hi all, > > This is an updated version of patch in > https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00924.html . It prevents > optimization in presense of sNaNs (and qNaNs when comparison operator is > >>= < <=) to

Re: [PATCH] Fold (int *) + 4 to a[1] using offset_int (PR middle-end/81695)

2017-08-04 Thread Richard Biener
On Fri, Aug 4, 2017 at 10:38 AM, Marek Polacek wrote: > We were crashing because size_binop_loc got operands of different types: > sizetype and ssizetype. Richi suggested performing the computation in > offset_int which my patch tries to do. Unsure about the sdiv_trunc part,

Re: Handle data dependence relations with different bases

2017-08-04 Thread Richard Biener
On Thu, Jul 27, 2017 at 2:19 PM, Richard Sandiford wrote: > Richard Sandiford writes: >> Eric Botcazou writes: >>> [Sorry for missing the previous messages] >>> Thanks. Just been retesting, and I think I

[Bug c++/81714] New: incorrect location for uninitialised variable

2017-08-04 Thread akim.demaille at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81714 Bug ID: 81714 Summary: incorrect location for uninitialised variable Product: gcc Version: 7.0.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug hsa/81713] BIT_FIELD_REF produced by Brig FE do not pass new verification

2017-08-04 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81713 Martin Jambor changed: What|Removed |Added CC||pekka.jaaskelainen@parmance

Re: RFC: C extension to support variable-length vector types

2017-08-04 Thread Richard Biener
On Fri, Aug 4, 2017 at 9:42 AM, Richard Sandiford wrote: > Richard Biener writes: >> On August 3, 2017 7:05:05 PM GMT+02:00, Richard Sandiford >> wrote: >>>Torvald Riegel writes: On Wed,

[Bug hsa/81713] New: BIT_FIELD_REF produced by Brig FE do not pass new verification

2017-08-04 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81713 Bug ID: 81713 Summary: BIT_FIELD_REF produced by Brig FE do not pass new verification Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal

[Bug sanitizer/81697] Incorrect ASan global variables alignment on arm

2017-08-04 Thread m.ostapenko at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81697 Maxim Ostapenko changed: What|Removed |Added CC||m.ostapenko at samsung dot com ---

[Bug bootstrap/81710] build fail with glibc 2.26 due to removing ucontext/sigaltstack struct tags

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81710 Petr Ovtchenkov changed: What|Removed |Added Attachment #41921|0 |1 is patch|

Re: [PING^4][PATCH v2] Generate reproducible output independently of the build-path

2017-08-04 Thread Yury Gribov
On Thu, Aug 3, 2017 at 12:45 PM, Ximin Luo wrote: > Yury Gribov: >> On 03.08.2017 3:06, Ximin Luo wrote: >>> Jeff Law: On 07/21/2017 10:15 AM, Ximin Luo wrote: > (Please keep me on CC, I am not subscribed) > > > Proposal > > > This

[PATCH] Fold (int *) + 4 to a[1] using offset_int (PR middle-end/81695)

2017-08-04 Thread Marek Polacek
We were crashing because size_binop_loc got operands of different types: sizetype and ssizetype. Richi suggested performing the computation in offset_int which my patch tries to do. Unsure about the sdiv_trunc part, what do I use instead of EXACT_DIV_EXPR? Bootstrapped/regtested on

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 --- Comment #2 from Jakub Jelinek --- Created attachment 41923 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41923=edit gcc8-pr81706.patch The reason for why cmath does this is that C++ wants to have sin overloads. And the problem is

[Bug middle-end/81705] [8 Regression] UBSAN: yet another false positive

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 --- Comment #4 from Richard Biener --- Oops. Simple mistake in my recent patch. Index: gcc/fold-const.c === --- gcc/fold-const.c(revision 250865) +++ gcc/fold-const.c

[Bug bootstrap/81710] build fail with glibc 2.26 due to removing ucontext/sigaltstack struct tags

2017-08-04 Thread abominable-snowman at yandex dot ru
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81710 Petr Ovtchenkov changed: What|Removed |Added CC||jakub at gcc dot gnu.org,

[Bug middle-end/81705] [8 Regression] UBSAN: yet another false positive

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: How to migrate POINTER_TYPE_OVERFLOW_UNDEFINED for GCC v8.x?

2017-08-04 Thread Leslie Zhai
Hi Bin, Thanks for your kind response! 在 2017年08月04日 15:45, Bin.Cheng 写道: On Fri, Aug 4, 2017 at 8:00 AM, Leslie Zhai wrote: Hi GCC developers, As ChangeLog mentioned: 2017-08-01 Bin Cheng * tree.h (POINTER_TYPE_OVERFLOW_UNDEFINED):

[Bug middle-end/81705] [8 Regression] UBSAN: yet another false positive

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 Richard Biener changed: What|Removed |Added Keywords||wrong-code Priority|P3

[Bug bootstrap/81710] build fail with glibc 2.26 due to removing ucontext/sigaltstack struct tags

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81710 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Known to work|

[Bug other/81712] gcc does not compile when using glib 2.26 (everything works fine with 2.25)

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81712 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Known to work|

[Bug libstdc++/81706] std::sin vectorization bug

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81706 Richard Biener changed: What|Removed |Added Keywords||missed-optimization

[Bug tree-optimization/81704] strncpy folding defeats strlen optimization

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81704 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug tree-optimization/81703] memcpy folding defeats strlen optimization

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81703 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug other/81712] gcc does not compile when using glib 2.26 (everything works fine with 2.25)

2017-08-04 Thread igor at chub dot in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81712 --- Comment #3 from Igor Chubin --- Perfect. I'll try to build gcc-7.2-RC-20170802

[Bug c++/81702] [7/8 Regression] ICE in gimple_get_virt_method_for_vtable

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81702 Richard Biener changed: What|Removed |Added Keywords||ice-on-valid-code

[Bug fortran/81701] -fstack-arrays hehavior does not match documentation

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81701 Richard Biener changed: What|Removed |Added Keywords||documentation --- Comment #1 from

[Bug middle-end/81698] expand_case uses wrong edge as default edge

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81698 Richard Biener changed: What|Removed |Added Keywords||wrong-code

[Bug testsuite/81699] [8 regression] gcc.dg/tree-ssa/reassoc-23.c fails starting with r250853

2017-08-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81699 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

Re: How to migrate POINTER_TYPE_OVERFLOW_UNDEFINED for GCC v8.x?

2017-08-04 Thread Bin.Cheng
On Fri, Aug 4, 2017 at 8:00 AM, Leslie Zhai wrote: > Hi GCC developers, > > As ChangeLog mentioned: > > 2017-08-01 Bin Cheng > > * tree.h (POINTER_TYPE_OVERFLOW_UNDEFINED): Delete. > > > Then how to migrate it for GCC v8.x? for example: > >

Re: RFC: C extension to support variable-length vector types

2017-08-04 Thread Richard Sandiford
Richard Biener writes: > On August 3, 2017 7:05:05 PM GMT+02:00, Richard Sandiford > wrote: >>Torvald Riegel writes: >>> On Wed, 2017-08-02 at 17:59 +0100, Richard Sandiford wrote: Torvald Riegel

[Bug sanitizer/81066] sanitizer_stoptheworld_linux_libcdep.cc:276:22: error: aggregate ‘sigaltstack handler_stack’ has incomplete type and cannot be defined

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81066 Jakub Jelinek changed: What|Removed |Added CC||abominable-snowman at yandex dot r

[Bug sanitizer/81711] __res_state is a struct and a function in glibc 2.26

2017-08-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81711 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug tree-optimization/81705] UBSAN: yet another false positive

2017-08-04 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81705 --- Comment #2 from Marek Polacek --- I don't see this with g++-7 which has (void) (a = (-516151698 - NON_LVALUE_EXPR ) - -NON_LVALUE_EXPR ) trunks produces (void) (a = -516151698 - (-NON_LVALUE_EXPR + NON_LVALUE_EXPR ))

<    1   2   3   >