Re: introduce --enable-mingw-full32 to default to --large-address-aware

2018-10-08 Thread Alexandre Oliva
On Oct 5, 2018, Joseph Myers wrote: > A new configure option needs documenting in install.texi. Ah, yes, thanks for the reminder. On Oct 6, 2018, JonY <10wa...@gmail.com> wrote: > They're both OK as far as I can see. I just don't like the configure > name implying all 32bit pointers are

[PATCH] Make std::list::iterator == and != global inline friend

2018-10-08 Thread François Dumont
As we talked one day I would like to make all iterator operators global for consistency. So here is the patch to do so for std::list iterators. Thanks to this change the operators ==(iterator, const_iterator) and != are not necessary anymore, one less ==|!= operator candidate.     *

[PATCH] __debug::list use C++11 direct initialization

2018-10-08 Thread François Dumont
Here is the communication for my yesterday's patch which I thought svn had failed to commit (I had to interrupt it). Similarly to what I've done for associative containers here is a cleanup of the std::__debug::list implementation leveraging more on C++11 direct initialization. I also made

[PING][PATCH] tighten up -Wclass-memaccess for ctors/dtors (PR 84851)

2018-10-08 Thread Martin Sebor
Ping: https://gcc.gnu.org/ml/gcc-patches/2018-05/msg01528.html I will go ahead and commit this as obvious this week if there are no objections. On 05/25/2018 02:20 PM, Martin Sebor wrote: (I just now noticed the first two attempts were sent to the wrong list. Sorry about that.) On 05/25/2018

Re: [PATCH] handle attribute positional arguments consistently (PR 87541, 87542)

2018-10-08 Thread Martin Sebor
Attached is an updated patch with the INTEGRAL_TYPE_P test added to detect constant non-integer arguments like (void*)0, and with quoting made unconditional. I also removed the pretty printer business to avoid the "value%s" format, and modified the manual to clarify when each of the attributes

Print column information in RTL dumps

2018-10-08 Thread Eric Botcazou
Now that -gcolum-info is the default, I think that it makes sense to print the column info in the RTL dumps to be able to find out where numbers come from. Tested on x86_64-suse-linux, applied on the mainline. 2018-10-08 Eric Botcazou * print-rtl.c

Re: [Patch, regrename] Fix PR87330 : ICE in scan_rtx_reg, at regrename.c

2018-10-08 Thread Eric Botcazou
> Other notes need not be changed, as they don't hold renamed register > information. > > Ok for trunk? No, REG_DEAD & REG_UNUSED note must be recomputed by passes consuming them. > 2018-10-09 Sameera Deshpande > * gcc/regrename.c (regrename_do_replace): Add condition to alter > regname if

[Patch, regrename] Fix PR87330 : ICE in scan_rtx_reg, at regrename.c

2018-10-08 Thread Sameera Deshpande
Hi! Please find attached the patch fixing the issue PR87330 : ICE in scan_rtx_reg, at regrename.c:1097. The regrename pass does not rename the registers which are in notes, because of which the REG_DEAD note had previous register names, which caused conflicting liveness information generated for

Re: [Patch, fortran] PR87151 - allocating array of character

2018-10-08 Thread Thomas Koenig
Hi Paul, Bootstraps and regtests on FC28/x86_64 - OK for trunk and later for 8-branch? OK. Thanks for the patch! Regards Thomas

Re: [Patch, Fortran] PR fortran/83522 – reject array-valued substrings

2018-10-08 Thread Thomas Koenig
Hi Tobias, nice to hear from you again! Build and regtested on x86_64-linux. OK for the trunk? OK. Thanks for the patch! Regards Thomas

Re: [PATCHv2] Handle not explicitly zero terminated strings in merge sections

2018-10-08 Thread Eric Botcazou
> Besides, the patch seems to have produced more fallout on Solaris: I see > many new Go testsuite failures on Solaris 10 which probably are related, > and Solaris bootstrap with Ada included is broken due to the extended > usage of string merging. I'm currently investigating what's going on >

Re: [PING] [PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-10-08 Thread Martin Sebor
Ping: https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01818.html As with the other patch (bug 84561), there may be ways to redesign the warning, but I don't have the cycles to undertake it before stage 1 ends. Unless someone has a simpler suggestion for how to avoid this false positive now can we

[PING #2] [PATCH] look harder for MEM_REF operand equality to avoid -Wstringop-truncation (PR 84561)

2018-10-08 Thread Martin Sebor
Ping: https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01934.html On 10/01/2018 03:30 PM, Martin Sebor wrote: Ping: https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01934.html We have discussed a number of different approaches to moving the warning somewhere else but none is feasible in the limited

[PATCH] Folding and check_function_arguments

2018-10-08 Thread David Malcolm
On Mon, 2018-10-08 at 10:37 -0400, Jason Merrill wrote: > On Thu, Oct 4, 2018 at 10:12 AM David Malcolm > wrote: > > > > -Wformat in the C++ FE doesn't work as well as it could: > > (a) it doesn't report precise locations within the string literal, > > and > > (b) it doesn't underline arguments

Re: [PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-10-08 Thread Martin Sebor
On 10/08/2018 04:05 AM, Richard Biener wrote: On Thu, Oct 4, 2018 at 5:51 PM Martin Sebor wrote: On 10/04/2018 08:58 AM, Jeff Law wrote: On 8/27/18 9:42 AM, Richard Biener wrote: On Mon, Aug 27, 2018 at 5:32 PM Jeff Law wrote: On 08/27/2018 02:29 AM, Richard Biener wrote: On Sun, Aug

Re: [C++ PATCH] Fix pretty-print of enumerator ids (PR c++/87364)

2018-10-08 Thread Jason Merrill
OK. On Mon, Oct 8, 2018 at 1:12 PM will wray wrote: > > Hi, > > A PR to fix Bug 87364 - Pretty print of enumerator never prints the id, > always falls back to C-style cast output > > The fix is tested with the code attached to the bug report and additional > usage over the past few weeks. > >

Re: [PATCH, rs6000] Fix PR86731 vec_sl()

2018-10-08 Thread Segher Boessenkool
On Mon, Oct 08, 2018 at 02:34:53PM -0500, Will Schmidt wrote: > On Sat, 2018-08-25 at 12:46 -0500, Segher Boessenkool wrote: > > Rest looks fine to me (whatever that means :-) ) > > I took that as an OK for trunk, so this (with whitespace adjustments as > recommended) went into trunk via >(

[RFC][PATCH IRA] Fix PR87507, IRA unnecessarily uses non-volatile registers during register assignment

2018-10-08 Thread Peter Bergner
PR87507 shows a problem where IRA assigns a non-volatile TImode reg pair to a pseudo when there is a volatile reg pair available to use. This then causes us to emit save/restore code for the non-volatile reg usage. The problem here is that the only volatile reg pair that is available is an

Re: [PATCH, rs6000] Fix PR86731 vec_sl()

2018-10-08 Thread Will Schmidt
On Sat, 2018-08-25 at 12:46 -0500, Segher Boessenkool wrote: > Hi! > > On Tue, Aug 14, 2018 at 06:18:28PM -0500, Will Schmidt wrote: > > PR target/86731 > > * config/rs6000/rs6000.c (rs6000_gimple_fold_builtin): Update logic > > around folding of vec_sl() to handle out of range

[gomp5] Fix task reduction handling in implicit parallel regions

2018-10-08 Thread Jakub Jelinek
Hi! In implicit parallel regions, we have NULL teams and often NULL task. For task reductions we need both non-NULL, so this patch creates such a team in that case, like we do already for target nowait. Tested on x86_64-linux, committed to gomp-5_0-branch. 2018-10-08 Jakub Jelinek *

Re: [PATCH] handle attribute positional arguments consistently (PR 87541, 87542)

2018-10-08 Thread Martin Sebor
On 10/08/2018 07:27 AM, Manuel López-Ibáñez wrote: On 07/10/18 23:38, Martin Sebor wrote: + pretty_printer posval; + if (pos != error_mark_node) +{ + /* Only format the position value when it's valid. By convention + do not quote constant integers. */ + pp_space (); +

Re: [PATCH] handle attribute positional arguments consistently (PR 87541, 87542)

2018-10-08 Thread Martin Sebor
On 10/08/2018 05:33 AM, Joseph Myers wrote: On Sun, 7 Oct 2018, Martin Sebor wrote: While still testing an enhancement in the area of attributes I ran across bugs and inconsistencies in how different handlers deal with positional arguments. The bugs are either an ICE due to the handlers not

Re: Use FOR_EACH_IMM_USE_FAST in gimple-ssa-backprop.c

2018-10-08 Thread Jeff Law
On 10/8/18 11:27 AM, Richard Sandiford wrote: > As pointed out by Richard in PR63155. It speeds up the testcase a few %. > > Tested on aarch64-linux-gnu. OK to install? > > Richard > > > 2018-10-08 Richard Sandiford > > gcc/ > PR middle-end/63155 > * gimple-ssa-backprop.c

Re: [PATCH][4/n] Remove BLOCK_ABSTRACT

2018-10-08 Thread Jason Merrill
On Fri, Oct 5, 2018 at 7:47 AM Richard Biener wrote: > > On Fri, 28 Sep 2018, Richard Biener wrote: > > > > > It turns out that nobody sets this anymore (dwarf2out did with the > > original code of outputting abstract instances, temporarily so IIRC). > > > > Bootstrap and regtest running on

Use FOR_EACH_IMM_USE_FAST in gimple-ssa-backprop.c

2018-10-08 Thread Richard Sandiford
As pointed out by Richard in PR63155. It speeds up the testcase a few %. Tested on aarch64-linux-gnu. OK to install? Richard 2018-10-08 Richard Sandiford gcc/ PR middle-end/63155 * gimple-ssa-backprop.c (backprop::intersect_uses): Use FOR_EACH_IMM_USE_FAST instead

[C++ PATCH] Fix pretty-print of enumerator ids (PR c++/87364)

2018-10-08 Thread will wray
Hi, A PR to fix Bug 87364 - Pretty print of enumerator never prints the id, always falls back to C-style cast output The fix is tested with the code attached to the bug report and additional usage over the past few weeks. Bootstrap and regtested on x86_64-linux. OK for trunk? 2018-10-08 Will

[PATCH, pdp11] libgcc: remove -mfloat32

2018-10-08 Thread Paul Koning
I missed a file that needed to be updated for the removal of -mfloat32. Committed. paul ChangeLog: 2018-10-08 Paul Koning * config/pdp11/t-pdp11: Remove -mfloat32 switch. Index: config/pdp11/t-pdp11 === ---

[gomp5] Don't recompute addresses of original array sections at the end of taskgroup

2018-10-08 Thread Jakub Jelinek
Hi! Last week we've agreed that it is allowed to modify the base of array section used in task_reductions within the taskgroup construct, so we can't reread/recompute the base and bias again, either we'd need to save those at the start of the region, or we can just read it from where we've

[PATCH][i386] Split reductions (was: Re: [PATCH][RFC][i386] Change sminmax reduction patterns)

2018-10-08 Thread Richard Biener
On Fri, 5 Oct 2018, Uros Bizjak wrote: > On Thu, Oct 4, 2018 at 2:05 PM Richard Biener wrote: > > > > > > This tries to apply the same trick to sminmax reduction patterns > > as for the reduc_plus_scal ones, namely reduce %zmm -> %ymm -> %xmm > > first. On a microbenchmark this improves

Re: [PATCHv2] Handle not explicitly zero terminated strings in merge sections

2018-10-08 Thread Bernd Edlinger
On 10/08/18 13:14, Rainer Orth wrote: > Hi Bernd, > >> On 10/05/18 20:15, Andreas Schwab wrote: >>> On Sep 14 2018, Bernd Edlinger wrote: >>> diff -Npur gcc/testsuite/gnat.dg/string_merge1.adb gcc/testsuite/gnat.dg/string_merge1.adb --- gcc/testsuite/gnat.dg/string_merge1.adb

Re: [PATCH] i386: Correct _mm512_mask3_fmaddsub_round_pd

2018-10-08 Thread H.J. Lu
On Thu, Oct 4, 2018 at 5:46 AM H.J. Lu wrote: > > Define _mm512_mask3_fmaddsub_round_pd with > __builtin_ia32_vfmaddsubpd512_mask, instead of > __builtin_ia32_vfmaddpd512_mask. > > PR target/87517 > * config/i386/avx512fintrin.h (_mm512_mask_fmaddsub_round_pd): > Defined

Re: [PATCH 2/2 v3][IRA,LRA] Fix PR86939, IRA incorrectly creates an interference between a pseudo register and a hard register

2018-10-08 Thread Jeff Law
On 10/8/18 8:43 AM, Christophe Lyon wrote: > On Mon, 8 Oct 2018 at 16:13, Peter Bergner wrote: >> >> On 10/8/18 4:14 AM, Christophe Lyon wrote: >>> Since r264897, we are seeing lots of regressions when bootstrapping on arm. >>> There are execution errors as well as ICEs. >>> A detailed list can

Re: [PATCH v3] Change default to -fno-math-errno

2018-10-08 Thread Jeff Law
On 10/4/18 1:43 PM, Joseph Myers wrote: > On Thu, 4 Oct 2018, Jeff Law wrote: > >>> I doubt you could prove that without LTO of the whole program because an >>> errno value set by a libm function call could always be checked in the >>> caller of whatever function is being compiled. >> Right,

Re: [PATCH][i386] Update Zen tuning for vector load cost

2018-10-08 Thread Richard Biener
On Mon, 8 Oct 2018, Jan Hubicka wrote: > > > > This adjusts Zen AVX256 vector load cost to be twice as expensive > > than AVX128 vector load cost (twice via ix86_vec_cost keying on > > TARGET_AVX128_OPTIMAL - should rather use some > > TARGET_VECTOR_IMPL_WIDTH or so). > > > > Likely the current

Re: Merge from trunk to gccgo branch

2018-10-08 Thread Ian Lance Taylor
I merged trunk revision 264932 to the gccgo branch. Ian

Re: [PATCH 2/2] Support string locations for C++ in -Wformat (PR c++/56856)

2018-10-08 Thread Jason Merrill
On Thu, Oct 4, 2018 at 10:12 AM David Malcolm wrote: > > -Wformat in the C++ FE doesn't work as well as it could: > (a) it doesn't report precise locations within the string literal, and > (b) it doesn't underline arguments for those arguments !CAN_HAVE_LOCATION_P, > despite having location

[PATCH] libgcc: remove t-ppccomm from powerpc-wrs-vxworks

2018-10-08 Thread Rasmus Villemoes
VxWorks doesn't use the __eabi function (in fact, the ecrti.o and ecrtn.o files are not added to extra_parts). This means that the __SBSS_END__ etc. symbols in eabi.S are always undefined. This is mostly harmless, but it is inconvenient when one wants to create a libgcc runtime module by doing a

Re: [PATCH][i386] Update Zen tuning for vector load cost

2018-10-08 Thread Jan Hubicka
> > This adjusts Zen AVX256 vector load cost to be twice as expensive > than AVX128 vector load cost (twice via ix86_vec_cost keying on > TARGET_AVX128_OPTIMAL - should rather use some > TARGET_VECTOR_IMPL_WIDTH or so). > > Likely the current cost value was meant to make AVX256 loads _cheaper_ >

libgo patch committed: Update to 1.11.1 release

2018-10-08 Thread Ian Lance Taylor
This patch updates libgo to the 1.11.1 release. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE === --- gcc/go/gofrontend/MERGE (revision 264882) +++

[PATCH, pdp11] Fix LRA failure

2018-10-08 Thread Paul Koning
This patch fixes a failure handling block moves when the LRA register allocator is used. Committed. paul ChangeLog: 2018-10-08 Paul Koning * config/pdp11/pdp11-protos.h (output_block_move): Remove. (expand_block_move): New function. * config/pdp11/pdp11.c

Re: [PATCH 2/2 v3][IRA,LRA] Fix PR86939, IRA incorrectly creates an interference between a pseudo register and a hard register

2018-10-08 Thread Peter Bergner
On 10/8/18 4:14 AM, Christophe Lyon wrote: > Since r264897, we are seeing lots of regressions when bootstrapping on arm. > There are execution errors as well as ICEs. > A detailed list can be found at: >

Re: C++ PATCH to implement P1064R0, Virtual Function Calls in Constant Expressions (v4)

2018-10-08 Thread Andreas Schwab
On Sep 20 2018, Jakub Jelinek wrote: > --- gcc/cp/class.c.jj 2018-09-20 09:56:59.229751895 +0200 > +++ gcc/cp/class.c2018-09-20 10:12:17.447370890 +0200 > @@ -9266,7 +9266,6 @@ build_vtbl_initializer (tree binfo, >tree vcall_index; >tree fn, fn_original; >tree init =

Re: [PATCH] handle attribute positional arguments consistently (PR 87541, 87542)

2018-10-08 Thread Manuel López-Ibáñez
On 07/10/18 23:38, Martin Sebor wrote: + pretty_printer posval; + if (pos != error_mark_node) +{ + /* Only format the position value when it's valid. By convention +do not quote constant integers. */ + pp_space (); + if (TREE_CODE (pos) != INTEGER_CST) +

Re: [Patch, fortran] PR87151 - allocating array of character

2018-10-08 Thread Paul Richard Thomas
Hi Dominique, Thanks for the testing. Although the tests for PR80931 and PR83196, comment #4, compiled OK, when I attempted to use the modules both segfaulted for the same reason ('span' not being set on the array descriptor) and these required a slightly different version of the same tweak. The

Re: [PATCH 1/2] zEC12 pipeline

2018-10-08 Thread Robin Dapp
Hi, committed only the zEC12 part for now. Performance behavior of z13 with the patch is still unclear and will be tackled separately. Regards Robin

{Patch, fortran] PR86372 - [8/9 Regression] Segfault on ASSOCIATE statement with CHARACTER variable

2018-10-08 Thread Paul Richard Thomas
I have fixed this as 'obvious' on 8-branch(r264925) and trunk(r264915). Paul 2018-10-08 Paul Thomas Backport from trunk PR fortran/86372 * trans-stmt.c (trans_associate_var): Character associate names with variable string length do not have to be deferred length for the

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-08 Thread Uros Bizjak
On Mon, Oct 8, 2018 at 2:20 PM Martin Liška wrote: > > On 10/8/18 1:29 PM, Uros Bizjak wrote: > > On Mon, Oct 8, 2018 at 1:21 PM Jakub Jelinek wrote: > >> > >> On Mon, Oct 08, 2018 at 01:17:14PM +0200, Martin Liška wrote: > >>> I'm suggesting following patch that comes up with new g++.target >

[PATCH v2] fixincludes: vxworks: regs.h: Fix includes in regs.h wrapper

2018-10-08 Thread Rasmus Villemoes
A quick experiment reveals that this hack is needed for C code - simply removing this hack entirely breaks the build of libstdc++, since regs.h (more accurately, the cpu-specific header it pulls in) defines structs in terms of types from vxTypesOld. Those definitions are properly guarded by

Re: [Patch 3/3][Aarch64] Implement Aarch64 SIMD ABI

2018-10-08 Thread Richard Sandiford
Steve Ellcey writes: > This is the third of three patches for Aarch64 SIMD ABI support.  This > patch is not fully tested yet but I want to post it to get comments. > > This is the only patch of the three that touches non-aarch64 specific > code.  The changes here are made to allow GCC to have

Re: [PATCHv2] Handle not explicitly zero terminated strings in merge sections

2018-10-08 Thread Rainer Orth
Hi Bernd, > Besides, the patch seems to have produced more fallout on Solaris: I see > many new Go testsuite failures on Solaris 10 which probably are related, > and Solaris bootstrap with Ada included is broken due to the extended > usage of string merging. I'm currently investigating what's

[PATCH] Better vectorizer cost model debugging

2018-10-08 Thread Richard Biener
We dump the target hook calls but sofar omit the resulting cost (which might not be final). It's still useful info for debugging a target cost model. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-10-08 Richard Biener * tree-vect-loop.c

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-08 Thread Martin Liška
On 10/8/18 1:29 PM, Uros Bizjak wrote: > On Mon, Oct 8, 2018 at 1:21 PM Jakub Jelinek wrote: >> >> On Mon, Oct 08, 2018 at 01:17:14PM +0200, Martin Liška wrote: >>> I'm suggesting following patch that comes up with new g++.target subfolder. >>> I moved there i386 multiversioning tests. >> >> I

Re: [Patch 1/3][Aarch64] Implement Aarch64 SIMD ABI

2018-10-08 Thread Richard Sandiford
Steve Ellcey writes: > @@ -1005,6 +1005,15 @@ static const struct processor *selected_tune; > /* The current tuning set. */ > struct tune_params aarch64_tune_params = generic_tunings; > > +/* Table of machine attributes. */ > +static const struct attribute_spec aarch64_attribute_table[] = >

Re: [PATCH] PR libstdc++/87538 fix std::not_fn exception specifications

2018-10-08 Thread Jonathan Wakely
On 08/10/18 12:59 +0100, Jonathan Wakely wrote: PR libstdc++/87538 * include/std/functional (_Not_fn::operator()): Check value of __is_nothrow_invocable as well. * testsuite/20_util/function_objects/not_fn/87538.cc: New test. Tested x86_64-linux, committed to

[gomp5] Tweak OMP_AFFINITY_FORMAT handling

2018-10-08 Thread Jakub Jelinek
Hi! With the addition of host teams, it makes a lot of sense to be able to print also the team number and number of teams. This change made it into OpenMP 5.0, together with various changes to mostly the short names, but two long names too. The following adjusts our implementation to match what

[PATCH] PR libstdc++/87538 fix std::not_fn exception specifications

2018-10-08 Thread Jonathan Wakely
PR libstdc++/87538 * include/std/functional (_Not_fn::operator()): Check value of __is_nothrow_invocable as well. * testsuite/20_util/function_objects/not_fn/87538.cc: New test. Tested x86_64-linux, committed to trunk. I'll backport this to gcc-7 and gcc-8 too.

Re: [PATCH v3] Change default to -fno-math-errno

2018-10-08 Thread Joseph Myers
On Mon, 8 Oct 2018, Richard Biener wrote: > So I think it would be fine if we'd have -fno-math-errno as documented > and then the C library would annotate their math functions according > to whether they will ever set errno or not. Once a math function is > const or pure it cannot ever set errno

Re: [PATCH] fixincludes: vxworks: regs.h: Guard include of vxTypesOld.h by !_ASMLANGUAGE

2018-10-08 Thread Rasmus Villemoes
On 2018-09-14 14:39, Olivier Hainque wrote: > > >> On 13 Sep 2018, at 00:25, Rasmus Villemoes wrote: > >>> What happens on your end if you just remove the hack ? > >> Unfortunately, the libstdc++ build breaks: >> >> In file included from >>

Re: [PATCH] handle attribute positional arguments consistently (PR 87541, 87542)

2018-10-08 Thread Joseph Myers
On Sun, 7 Oct 2018, Martin Sebor wrote: > While still testing an enhancement in the area of attributes > I ran across bugs and inconsistencies in how different handlers > deal with positional arguments. The bugs are either an ICE due > to the handlers not consistently converting positional

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-08 Thread Uros Bizjak
On Mon, Oct 8, 2018 at 1:21 PM Jakub Jelinek wrote: > > On Mon, Oct 08, 2018 at 01:17:14PM +0200, Martin Liška wrote: > > I'm suggesting following patch that comes up with new g++.target subfolder. > > I moved there i386 multiversioning tests. > > I think you want Uros to review and decide this

Re: [PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-08 Thread Jakub Jelinek
On Mon, Oct 08, 2018 at 01:17:14PM +0200, Martin Liška wrote: > I'm suggesting following patch that comes up with new g++.target subfolder. > I moved there i386 multiversioning tests. I think you want Uros to review and decide this one. > 2018-10-08 Martin Liska > > *

[PATCH] Come up with gcc/testsuite/g++.target/i386/i386.dg and move there some tests.

2018-10-08 Thread Martin Liška
Hi. I'm suggesting following patch that comes up with new g++.target subfolder. I moved there i386 multiversioning tests. Ready for trunk? Martin >From 152e66a766b69019ef6632276579d9fa9f14f4c4 Mon Sep 17 00:00:00 2001 From: marxin Date: Mon, 8 Oct 2018 13:13:23 +0200 Subject: [PATCH] Come up

Re: [PATCHv2] Handle not explicitly zero terminated strings in merge sections

2018-10-08 Thread Rainer Orth
Hi Bernd, > On 10/05/18 20:15, Andreas Schwab wrote: >> On Sep 14 2018, Bernd Edlinger wrote: >> >>> diff -Npur gcc/testsuite/gnat.dg/string_merge1.adb >>> gcc/testsuite/gnat.dg/string_merge1.adb >>> --- gcc/testsuite/gnat.dg/string_merge1.adb 1970-01-01 >>> 01:00:00.0 +0100 >>> +++

Re: Add new warning flag "warn_prio_ctor_dtor"

2018-10-08 Thread Rainer Orth
Jeff Law writes: > On 9/3/18 8:48 AM, Vinay Kumar wrote: >> Hi Joseph, >> The documentation is of a new option, not of -Wreturn-type >> Done >> you should list the option as -Wno-prio-ctor-dtor when documenting >> Done >> You're not meant to have the literal text ", no ()" in

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Richard Earnshaw (lists)
On 08/10/18 11:29, Richard Biener wrote: > On Mon, Oct 8, 2018 at 12:14 PM Martin Liška wrote: >> >> On 10/8/18 11:55 AM, Richard Earnshaw (lists) wrote: >>> On 08/10/18 10:47, Martin Liška wrote: On 10/8/18 10:46 AM, Renlin Li wrote: > Hi Martin, > > pr82625.C failed on compiler

[PATCH][i386] Update Zen tuning for vector load cost

2018-10-08 Thread Richard Biener
This adjusts Zen AVX256 vector load cost to be twice as expensive than AVX128 vector load cost (twice via ix86_vec_cost keying on TARGET_AVX128_OPTIMAL - should rather use some TARGET_VECTOR_IMPL_WIDTH or so). Likely the current cost value was meant to make AVX256 loads _cheaper_ than two

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Martin Liška
On 10/8/18 12:46 PM, Jakub Jelinek wrote: > On Mon, Oct 08, 2018 at 12:34:19PM +0200, Martin Liška wrote: gcc/testsuite/g++.dg/ext/mv*.C >>> >>> You cannot move C++ tests to gcc.target/ >> >> I realized that we don't have ./gcc/testsuite/g++.target/i386 yet :/ > > But we want it and there

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Jakub Jelinek
On Mon, Oct 08, 2018 at 12:34:19PM +0200, Martin Liška wrote: > >> gcc/testsuite/g++.dg/ext/mv*.C > > > > You cannot move C++ tests to gcc.target/ > > I realized that we don't have ./gcc/testsuite/g++.target/i386 yet :/ But we want it and there are multiple tests waiting for that move.

Re: [PATCH] Provide extension hint for aarch64 target (PR driver/83193).

2018-10-08 Thread Martin Liška
Hi. I'm attaching updated version of the patch. Martin >From d36974540cda9fb0e159103fdcf92d26ef2f1b94 Mon Sep 17 00:00:00 2001 From: marxin Date: Thu, 4 Oct 2018 16:31:49 +0200 Subject: [PATCH] Provide extension hint for aarch64 target (PR driver/83193). gcc/ChangeLog: 2018-10-05 Martin

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Martin Liška
On 10/8/18 12:29 PM, Richard Biener wrote: > On Mon, Oct 8, 2018 at 12:14 PM Martin Liška wrote: >> >> On 10/8/18 11:55 AM, Richard Earnshaw (lists) wrote: >>> On 08/10/18 10:47, Martin Liška wrote: On 10/8/18 10:46 AM, Renlin Li wrote: > Hi Martin, > > pr82625.C failed on

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Richard Biener
On Mon, Oct 8, 2018 at 12:14 PM Martin Liška wrote: > > On 10/8/18 11:55 AM, Richard Earnshaw (lists) wrote: > > On 08/10/18 10:47, Martin Liška wrote: > >> On 10/8/18 10:46 AM, Renlin Li wrote: > >>> Hi Martin, > >>> > >>> pr82625.C failed on compiler builds which don't support "default" and >

Re: [patch] Fix PR tree-optimization/86659

2018-10-08 Thread Richard Biener
On Fri, Oct 5, 2018 at 10:29 AM Eric Botcazou wrote: > > > So I wonder why it is necessary to track 'reverse' in gimple_match_op > > at all given we bail out without optimizing as far as I can see? > > Because of the valueization? If it is done, gimple_simplify returns true so > the result will

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Martin Liška
On 10/8/18 11:55 AM, Richard Earnshaw (lists) wrote: > On 08/10/18 10:47, Martin Liška wrote: >> On 10/8/18 10:46 AM, Renlin Li wrote: >>> Hi Martin, >>> >>> pr82625.C failed on compiler builds which don't support "default" and "avx" >>> target. >>> For example, arm/aarch64 native linux gcc

Re: [PATCH v3] Change default to -fno-math-errno

2018-10-08 Thread Richard Biener
On Thu, Oct 4, 2018 at 9:43 PM Joseph Myers wrote: > > On Thu, 4 Oct 2018, Jeff Law wrote: > > > > I doubt you could prove that without LTO of the whole program because an > > > errno value set by a libm function call could always be checked in the > > > caller of whatever function is being

Re: [PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-10-08 Thread Richard Biener
On Thu, Oct 4, 2018 at 5:51 PM Martin Sebor wrote: > > On 10/04/2018 08:58 AM, Jeff Law wrote: > > On 8/27/18 9:42 AM, Richard Biener wrote: > >> On Mon, Aug 27, 2018 at 5:32 PM Jeff Law wrote: > >>> > >>> On 08/27/2018 02:29 AM, Richard Biener wrote: > On Sun, Aug 26, 2018 at 7:26 AM Jeff

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Richard Earnshaw (lists)
On 08/10/18 10:47, Martin Liška wrote: > On 10/8/18 10:46 AM, Renlin Li wrote: >> Hi Martin, >> >> pr82625.C failed on compiler builds which don't support "default" and "avx" >> target. >> For example, arm/aarch64 native linux gcc compiler. >> >> >> As I found in this gcc wiki:

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Martin Liška
On 10/8/18 10:46 AM, Renlin Li wrote: > Hi Martin, > > pr82625.C failed on compiler builds which don't support "default" and "avx" > target. > For example, arm/aarch64 native linux gcc compiler. > > > As I found in this gcc wiki: https://gcc.gnu.org/wiki/FunctionMultiVersioning > ''' > This

Re: [PATCH] EfficiencySanitizer implementation.

2018-10-08 Thread Denis Khalikov
I assume that the compiler based instrumentation, should be more efficient than binary instrumentation. But, I was just interested in the process of implementation for that tool. Sorry for the noise. On 10/07/2018 11:03 AM, Richard Biener wrote: > On October 6, 2018 10:17:48 PM GMT+02:00, Denis

Re: sched2 priorities and replacements

2018-10-08 Thread Robin Dapp
ping, any insight on this? Regards Robin

Re: [PATCH 2/2 v3][IRA,LRA] Fix PR86939, IRA incorrectly creates an interference between a pseudo register and a hard register

2018-10-08 Thread Christophe Lyon
On Sat, 6 Oct 2018 at 04:38, Peter Bergner wrote: > > On 10/5/18 4:12 PM, Vladimir Makarov wrote: > > On 10/05/2018 04:00 PM, Peter Bergner wrote: > >> How about non_conflicting_reg_copy or non_conflicting_copy_insn? > > OK. I like the first name more. > > Ok, I committed the patch using the

Re: [Patch, fortran] PR87151 - allocating array of character

2018-10-08 Thread Dominique d'Humières
Hi Paul, Your patch works as expected. It also fixes the ICEs for the tests in pr80931 (and the test accidentally attached to pr83196). Thanks for the patch. Dominique

Re: [PATCH] Redirect call within specific target attribute among MV clones (PR ipa/82625).

2018-10-08 Thread Renlin Li
Hi Martin, pr82625.C failed on compiler builds which don't support "default" and "avx" target. For example, arm/aarch64 native linux gcc compiler. As I found in this gcc wiki: https://gcc.gnu.org/wiki/FunctionMultiVersioning ''' This support is available in GCC 4.8 and later. Support is only

Re: [patch] tighten toplevel guard on ibm-ldouble.c

2018-10-08 Thread Olivier Hainque
Hi Segher, > On 05 Oct 2018, at 11:35, Segher Boessenkool > wrote: > > I think it looks fine. Okay for trunk after some testing. Thanks for your feedback. I can't get a pristine mainline to build on our powerpc-linux host - SEGVs from the newly built cc1 during libgcc. Updated ulimits

Re: [wwwdocs] Add no_sanitize attribute

2018-10-08 Thread Martin Liška
On 10/7/18 2:15 PM, Gerald Pfeifer wrote: > On Wed, 18 Apr 2018, Martin Liška wrote: >> I would like to mention the attribute in GCC 8 changes. > > Index: htdocs/gcc-8/changes.html > === >> >> New no_sanitize attribute has

[PATCH] Minor cleanups / PR63155

2018-10-08 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-10-08 Richard Biener PR tree-optimization/63155 * tree-ssa-propagate.c (add_ssa_edge): Do cheap check first. (ssa_propagation_engine::ssa_propagate): Remove redundant bitmap bit