[C++ Patch] PR 78344 ("ICE on invalid c++ code (tree check: expected tree_list, have error_mark in cp_check_const_attributes, at cp/decl2.c:1347")

Hi, in this error recovery issue cp_check_const_attributes and more generally cplus_decl_attributes have lots of troubles handling the error_mark_node returned by cp_parser_std_attribute_spec_seq, as called by cp_parser_direct_declarator. I fiddled quite a bit with these parsing facilities

[PATCH] Change PRED_LOOP_EXIT from 85 to 92

With a value of 85 GCC has a CPU performance degradation of 11%, reverting PRED_LOOP_EXIT to 92 this degradation disappear. Those values where measured by running c-ray ray-tracer that is a floating point benchmark that runs out of L1 cache. Signed-off-by: David Esparza

Re: [045/nnn] poly_int: REG_ARGS_SIZE

This breaks gcc.dg/tls/opt-3.c, gcc.dg/tls/pr47715-3.c and gcc.dg/tls/struct-1.c on m68k: /daten/aranym/gcc/gcc-20171222/gcc/testsuite/gcc.dg/tls/opt-3.c:11:3: internal compiler error: in add_args_size_note, at rtlanal.c:2379 0xae7aa9 add_args_size_note(rtx_insn*, poly_int<1u, l

Re: [PATCH] Fix various x86 avx512{bitalg,vpopcntdq,vbmi2} issues (PR target/83488)

On Fri, Dec 22, 2017 at 07:40:11PM +0100, Jakub Jelinek wrote: > Starting bootstrap/regtest on x86_64-linux and i686-linux right now, ok for > trunk if it passes? Bootstrapped/regtested successfully on both. Jakub

[v3 of PATCH 13/14] c-format.c: handle location wrappers

On Thu, 2017-12-21 at 00:00 -0500, Jason Merrill wrote: > On Wed, Dec 20, 2017 at 12:33 PM, David Malcolm > wrote: > > On Tue, 2017-12-19 at 14:55 -0500, Jason Merrill wrote: > > > On 12/17/2017 11:29 AM, David Malcolm wrote: > > > > On Mon, 2017-12-11 at 18:45 -0500, Jason

[PATCH] Fix various x86 avx512{bitalg,vpopcntdq,vbmi2} issues (PR target/83488)

On Fri, Dec 22, 2017 at 03:38:03PM +0300, Kirill Yukhin wrote: > Hello, Julia, > On 12 Nov 12:51, Koval, Julia wrote: > > Hi, this patch enables AVX512BITALG and AVX512VPOPCNTDQ instructions from > >

Re: [PATCH] Fix C FE debug regression introduced with early debug (PR debug/83550)

On Fri, 22 Dec 2017, Jakub Jelinek wrote: > Hi! > > Starting with GCC6 we emit for C (C++ is correct) incorrect DW_AT_decl_* > location for structs which are forward declared, then have some incomplete > var defined with it and finally defined. > > The problem is that we process incomplete_vars

Re: [PATCH] Fix recent DEBUG_BEGIN_STMT related regressions (PR debug/83547)

On Fri, 22 Dec 2017, Jakub Jelinek wrote: > Hi! > > The recent change to clear TREE_SIDE_EFFECTS on STATEMENT_LIST containing > DEBUG_BEGIN_STMTs and a single other statement without TREE_SIDE_EFFECTS on > it breaks the C stmt expr handling. The problem is that it assumes if > TREE_SIDE_EFFECTS

Re: [PATCH] Assign result of get_string_lenth to a SSA_NAME (PR tree-optimization/83552).

On Fri, Dec 22, 2017 at 04:18:05PM +0100, Martin Liška wrote: > In order to fix the PR, I save temporary expression to a SSA_NAME that > is then used in the gcall. You need to do that only if (!is_gimple_val (arg1_len)). Can you please emit the additional stmt only if that isn't true? > ---

Go patch committed: do not propagate address-taken of slice element to slice

This patch by Cherry Zhang fixes the Go frontend to not propagate address-taken of a slice element to the slice. Array_index_expression may be used for indexing/slicing array or slice. If a slice element is address taken, the slice itself is not necessarily address taken. Only propagate

Go patch committed: Bring escape analysis mostly in line with gc compiler

This patch to the Go frontend by Cherry Zhang brings the Go frontend's escape analysis mostly in line with gc compiler. Changes include: - In the gc compiler, the variable expression is represented with the variable node itself (ONAME). It is the same node used in the AST for multiple var

[PATCH] Assign result of get_string_lenth to a SSA_NAME (PR tree-optimization/83552).

Hi. In order to fix the PR, I save temporary expression to a SSA_NAME that is then used in the gcall. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2017-12-22 Martin Liska PR

[PATCH] Fix C FE debug regression introduced with early debug (PR debug/83550)

Hi! Starting with GCC6 we emit for C (C++ is correct) incorrect DW_AT_decl_* location for structs which are forward declared, then have some incomplete var defined with it and finally defined. The problem is that we process incomplete_vars before actually updating DECL_SOURCE_LOCATION

[PATCH] Fix recent DEBUG_BEGIN_STMT related regressions (PR debug/83547)

Hi! The recent change to clear TREE_SIDE_EFFECTS on STATEMENT_LIST containing DEBUG_BEGIN_STMTs and a single other statement without TREE_SIDE_EFFECTS on it breaks the C stmt expr handling. The problem is that it assumes if TREE_SIDE_EFFECTS is clear on a STATEMENT_LIST then that means the

Re: [patch][x86,avx] Enable AVX512BITALG

On Fri, Dec 22, 2017 at 03:38:03PM +0300, Kirill Yukhin wrote: > Hello, Julia, > On 12 Nov 12:51, Koval, Julia wrote: > > Hi, this patch enables AVX512BITALG and AVX512VPOPCNTDQ instructions from > >

Re: [PATCH] Fix avx512vnnivlintrin.h builtins (PR target/83488)

Hello Jakub, On 22 Dec 12:05, Jakub Jelinek wrote: > On Fri, Dec 22, 2017 at 11:41:27AM +0100, Uros Bizjak wrote: > > > Ok for trunk? > > > > OK. > > On top of this patch and the reason why we needed to move avx512vnni > from flags2 to flags instead of just: > case OPT_mavx512vnni: >

Re: [patch][x86,avx] Enable AVX512BITALG

Hello, Julia, On 12 Nov 12:51, Koval, Julia wrote: > Hi, this patch enables AVX512BITALG and AVX512VPOPCNTDQ instructions from > https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf. > Ok for trunk? OK for trunk. I've

Re: [PATCH] Add NON_RECURSIVE attribute for procedures

Hi Janne, I think, in general, our current system where -std=gnu means roughly "everything we have implemented so far, except legacy extensions that break standards conformance" is pretty good. The problem that I see with implementing this change is stack space, especially the traditionally

[PATCH] Fix avx512vnnivlintrin.h builtins (PR target/83488)

On Fri, Dec 22, 2017 at 11:41:27AM +0100, Uros Bizjak wrote: > > Ok for trunk? > > OK. On top of this patch and the reason why we needed to move avx512vnni from flags2 to flags instead of just: case OPT_mavx512vnni: if (value) { opts->x_ix86_isa_flags2 |=

Re: [PATCH/x86] Move mavx512vnni option from ix86_isa_flags2 to ix86_isa_flags.

On Fri, Dec 22, 2017 at 11:35 AM, Tsimbalist, Igor V wrote: > This is a follow up patch for pr83488 to fix an error in setting > > OPTION_MASK_ISA_AVX512VNNI_SET and OPTION_MASK_ISA_AVX512F_SET bits. > > There were both set in ix86_isa_flags2 while being defined in >

[PATCH/x86] Move mavx512vnni option from ix86_isa_flags2 to ix86_isa_flags.

This is a follow up patch for pr83488 to fix an error in setting OPTION_MASK_ISA_AVX512VNNI_SET and OPTION_MASK_ISA_AVX512F_SET bits. There were both set in ix86_isa_flags2 while being defined in different ISA sets. Additionally move OPTION_MASK_ISA_AVX512VNNI_SET to ix86_isa_flags as it can be

[PING 4] Ability to remap file names in __FILE__, etc (PR other/70268)

Hi, I would like to again ping this patch: https://gcc.gnu.org/ml/gcc-patches/2017-11/msg01451.html It has been reviewed (with thanks) by David Malcolm[1] and Martin Sebor[2]. Their concerns are addressed in the latest revision of the patch:

Backports to 7.3

Hi! 5 further backports to 7 branch, bootstrapped/regtested on x86_64-linux and i686-linux, committed to branch. Jakub 2017-12-22 Jakub Jelinek Backported from mainline 2017-12-19 Jakub Jelinek PR ipa/82801 PR