Re: [SFN+LVU+IEPM v4 7/9] [LVU] Introduce location views

2018-02-11 Thread Alexandre Oliva
On Feb 11, 2018, Andreas Schwab wrote: > On Feb 09 2018, Alexandre Oliva wrote: >> + if (list_head->vl_symbol && dwarf2out_locviews_in_attribute ()) >> +{ >> + ASM_OUTPUT_LABEL (asm_out_file, list_head->vl_symbol); > That needs to use

Re: [SFN+LVU+IEPM v4 7/9] [LVU] Introduce location views

2018-02-11 Thread Alexandre Oliva
On Feb 11, 2018, Andreas Schwab wrote: >> + if (list_head->vl_symbol && dwarf2out_locviews_in_attribute ()) >> +{ >> + ASM_OUTPUT_LABEL (asm_out_file, list_head->vl_symbol); > That needs to use ASM_OUTPUT_DEBUG_LABEL. There's another use of the same macro that

Re: [PATCH][C++] Fix PR84281

2018-02-11 Thread Richard Biener
On Fri, 9 Feb 2018, Jason Merrill wrote: > OK. Failed to notice that vec_safe_reserve reserves additional space so applied as follows restricting it to the very first iteration. Richard. 2018-02-12 Richard Biener PR c++/84281 * constexpr.c

Re: [Patch, Fortran] PR 84273: Reject allocatable passed-object dummy argument (proc_ptr_47.f90)

2018-02-11 Thread Richard Biener
On Sun, 11 Feb 2018, Janus Weil wrote: > Dear release managers, > > 2018-02-10 0:21 GMT+01:00 Steve Kargl : > > On Fri, Feb 09, 2018 at 06:13:34PM +0100, Janus Weil wrote: > >> > >> the attached patch fixes some checking code for PASS arguments in > >>

Re: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake isa

2018-02-11 Thread Kirill Yukhin
Hello Julia. On 15 Jan 08:28, Koval, Julia wrote: > Hi, > This patch fixes subj. Ok for trunk? > > gcc/ > * config/i386/i386.c (F_AVX512VBMI2, F_GFNI, F_VPCLMULQDQ, F_AVX512VNNI, > F_AVX512BITALG): New. > > gcc/testsuite/ > * gcc.target/i386/builtin_target.c

Re: [patch][i386] Fix for PR83828

2018-02-11 Thread Kirill Yukhin
Hello Olga, On 06 Feb 08:13, Makhotina, Olga wrote: > Hi, > > This patch repairs vpopcnt tests. > > 06.02.2018 Olga Makhotina  > > gcc/testsuite/ > PR target/83828 > * gcc.target/i386/avx512bitalg-vpopcntb-1.c: Fix test. > *

Re: [patch][i386, AVX] Adding missing mask[z]_scalef_round_s[d,s] intrinsics

2018-02-11 Thread Kirill Yukhin
Hello Olga, On 14 Nov 09:56, Makhotina, Olga wrote: > Hi, > > Attachment got lost by accident. Attaching it again. > > Thanks, > Olga > > -Original Message- > From: Makhotina, Olga > Sent: Tuesday, November 14, 2017 10:49 AM > To: gcc-patches@gcc.gnu.org > Cc: Uros Bizjak

Re: [patch][i386, AVX] Adding missing mask[z]_sqrt_round_s[d,s] intrinsics

2018-02-11 Thread Kirill Yukhin
Hello Olga, On 21 Nov 12:46, Makhotina, Olga wrote: > Hi, > > This patch adds missing intrinsics for _mm_mask[z]_sqrt_round_[sd,ss]. > > 21.11.2017 Olga Makhotina  > > gcc/ >   * config/i386/avx512fintrin.h (_mm_mask_sqrt_round_sd, >  

[PATCH 2/3] Add a pass to automatically add ptwrite instrumentation

2018-02-11 Thread Andi Kleen
From: Andi Kleen Add a new pass to automatically instrument changes to variables with the new PTWRITE instruction on x86. PTWRITE writes a 4 or 8 byte field into an external Processor Trace log. This allows to reconstruct how values later, which can be useful for debugging

[PATCH 3/3] Add tests for the vartrace pass

2018-02-11 Thread Andi Kleen
From: Andi Kleen gcc/testsuite/: 2018-02-10 Andi Kleen * g++.dg/vartrace-3.C: New test. * g++.dg/vartrace-ret.C: New test. * g++.dg/vartrace-ret2.C: New test. * gcc.target/i386/vartrace-1.c: New test. *

[PATCH 1/3] Add PTWRITE builtins for x86

2018-02-11 Thread Andi Kleen
From: Andi Kleen Add builtins/intrinsics for PTWRITE. PTWRITE is a new instruction on Intel Cherry Trail that allows to write values into the Processor Trace log. This is fairly straight forward, except I had to add isa2 support for variable number of operands. gcc/:

C++ PATCH for c++/84036, ICE with variadic lambda capture

2018-02-11 Thread Jason Merrill
The old lambda model handled variadic capture by focusing on the FIELD_DECL rather than trying to map between capture proxies. The new model relies more on capture proxies, so it makes sense to use them more for variadic capture as well. So with this patch we treat a variadic capture proxy as a

[PATCH] matcher-1.m: Change return type to int

2018-02-11 Thread H.J. Lu
my_exception_matcher must return int. Otherwise, this test fails. Tested on i686 and x86-64. OK for trunk? H.J. --- PR testsuite/84324 * objc/execute/exceptions/matcher-1.m (my_exception_matcher): Change return type to int. ---

Re: [Patch, Fortran] PR 84273: Reject allocatable passed-object dummy argument (proc_ptr_47.f90)

2018-02-11 Thread Janus Weil
Dear release managers, 2018-02-10 0:21 GMT+01:00 Steve Kargl : > On Fri, Feb 09, 2018 at 06:13:34PM +0100, Janus Weil wrote: >> >> the attached patch fixes some checking code for PASS arguments in >> procedure-pointer components, which does not properly account

Re: [SFN+LVU+IEPM v4 7/9] [LVU] Introduce location views

2018-02-11 Thread Andreas Schwab
On Feb 09 2018, Alexandre Oliva wrote: > @@ -9681,34 +9978,85 @@ gen_llsym (dw_loc_list_ref list) > static void > output_loc_list (dw_loc_list_ref list_head) > { > + int vcount = 0, lcount = 0; > + >if (list_head->emitted) > return; >list_head->emitted = true;

Re: [Fortran, patch, coarray, v1] Fix a coarray issue and some documentation

2018-02-11 Thread Andre Vehreschild
Er, the revision is 257567. Sorry pressed to quickly. - Andre On Sun, 11 Feb 2018 20:00:05 +0100 Andre Vehreschild wrote: > Hi all, > > approved on IRC by Kargl. Thanks for the quick review. Committed as r > > Regards, > Andre > > On Sun, 11 Feb 2018 18:13:35 +0100 >

Re: [SFN+LVU+IEPM v4 7/9] [LVU] Introduce location views

2018-02-11 Thread Andreas Schwab
a separate attribute (DW_AT_GNU_locviews) or > (DW_LLE_view_pair) entries in DWARF5+ loclists. This breaks ia64 bootstrap. While building the stage3 compiler I get this error: /usr/local/gcc/gcc-20180211/Build/./prev-gcc/xg++ -B/usr/local/gcc/gcc-20180211/Build/./prev-gcc/ -B/usr/ia64-suse-linux/bin/ -nost

Re: [Fortran, patch, coarray, v1] Fix a coarray issue and some documentation

2018-02-11 Thread Andre Vehreschild
Hi all, approved on IRC by Kargl. Thanks for the quick review. Committed as r Regards, Andre On Sun, 11 Feb 2018 18:13:35 +0100 Andre Vehreschild wrote: > Hi all, > > attached patch fixes a small coarray issue on ultimate component coarrays. The > coarray was registered

Re: [Patch, fortran] PR84074 Incorrect indexing of array when actual argument is an array expression and dummy is polymorphic

2018-02-11 Thread Paul Richard Thomas
Committed as revision 257564 with the comment reworded. Thanks Paul On 11 February 2018 at 16:37, Steve Kargl wrote: > On Sun, Feb 11, 2018 at 03:08:52PM +, Paul Richard Thomas wrote: >> >> PR fortran/84074 >> * trans-expr.c (gfc_conv_derived_to_class):

Re: Merge from trunk to gccgo branch

2018-02-11 Thread Ian Lance Taylor
I merged trunk revision 257559 to the gccgo branch. Ian

[Fortran, patch, coarray, v1] Fix a coarray issue and some documentation

2018-02-11 Thread Andre Vehreschild
Hi all, attached patch fixes a small coarray issue on ultimate component coarrays. The coarray was registered with the caf-library as if it were an allocatable component in a derived type coarray, which is just wasting resources. An ultimate component coarray has to be treated like a regular

Re: [PATCH] Fix handling of arguments in statement functions

2018-02-11 Thread Steve Kargl
On Sun, Feb 11, 2018 at 08:34:05AM -0800, Steve Kargl wrote: > On Sun, Feb 11, 2018 at 01:48:10PM +0100, Dominique d'Humières wrote: > > Hi Steve, > > > > With your patch applied to revision r257559, I get the following failures > > > >

Re: [Patch, fortran] PR84074 Incorrect indexing of array when actual argument is an array expression and dummy is polymorphic

2018-02-11 Thread Steve Kargl
On Sun, Feb 11, 2018 at 03:08:52PM +, Paul Richard Thomas wrote: > > PR fortran/84074 > * trans-expr.c (gfc_conv_derived_to_class): Set the use_offset > flag. If the is a vector subscript or the expression is not a > variable, make the descriptor one-based. "If the is a" ? > + > +

Re: [PATCH] Fix handling of arguments in statement functions

2018-02-11 Thread Steve Kargl
On Sun, Feb 11, 2018 at 01:48:10PM +0100, Dominique d'Humières wrote: > Hi Steve, > > With your patch applied to revision r257559, I get the following failures > > /opt/gcc/p_work/gcc/testsuite/gfortran.dg/statement_function_3.f:5:72: > Warning: Obsolescent feature: Statement function at (1) >

Re: [SFN+LVU+IEPM v4 9/9] [IEPM] Introduce inline entry point markers

2018-02-11 Thread Alexandre Oliva
On Feb 10, 2018, Jeff Law wrote: >> Ports call final_scan_insn with seen == NULL, and then >> maybe_output_next_view crashes because it assumes it's >> non-NULL. Oops. Fixed. > A bit icky. But OK. Thanks. Testing revealed some ports had already introduced their own 'seen'

[Patch, fortran] PR84074 Incorrect indexing of array when actual argument is an array expression and dummy is polymorphic

2018-02-11 Thread Paul Richard Thomas
Hi All, This patch makes sure that offsets and bounds are correct in passing derived types to class formal arrays. It is straightforward enough as not to require explanation. Bootstraps and regtests on FC25/x86_64 - OK for trunk? Paul 2018-02-11 Paul Thomas PR

Re: [Patch, Fortran] PR 84273: Reject allocatable passed-object dummy argument (proc_ptr_47.f90)

2018-02-11 Thread Janus Weil
Hi guys, 2018-02-10 12:46 GMT+01:00 Paul Richard Thomas : > As Steve said, welcome back! thanks for the warm welcome :) > I hope that you will post the news of this fix and the correction of > the testcases on clf. Talking of which, have you posted the problems >

Re: [PATCH] Fix handling of arguments in statement functions

2018-02-11 Thread Dominique d'Humières
Hi Steve, With your patch applied to revision r257559, I get the following failures /opt/gcc/p_work/gcc/testsuite/gfortran.dg/statement_function_3.f:5:72: Warning: Obsolescent feature: Statement function at (1) /opt/gcc/p_work/gcc/testsuite/gfortran.dg/statement_function_3.f:3:24: Error:

New Swedish PO file for 'gcc' (version 8.1-b20180128)

2018-02-11 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-8.1-b20180128.sv.po',