Re: [PING] [PATCH] Avoid excessive function type casts with splay-trees

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 4:04 PM Jakub Jelinek wrote: > On Thu, May 17, 2018 at 03:39:42PM +0200, Richard Biener wrote: > > On Thu, May 17, 2018 at 3:21 PM Bernd Edlinger < bernd.edlin...@hotmail.de> > > wrote: > > > > > Ping... > > > > So this makes all traditional users go

Re: [PATCH GCC][4/6]Support regional coalesce and live range computation

2018-05-17 Thread Richard Biener
On Tue, May 15, 2018 at 5:44 PM Bin.Cheng wrote: > On Fri, May 11, 2018 at 1:53 PM, Richard Biener > wrote: > > On Fri, May 4, 2018 at 6:23 PM, Bin Cheng wrote: > >> Hi, > >> Following Jeff's suggestion, I am now using

Re: [PING] [PATCH] Avoid excessive function type casts with splay-trees

2018-05-17 Thread Jakub Jelinek
On Thu, May 17, 2018 at 03:39:42PM +0200, Richard Biener wrote: > On Thu, May 17, 2018 at 3:21 PM Bernd Edlinger > wrote: > > > Ping... > > So this makes all traditional users go through the indirect > splay_tree_compare_wrapper > and friends (which is also exported

[Bug target/85819] New: conversion from __v[48]su to __v[48]sf should use FMA

2018-05-17 Thread kretz at kde dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85819 Bug ID: 85819 Summary: conversion from __v[48]su to __v[48]sf should use FMA Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal Priority: P3

Re: [PATCH][AArch64] Unify vec_set patterns, support floating-point vector modes properly

2018-05-17 Thread Kyrill Tkachov
On 17/05/18 09:46, Kyrill Tkachov wrote: On 15/05/18 18:56, Richard Sandiford wrote: Kyrill Tkachov writes: Hi all, We've a deficiency in our vec_set family of patterns. We don't support directly loading a vector lane using LD1 for V2DImode and all the vector

[Bug fortran/85599] Prevent short-circuiting of logical expressions for non-pure functions

2018-05-17 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85599 --- Comment #29 from janus at gcc dot gnu.org --- (In reply to Thomas Koenig from comment #27) > > If an impure function is found (recursively) in the operands of an .AND. > > expression, issue a > > > > gfc_warning(OPT_Wsurprising, "Impure

[Bug target/81652] [meta-bug] -fcf-protection=full bugs

2018-05-17 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81652 Bug 81652 depends on bug 85620, which changed state. Bug 85620 Summary: Missing ENDBR after swapcontext https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85620 What|Removed |Added

[Bug middle-end/85620] Missing ENDBR after swapcontext

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

[Bug middle-end/85620] Missing ENDBR after swapcontext

2018-05-17 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85620 H.J. Lu changed: What|Removed |Added Resolution|FIXED |WONTFIX

Re: RFA (ipa-prop): PATCHes to avoid use of deprecated copy ctor and op=

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 3:25 PM Andreas Schwab wrote: > On Mai 17 2018, Jason Merrill wrote: > > On Thu, May 17, 2018 at 4:14 AM, Andreas Schwab wrote: > >> On Mai 16 2018, Andreas Schwab wrote: > >>> On Mai 15 2018, Jason

Re: [PING] [PATCH] Avoid excessive function type casts with splay-trees

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 3:21 PM Bernd Edlinger wrote: > Ping... So this makes all traditional users go through the indirect splay_tree_compare_wrapper and friends (which is also exported for no good reason?). And all users are traditional at the moment. So I wonder

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 1:42 PM Marc Glisse wrote: > On Thu, 17 May 2018, Richard Biener wrote: > > On Thu, May 17, 2018 at 1:14 PM Marc Glisse wrote: > > > >> On Thu, 17 May 2018, Jonathan Wakely wrote: > > > >>> On 17/05/18 12:54 +0200, Marc Glisse

Re: RFA (ipa-prop): PATCHes to avoid use of deprecated copy ctor and op=

2018-05-17 Thread Andreas Schwab
On Mai 17 2018, Jason Merrill wrote: > On Thu, May 17, 2018 at 4:14 AM, Andreas Schwab wrote: >> On Mai 16 2018, Andreas Schwab wrote: >>> On Mai 15 2018, Jason Merrill wrote: >>> commit

Re: [PATCH] Support lower and upper limit for -fdbg-cnt flag.

2018-05-17 Thread Richard Biener
On Wed, May 16, 2018 at 3:54 PM Martin Liška wrote: > On 05/16/2018 03:39 PM, Alexander Monakov wrote: > > On Wed, 16 May 2018, Martin Liška wrote: > >>> Hm, is the off-by-one in the new explanatory text really intended? I think > >>> the previous text was accurate, and the new

[PING] [PATCH] Avoid excessive function type casts with splay-trees

2018-05-17 Thread Bernd Edlinger
Ping... On 05/03/18 22:13, Bernd Edlinger wrote: > Hi, > > this is basically the same patch I posted a few months ago, > with a few formatting nits by Jakub fixed. > > Bootstrapped and reg-tested again with current trunk. > > Is it OK for trunk? > > > Bernd. > > On 12/15/17 11:44, Bernd

[Bug ada/85760] Using generic function to initialize a type created by a generic package crashes GNAT

2018-05-17 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85760 --- Comment #9 from Eric Botcazou --- > OH! I thought you meant here. I wasn't sure this was the same bug > internally. They have different results in the two different versions of > GNAT. The AdaCore GNAT Community put out a crash report

[Bug fortran/85599] Prevent short-circuiting of logical expressions for non-pure functions

2018-05-17 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85599 Dominique d'Humieres changed: What|Removed |Added Keywords|wrong-code | Priority|P3

Re: RFA (ipa-prop): PATCHes to avoid use of deprecated copy ctor and op=

2018-05-17 Thread Jason Merrill
On Thu, May 17, 2018 at 4:14 AM, Andreas Schwab wrote: > On Mai 16 2018, Andreas Schwab wrote: >> On Mai 15 2018, Jason Merrill wrote: >> >>> commit 648ffd02e23ac2695de04ab266b4f8862df6c2ed >>> Author: Jason Merrill >>> Date:

Re: Use conditional internal functions in if-conversion

2018-05-17 Thread Richard Biener
On Wed, May 16, 2018 at 12:17 PM Richard Sandiford < richard.sandif...@linaro.org> wrote: > This patch uses IFN_COND_* to vectorise conditionally-executed, > potentially-trapping arithmetic, such as most floating-point > ops with -ftrapping-math. E.g.: > if (cond) { ... x = a + b; ... } >

So what's the status of the Git migration?

2018-05-17 Thread Richard Earnshaw (lists)
Another year; another release; and still no sign of progress on the git migration. Any ideas on how much longer this is going to take? R.

[Bug ipa/85655] [8/9 Regression] ICE with -flto and -O2 during IPA pass: cp lto1: internal compiler error: Segmentation fault

2018-05-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85655 Martin Jambor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/85655] [8/9 Regression] ICE with -flto and -O2 during IPA pass: cp lto1: internal compiler error: Segmentation fault

2018-05-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85655 --- Comment #10 from Martin Jambor --- Author: jamborm Date: Thu May 17 12:23:34 2018 New Revision: 260320 URL: https://gcc.gnu.org/viewcvs?rev=260320=gcc=rev Log: Check is_single_const in intersect_with_plats 2018-05-17 Martin Jambor

Re: [wwwdocs] Add GCC 8 Fortran feature description

2018-05-17 Thread Damian Rouson
Thank you! Damian On May 17, 2018 at 12:32:19 AM, Thomas Koenig (tkoe...@netcologne.de) wrote: > Hi Damian, > > Partial support is provided for Fortran 2018 teams, which are > hierarchical > subsets of images that execute independently of other image subsets. > > > Committed. > >

[Bug ipa/85655] [8/9 Regression] ICE with -flto and -O2 during IPA pass: cp lto1: internal compiler error: Segmentation fault

2018-05-17 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85655 --- Comment #9 from Martin Jambor --- Author: jamborm Date: Thu May 17 12:18:06 2018 New Revision: 260319 URL: https://gcc.gnu.org/viewcvs?rev=260319=gcc=rev Log: Check is_single_const in intersect_with_plats 2018-05-17 Martin Jambor

[Patch, fortran] PR82923 - Automatic allocation of deferred length character using function result

2018-05-17 Thread Paul Richard Thomas
The ChangeLog and the comments in the patch tell all. Bootstrapped and regtested on FC27/x86_64. OK for 7-branch through to trunk? Paul 2018-05-17 Paul Thomas PR fortran/82923 * trans-array.c (gfc_alloc_allocatable_for_assignment): Set the charlen

[PATCH] Another DSE improvement and thinko fix

2018-05-17 Thread Richard Biener
The previous DSE improvements left us with skipping elements we could have possibly removed because I messed up the iterator increment upon removal. The following fixes this and also adds another pruning opportunity in case the only stmt feeded by the def is an already visited PHI. Bootstrap

[PATCH] Improve get_ref_base_and_extent with range-info

2018-05-17 Thread Richard Biener
The following makes use of range-info to improve the basic building block of the alias-oracle so we can tell that in a[0] = 1; for (int i = 5; i < 17; ++i) a[i] = i; a[0] = 2; the ao_ref for a[i] does not alias the a[0] acceses. Given range-info is not always going to improve things

[PATCH] Improve memset handling in value-numbering

2018-05-17 Thread Richard Biener
Noticed in PR63185. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-05-17 Richard Biener * tree-ssa-sccvn.c (vn_reference_lookup_3): Improve memset handling. * gcc.dg/tree-ssa/ssa-fre-63.c: New testcase. diff --git

[Bug libstdc++/85818] [8/9 Regression] undefined reference to `std::experimental::filesystem::v1::__cxx11::path::preferred_separator'

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85818 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug libstdc++/85818] New: [8/9 Regression] undefined reference to `std::experimental::filesystem::v1::__cxx11::path::preferred_separator'

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85818 Bug ID: 85818 Summary: [8/9 Regression] undefined reference to `std::experimental::filesystem::v1::__cxx11::path::pre ferred_separator' Product: gcc Version:

[Bug middle-end/85599] Prevent short-circuiting of logical expressions for non-pure functions

2018-05-17 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85599 --- Comment #27 from Thomas Koenig --- [Replying to myself now] > A front end pass (what else did you expect? :-) > > If an impure function is found (recursively) in the operands of an .AND. > expression, issue a > >

Re: I want to dump something when I compile the program. How should I do ?

2018-05-17 Thread Martin Jambor
Hi, On Thu, May 17 2018, 冠人 王 via gcc wrote: > My work is to modify the gcc source code so as to customize the warning > message, when the programmer writing the program violating some rules.  > When the violation occurs, I want to reveal some message such as "guanjen375 > warning: the rule ###

[Bug tree-optimization/85804] [8/9 Regression][AArch64] Mis-compilation of loop with strided array access and xor reduction

2018-05-17 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85804 bin cheng changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

Re: Vector pointer modes

2018-05-17 Thread Andrew Stubbs
On 16/05/18 22:01, Richard Sandiford wrote: Andrew Stubbs writes: Hi all, I'm in the process of trying to update our AMD GCN port from GCC 7 to GCC 8+, but I've hit a problem ... It seems there's a new assumption that pointers and addresses will be scalar, but GCN load

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Marc Glisse
On Thu, 17 May 2018, Richard Biener wrote: On Thu, May 17, 2018 at 1:14 PM Marc Glisse wrote: On Thu, 17 May 2018, Jonathan Wakely wrote: On 17/05/18 12:54 +0200, Marc Glisse wrote: On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at

[Bug ada/85760] Using generic function to initialize a type created by a generic package crashes GNAT

2018-05-17 Thread jhb.chat at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85760 --- Comment #8 from Jere --- OH! I thought you meant here. I wasn't sure this was the same bug internally. They have different results in the two different versions of GNAT. The AdaCore GNAT Community put out a crash report while the FSF

Re: Implement SLP of internal functions

2018-05-17 Thread Richard Biener
On Wed, May 16, 2018 at 12:18 PM Richard Sandiford < richard.sandif...@linaro.org> wrote: > SLP of calls was previously restricted to built-in functions. > This patch extends it to internal functions. > Tested on aarch64-linux-gnu (with and without SVE), aarch64_be-elf > and x86_64-linux-gnu.

Re: PR83648

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 1:25 PM Prathamesh Kulkarni < prathamesh.kulka...@linaro.org> wrote: > On 15 May 2018 at 12:20, Richard Biener wrote: > > On Tue, 15 May 2018, Prathamesh Kulkarni wrote: > > > >> On 12 January 2018 at 18:26, Richard Biener wrote: >

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 1:14 PM Marc Glisse wrote: > On Thu, 17 May 2018, Jonathan Wakely wrote: > > On 17/05/18 12:54 +0200, Marc Glisse wrote: > >> On Mon, 14 May 2018, Jonathan Wakely wrote: > >> > >>> As discussed at

[Bug tree-optimization/85793] [8/9 Regression][AARCH64] ICE in verify_gimple during GIMPLE pass vect.

2018-05-17 Thread amker at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85793 --- Comment #3 from bin cheng --- Author: amker Date: Thu May 17 11:25:43 2018 New Revision: 260317 URL: https://gcc.gnu.org/viewcvs?rev=260317=gcc=rev Log: PR tree-optimization/85793 * tree-vect-stmts.c (vectorizable_load):

Re: PR83648

2018-05-17 Thread Prathamesh Kulkarni
On 15 May 2018 at 12:20, Richard Biener wrote: > On Tue, 15 May 2018, Prathamesh Kulkarni wrote: > >> On 12 January 2018 at 18:26, Richard Biener wrote: >> > On Fri, 12 Jan 2018, Prathamesh Kulkarni wrote: >> > >> >> On 12 January 2018 at 05:02, Jeff Law

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Marc Glisse
On Thu, 17 May 2018, Jonathan Wakely wrote: On 17/05/18 12:54 +0200, Marc Glisse wrote: On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the

[Bug ipa/85817] [9 Regression] ICE in expand_call at gcc/calls.c:4291

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

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Jonathan Wakely
On 17/05/18 12:54 +0200, Marc Glisse wrote: On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the _Array(size_t) constructor is never used.

[Bug preprocessor/77810] provide format_inform_at_substring to go with format_warning_at_substring

2018-05-17 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77810 Eric Gallager changed: What|Removed |Added Keywords||diagnostic

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Marc Glisse
On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the _Array(size_t) constructor is never used. * include/bits/valarray_array.h

Re: [PATCH PR85793]Fix ICE by loading vector(1) scalara_type for 1 element-wise case

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 12:11 PM Bin.Cheng wrote: > On Thu, May 17, 2018 at 11:07 AM, Richard Biener > wrote: > > On Wed, May 16, 2018 at 5:13 PM Bin Cheng wrote: > > > >> Hi, > >> This patch fixes ICE by loading vector(1)

[Bug ipa/85817] [9 Regression] ICE in expand_call at gcc/calls.c:4291

2018-05-17 Thread prathamesh3492 at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85817 --- Comment #1 from prathamesh3492 at gcc dot gnu.org --- Created attachment 44142 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44142=edit Untested fix Oops, sorry about that. I put the condition if (integer_zerop (retval)) continue;

Re: Replace FMA_EXPR with one internal fn per optab

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 10:56 AM Richard Sandiford < richard.sandif...@linaro.org> wrote: > Richard Biener writes: > >> @@ -2698,23 +2703,26 @@ convert_mult_to_fma_1 (tree mul_result, > >> } > > > >> if (negate_p) > >> - mulop1 =

Re: Gimple FE support for internal functions

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 10:27 AM Richard Sandiford < richard.sandif...@linaro.org> wrote: > This patch gets the gimple FE to parse calls to internal functions. > The only non-obvious thing was how the functions should be written > to avoid clashes with real function names. One option would be to

Re: Allow gimple_build with internal functions

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 10:21 AM Richard Sandiford < richard.sandif...@linaro.org> wrote: > This patch makes the function versions of gimple_build and > gimple_simplify take combined_fns rather than built_in_codes, > so that they work with internal functions too. The old > gimple_builds were

Re: [PATCH][RFC] Radically simplify emission of balanced tree for switch statements.

2018-05-17 Thread Martin Liška
On 01/15/2018 12:22 AM, Bernhard Reutner-Fischer wrote: > Can you please post CSiBE numbers? Ideally throwing in gcc-3.4.6 numbers too? > > thanks, Hi. I've just retested the patch and looks fine. There are numbers of CSiBE. I'm sorry I don't have such old version of GCC:

Re: [RFC][PR64946] "abs" vectorization fails for char/short types

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 4:56 AM Andrew Pinski wrote: > On Wed, May 16, 2018 at 7:14 PM, Kugan Vivekanandarajah > wrote: > > As mentioned in the PR, I am trying to add ABSU_EXPR to fix this > > issue. In the attached patch, in

[PATCH 07/14] Covert ipa-pure-const.c to symbol_summary.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * ipa-pure-const.c (struct funct_state_d): Do it class instead of struct. (class funct_state_summary_t): New function_summary class. (has_function_state): Remove. (get_function_state): Likewise.

[PATCH][arm][2/2] Remove support for -march=armv3 and older

2018-05-17 Thread Kyrill Tkachov
Hi all, We deprecated architecture versions earlier than Armv4T in GCC 6 [1]. This patch removes support for architectures lower than Armv4. That is the -march values armv2, armv2a, armv3, armv3m are removed with this patch. I did not remove armv4 because it's a bit more involved code-wise and

[PATCH][arm][1/2] Remove support for deprecated -march=armv5 and armv5e

2018-05-17 Thread Kyrill Tkachov
Hi all, The -march=armv5 and armv5e options have been deprecated in GCC 7 [1]. This patch removes support for them. It's mostly mechanical stuff. The functionality that was previously gated on arm_arch5 is now gated on arm_arch5t and the functionality that was gated on arm_arch5e is now gated on

[PATCH 08/14] Convert IPA CP to symbol_summary.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * ipa-cp.c (ipcp_store_bits_results): Use ipcp_transformation_sum. (ipcp_store_vr_results): Likewise. * ipa-prop.c (ipcp_grow_transformations_if_necessary): Renamed to ...

[PATCH 02/14] Code refactoring for call_summary.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * symbol-summary.h (release): Move definition out of class declaration. (symtab_removal): Likewise. (symtab_duplication): Likewise. --- gcc/symbol-summary.h | 123

[Bug libstdc++/85812] [7/8/9 Regression] make_exception_ptr can leak the allocated exception if construction throws

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85812 Jonathan Wakely changed: What|Removed |Added Status|NEW |ASSIGNED Known to work|

[PATCH 04/14] Add {symbol,call}_summary::get method and use it in HSA.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * hsa-common.h (enum hsa_function_kind): Rename HSA_NONE to HSA_INVALID. (hsa_function_summary::hsa_function_summary): Use the new enum value. (hsa_gpu_implementation_p): Use hsa_summaries::get.

[PATCH 00/14] Finish transition of {symbol,call}_summary.

2018-05-17 Thread marxin
Hi. Following patch series finishes transition of IPA summary info that is assigned either to cgraph_node or to cgraph_edge. Instead of using a vector indexed with uid, we use summaries that are internally implemented as hash_map. Doing the transition we can remove summary_uid and uid property

Re: [PATCH PR85793]Fix ICE by loading vector(1) scalara_type for 1 element-wise case

2018-05-17 Thread Bin.Cheng
On Thu, May 17, 2018 at 11:07 AM, Richard Biener wrote: > On Wed, May 16, 2018 at 5:13 PM Bin Cheng wrote: > >> Hi, >> This patch fixes ICE by loading vector(1) scalar_type if it's 1 > element-wise for VMAT_ELEMENTWISE. >> Bootstrap and test on

[PATCH 14/14] Come up with cgraph_node::get_uid and make cgraph_node::uid private.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * cgraph.c (function_version_hasher::hash): Use cgraph_node::get_uid (). (function_version_hasher::equal): * cgraph.h (cgraph_node::get_uid): New method. * ipa-inline.c (update_caller_keys): Use

[PATCH 12/14] Port edge_growth_cache to call_summary.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * ipa-inline-analysis.c (inline_edge_removal_hook): Remove. (initialize_growth_caches): Remove. (free_growth_caches): Likewise. (do_estimate_edge_time): Use edge_growth_cache.

[PATCH 03/14] Rename get methods in symbol-summary.h to get_create.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * config/i386/i386.c (ix86_can_inline_p): Use get_create instead of get. * hsa-common.c (hsa_summary_t::link_functions): Likewise. (hsa_register_kernel): Likewise. * hsa-common.h

[PATCH 11/14] Port IPA CP to edge_clone_summaries.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * ipa-cp.c (class edge_clone_summary): New summary. (grow_edge_clone_vectors): Remove. (ipcp_edge_duplication_hook): Remove. (class edge_clone_summary_t): New call_summary class.

[PATCH 05/14] Use summaries->get where possible. Small refactoring of multiple calls.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * ipa-fnsummary.c (dump_ipa_call_summary): Use ::get method. (analyze_function_body): Extract multiple calls of get_create. * ipa-inline-analysis.c (simple_edge_hints): Likewise. * ipa-inline.c

[PATCH 10/14] Add call_summary::get method and m_initialize_when_cloning.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * symbol-summary.h (get): New function. (call_summary::m_initialize_when_cloning): New class member. --- gcc/symbol-summary.h | 66 +++- 1 file changed, 50 insertions(+),

[PATCH 06/14] Use symtab_summary in ipa-reference.c.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * ipa-reference.c (remove_node_data): Remove. (duplicate_node_data): Likewise. (class ipa_ref_var_info_summary_t): New class. (class ipa_ref_opt_summary_t): Likewise. (get_reference_vars_info): Use

[PATCH 01/14] Code refactoring of symtab_summary.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * symbol-summary.h (function_summary): Move constructor implementation out of class declaration. (release): Likewise. (symtab_insertion): Likewise. (symtab_removal): Likewise.

[PATCH 13/14] Make cgraph_edge::uid really unique.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-04-24 Martin Liska * cgraph.c (symbol_table::create_edge): Always assign a new unique number. (symbol_table::free_edge): Do not recycle numbers. * cgraph.h (cgraph_edge::get): New method. * symbol-summary.h

[PATCH 09/14] Remove cgraph_node::summary_uid and make cgraph_node::uid really unique.

2018-05-17 Thread marxin
gcc/ChangeLog: 2018-05-16 Martin Liska * cgraph.c (cgraph_node::remove): Do not recycle uid. * cgraph.h (symbol_table::release_symbol): Do not pass uid. (symbol_table::allocate_cgraph_symbol): Do not set uid. * passes.c (uid_hash_t): Record

[Bug ipa/85817] New: [9 Regression] ICE in expand_call at gcc/calls.c:4291

2018-05-17 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85817 Bug ID: 85817 Summary: [9 Regression] ICE in expand_call at gcc/calls.c:4291 Product: gcc Version: unknown Status: UNCONFIRMED Keywords: ice-on-valid-code Severity:

Re: [PATCH PR85793]Fix ICE by loading vector(1) scalara_type for 1 element-wise case

2018-05-17 Thread Richard Biener
On Wed, May 16, 2018 at 5:13 PM Bin Cheng wrote: > Hi, > This patch fixes ICE by loading vector(1) scalar_type if it's 1 element-wise for VMAT_ELEMENTWISE. > Bootstrap and test on x86_64 and AArch64 ongoing. Is it OK? OK. Richard. > Thanks, > bin > 2018-05-16 Bin Cheng

[Bug c++/69905] Digit separators break literal operators

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69905 Jonathan Wakely changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/85323] SSE/AVX/AVX512 shift by 0 not optimized away

2018-05-17 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85323 --- Comment #8 from Jakub Jelinek --- Author: jakub Date: Thu May 17 10:07:12 2018 New Revision: 260313 URL: https://gcc.gnu.org/viewcvs?rev=260313=gcc=rev Log: PR target/85323 * config/i386/i386.c (ix86_fold_builtin): Handle

[Bug libstdc++/84671] [6/7 Regression] Chrono literals don't accept apostrophe as separator

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84671 Jonathan Wakely changed: What|Removed |Added CC||alisdairm at me dot com --- Comment

Re: [PATCH, aarch64] Patch to update pipeline descriptions in thunderx2t99.md

2018-05-17 Thread Richard Earnshaw (lists)
On 09/05/18 23:37, Steve Ellcey wrote: > On Fri, 2018-05-04 at 14:05 -0700, Andrew Pinski wrote: >>   >>>    (thunderx2t99_loadpair): Fix cpu unit ordering. >> I think the original ordering was correct.  The address calculation >> happens before the actual load. >> thunderx2t99_asimd_load1_ldp

[Bug target/85323] SSE/AVX/AVX512 shift by 0 not optimized away

2018-05-17 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85323 --- Comment #7 from Jakub Jelinek --- Author: jakub Date: Thu May 17 10:01:33 2018 New Revision: 260312 URL: https://gcc.gnu.org/viewcvs?rev=260312=gcc=rev Log: PR target/85323 * config/i386/i386.c (ix86_fold_builtin): Fold

Re: [RFC][PR82479] missing popcount builtin detection

2018-05-17 Thread Bin.Cheng
On Thu, May 17, 2018 at 2:39 AM, Kugan Vivekanandarajah wrote: > Hi Richard, > > On 6 March 2018 at 02:24, Richard Biener wrote: >> On Thu, Feb 8, 2018 at 1:41 AM, Kugan Vivekanandarajah >> wrote:

[Bug target/85323] SSE/AVX/AVX512 shift by 0 not optimized away

2018-05-17 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85323 --- Comment #6 from Jakub Jelinek --- Author: jakub Date: Thu May 17 09:54:36 2018 New Revision: 260311 URL: https://gcc.gnu.org/viewcvs?rev=260311=gcc=rev Log: PR target/85323 * config/i386/i386.c: Include

Re: [PATCH][GCC][AArch64] Correct 3 way XOR instructions adding missing patterns.

2018-05-17 Thread Kyrill Tkachov
Hi Tamar, On 30/04/18 15:12, Tamar Christina wrote: Hi All, This patch adds the missing neon intrinsics for all 128 bit vector Integer modes for the three-way XOR and negate and xor instructions for Arm8.2-a to Armv8.4-a. Bootstrapped and regtested on aarch64-none-linux-gnue and no issues.

Re: [PATCH][AARCH64][PR target/84882] Add mno-strict-align

2018-05-17 Thread Kyrill Tkachov
Hi Sudi, On 27/03/18 13:58, Sudakshina Das wrote: Hi This patch adds the no variant to -mstrict-align and the corresponding function attribute. To enable the function attribute, I have modified aarch64_can_inline_p () to allow checks even when the callee function has no attribute. The need for

[Bug middle-end/85599] Prevent short-circuiting of logical expressions for non-pure functions

2018-05-17 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85599 --- Comment #26 from janus at gcc dot gnu.org --- (In reply to Thomas Koenig from comment #25) > > We do have problems with compiler-dependent behavior already. The behavior > > I'm proposing is consistent with what ifort, PGI and flang do. > >

[Bug c++/85815] [7/8/9 Regression] incorrect "invalid use of incomplete type" in a lambda on valid code

2018-05-17 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85815 Richard Biener changed: What|Removed |Added Priority|P3 |P2 Status|UNCONFIRMED

Re: [patch AArch64] Do not perform a vector splat for vector initialisation if it is not useful

2018-05-17 Thread Richard Earnshaw (lists)
On 16/05/18 09:37, Kyrill Tkachov wrote: > > On 15/05/18 10:58, Richard Biener wrote: >> On Tue, May 15, 2018 at 10:20 AM Kyrill Tkachov >> >> wrote: >> >>> Hi all, >>> This is a respin of James's patch from: >>

[Bug c++/69905] Digit separators break literal operators

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69905 --- Comment #8 from Jonathan Wakely --- The testcase in comment 2 (the only one in the bug that's actually correct!) works now, fixed by r258157 for Bug 84671. The problem in comment 7 still exists, the corrected testcase is: #include int

[Bug libstdc++/85813] make_exception_ptr should support __cxa_init_primary_exception path even with -fno-exceptions

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85813 --- Comment #2 from Jonathan Wakely --- ... because if that constructor throws and the current translation unit is built without exceptions, we can't catch it.

[Bug c++/85713] [9 Regression] ICE in dependent_type_p, at cp/pt.c:24582 on valid code

2018-05-17 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85713 --- Comment #8 from paolo at gcc dot gnu.org --- Author: paolo Date: Thu May 17 09:17:56 2018 New Revision: 260308 URL: https://gcc.gnu.org/viewcvs?rev=260308=gcc=rev Log: 2018-05-17 Paolo Carlini PR

[Bug libstdc++/85813] make_exception_ptr should support __cxa_init_primary_exception path even with -fno-exceptions

2018-05-17 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85813 --- Comment #1 from Jonathan Wakely --- As you pointed out in PR 85812 the constructor of the exception can throw. That can happen even if the current translation unit is not built with exceptions, if the constructor is defined in a different

[Bug libstdc++/85812] make_exception_ptr can leak the allocated exception if construction throws

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

[Bug tree-optimization/85814] [8/9 Regression] ICE Segmentation fault during GIMPLE pass: strlen -O3 and above

2018-05-17 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85814 Martin Liška changed: What|Removed |Added Keywords|needs-bisection | CC|

Re: Replace FMA_EXPR with one internal fn per optab

2018-05-17 Thread Richard Sandiford
Richard Biener writes: >> @@ -2698,23 +2703,26 @@ convert_mult_to_fma_1 (tree mul_result, >> } > >> if (negate_p) >> - mulop1 = force_gimple_operand_gsi (, >> - build1 (NEGATE_EXPR, >> -

[Bug tree-optimization/85814] [8/9 Regression] ICE Segmentation fault during GIMPLE pass: strlen -O3 and above

2018-05-17 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85814 Martin Liška changed: What|Removed |Added Keywords|needs-reduction | --- Comment #3 from Martin Liška ---

[Bug middle-end/85811] Invalid optimization with fmax, fabs and nan

2018-05-17 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85811 --- Comment #9 from rguenther at suse dot de --- On Thu, 17 May 2018, sch...@linux-m68k.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85811 > > --- Comment #8 from Andreas Schwab --- > > Is there sth like

Re: [PATCH][AArch64] Unify vec_set patterns, support floating-point vector modes properly

2018-05-17 Thread Kyrill Tkachov
On 15/05/18 18:56, Richard Sandiford wrote: Kyrill Tkachov writes: Hi all, We've a deficiency in our vec_set family of patterns. We don't support directly loading a vector lane using LD1 for V2DImode and all the vector floating-point modes. We do do it

Re: [PATCH] PR gcc/84923 - gcc.dg/attr-weakref-1.c failed on aarch64

2018-05-17 Thread Richard Biener
On Thu, 17 May 2018, Kyrill Tkachov wrote: > Hi, > > Given this is a midend change it's a good idea to CC some of the maintainers > of that area. > I've copied richi and Honza. The patch is ok for trunk (it's actually mine...) and for the branch after a while. Thanks, Richard. > Thanks, >

[Bug tree-optimization/85814] [8/9 Regression] ICE Segmentation fault during GIMPLE pass: strlen -O3 and above

2018-05-17 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85814 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org --- Comment #2

[Bug tree-optimization/63185] Improve DSE with branches

2018-05-17 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63185 --- Comment #9 from Richard Biener --- So the following incomplete patch solves the value-numbering issue in FRE2. diff --git a/gcc/tree-dfa.c b/gcc/tree-dfa.c index a121b880bb0..745d60cbda5 100644 --- a/gcc/tree-dfa.c +++ b/gcc/tree-dfa.c @@

[Bug middle-end/85811] Invalid optimization with fmax, fabs and nan

2018-05-17 Thread sch...@linux-m68k.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85811 --- Comment #8 from Andreas Schwab --- > Is there sth like -NaN?! signbit can tell you.

<    1   2   3   >