Re: [Patch, Fortran] PR25829: Asynchronous I/O (v2)

2018-06-05 Thread Steve Kargl
On Wed, Jun 06, 2018 at 08:34:57AM +0300, Janne Blomqvist wrote: > On Wed, Jun 6, 2018 at 3:43 AM, Jerry DeLisle wrote: > > > On 06/05/2018 06:58 AM, Rainer Orth wrote: > > > >> Hi Nicolas, > >> > >> Because they were originally intended for the gfortran test suite, but I > >>> couldn't run it th

[PATCH] PR libstdc++/86008 add std::quoted support for string_view

2018-06-05 Thread Jonathan Wakely
PR libstdc++/86008 * include/bits/quoted_string.h (_Quoted_string): Define new partial specialization. * include/std/iomanip (quoted(basic_string_view, C, C)): Define new overload. (operator<<(basic_ostream&, const _Quoted_string&)): Use valu

Re: [Patch, Fortran] PR25829: Asynchronous I/O (v2)

2018-06-05 Thread Janne Blomqvist
On Wed, Jun 6, 2018 at 3:43 AM, Jerry DeLisle wrote: > On 06/05/2018 06:58 AM, Rainer Orth wrote: > >> Hi Nicolas, >> >> Because they were originally intended for the gfortran test suite, but I >>> couldn't run it there because of libpthread. I will change the numbering >>> scheme. >>> >> >> the

Re: [PATCH, rs6000] Fix PR target/63177: Powerpc no-vfa-vect-depend-2.c and no-vfa-vect-depend-3.c failures

2018-06-05 Thread Peter Bergner
On 6/5/18 3:22 PM, Segher Boessenkool wrote: > Ah, that's the crux. Thanks. Add a comment to the code please? Like so? I tried placing it next to the mcpu=power8 clause, but it seems you cannot place comments there, as it creates syntax errors during the build. /* Common ASM definitions used

Re: libgcov-driver-system bootstrap failure

2018-06-05 Thread David Edelsohn
Also, there seems to be a find-and-replace mistake in libgcov-driver.c #if !GCOV_LOCKED /* summary for all instances of program. */ struct gcov_summary *cs_all; #endif cs_all is declared within the !GCOV_LOCKED macro but later all references have been changed to "all", not "cs_all" #if !GC

Re: [PATCH (for gdb)] enum-flags.h: Add trailing semicolon to example in comment

2018-06-05 Thread Eric Gallager
On 6/5/18, David Malcolm wrote: > On Tue, 2018-06-05 at 17:13 +0100, Pedro Alves wrote: >> On 06/05/2018 03:49 PM, David Malcolm wrote: >> > On Tue, 2018-06-05 at 04:40 -0400, Trevor Saunders wrote: >> > > You may want to look at gdb's enum-flags.h which I think already >> > > implements what your

Re: [Patch, Fortran] PR25829: Asynchronous I/O (v2)

2018-06-05 Thread Jerry DeLisle
On 06/05/2018 06:58 AM, Rainer Orth wrote: Hi Nicolas, Because they were originally intended for the gfortran test suite, but I couldn't run it there because of libpthread. I will change the numbering scheme. the way that libpthread dependency is currently handled seems weird to me: right now

Re: [PATCH] add support for strnlen (PR 81384)

2018-06-05 Thread Eric Gallager
On 6/5/18, Martin Sebor wrote: > The attached patch adds basic support for handling strnlen > as a built-in function. It touches the strlen pass where > it folds constant results of the function, and builtins.c > to add simple support for expanding strnlen calls with known > results. It also cha

libgcov-driver-system bootstrap failure

2018-06-05 Thread David Edelsohn
Martin, With your recent patch, I receive the following failure building libgcov-driver-system.c In file included from /nasfarm/edelsohn/src/src/gcc/../libgcc/libgcov-driver.c:322:0: /nasfarm/edelsohn/src/src/gcc/../libgcc/libgcov-driver-system.c: In function 'char* replace_filename_variables(cha

Re: [PATCH v2, #3 of 3], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Michael Meissner
This patch makes the long double pack/unpack built-in functions only available if the long double format is IBM extended double. I added new pack and unpack functions that use __ibm128 instead of long double in case somebody needs to build IBM extended double after we have changed the default to I

Re: [PATCH v2, #2 of 3], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Michael Meissner
This patch makes __ibm128 use the long double type node if long double uses the IBM extended double representation, and similarly __float128 will use the long double type node if long double uses the IEEE 128-bit representation. I have fixed the pr85657.C test case to only test templates, and not

Re: [PATCH] Simplify gcov_histogram as it's used only for ARCS counters.

2018-06-05 Thread Jim Wilson
On 05/29/2018 06:08 AM, Martin Liška wrote: libgcc/ChangeLog: 2018-05-18 Martin Liska * libgcov-driver.c (gcov_compute_histogram): Remove usage of gcov_ctr_summary. (compute_summary): Do it just for a single summary. (merge_one_data): Likewise. (merge_

[PATCH] update Zero-length array documentation

2018-06-05 Thread Martin Sebor
Following the brief discussion Re: aliasing between internal zero-length-arrays and other members(*) the attached patch updates the documentation of Zero-length arrays to clarify that the only valid accesses are to those where the array is the last member of a struct. I also took the liberty to m

Re: [PATCH] add support for strnlen (PR 81384)

2018-06-05 Thread Jakub Jelinek
On Tue, Jun 05, 2018 at 03:43:17PM -0600, Martin Sebor wrote: > --- a/gcc/builtins.def > +++ b/gcc/builtins.def > @@ -733,6 +733,7 @@ DEF_EXT_LIB_BUILTIN(BUILT_IN_STRNCASECMP, > "strncasecmp", BT_FN_INT_CONST_STR > DEF_LIB_BUILTIN(BUILT_IN_STRNCAT, "strncat", > BT_FN_STRING_STRING_CO

Re: [PATCH,rs6000] Fix vec_permxor builtin support, fix test cases for vec_permxor and vec_insert4b

2018-06-05 Thread Bill Schmidt
> On Jun 5, 2018, at 5:04 PM, Carl Love wrote: > > On Tue, 2018-06-05 at 16:45 -0500, Bill Schmidt wrote: >> Hi Carl, >> >> That looks like a typo in the ABI document to me. The return type >> should match the >> argument types like it does for the other variants. Sorry -- I'll >> open a bug

Re: [PATCH,rs6000] Fix vec_permxor builtin support, fix test cases for vec_permxor and vec_insert4b

2018-06-05 Thread Carl Love
On Tue, 2018-06-05 at 16:45 -0500, Bill Schmidt wrote: > Hi Carl, > > That looks like a typo in the ABI document to me.  The return type > should match the > argument types like it does for the other variants.  Sorry -- I'll > open a bug against > the ABI doc. So, the ABI doc currently says:

Re: [PATCH,rs6000] Fix vec_permxor builtin support, fix test cases for vec_permxor and vec_insert4b

2018-06-05 Thread Bill Schmidt
Hi Carl, That looks like a typo in the ABI document to me. The return type should match the argument types like it does for the other variants. Sorry -- I'll open a bug against the ABI doc. Thanks! Good catch, Segher. -- Bill Bill Schmidt, Ph.D. STSM, GCC Architect for Linux on Power IBM L

[PATCH] add support for strnlen (PR 81384)

2018-06-05 Thread Martin Sebor
The attached patch adds basic support for handling strnlen as a built-in function. It touches the strlen pass where it folds constant results of the function, and builtins.c to add simple support for expanding strnlen calls with known results. It also changes calls.c to detect excessive bounds t

Re: [PATCH,rs6000] Fix vec_permxor builtin support, fix test cases for vec_permxor and vec_insert4b

2018-06-05 Thread Segher Boessenkool
Hi Carl, On Tue, Jun 05, 2018 at 01:57:52PM -0700, Carl Love wrote: > The following patch fixes the return type for the existing vec_permxor > builtin to match the ABI specification.  The test case for the builtin > is also updates. Hrm, is that a bug in the ABI doc though? Bill? Most older bui

Re: [PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Paul Koning
kes sense to skip that case). That simplifies the patch, which now looks like this. Ok for trunk? paul gcc/ChangeLog: 2018-06-05 Paul Koning * compare-elim.c (try_merge_compare): Don't merge compare if

[PATCH,rs6000] Fix vec_permxor builtin support, fix test cases for vec_permxor and vec_insert4b

2018-06-05 Thread Carl Love
GCC Maintainers: The following patch fixes the return type for the existing vec_permxor builtin to match the ABI specification.  The test case for the builtin is also updates. Secondly, the first argument of the vec_insert4b() builtin test case is fixed to match the ABI specification for the buil

libgo patch committed: Canonicalize types returned by StructOf, et. al.

2018-06-05 Thread Ian Lance Taylor
This libgo patch by Massimiliano Ghilardi changes the reflect package to canonicalize types returned by StructOf() and friends. Since gccgo does not currently merge identical types at link time, the reflect function canonicalize() exists to choose a canonical specimen for each set of identical typ

Re: [PATCH, rs6000] Fix PR target/63177: Powerpc no-vfa-vect-depend-2.c and no-vfa-vect-depend-3.c failures

2018-06-05 Thread Segher Boessenkool
On Tue, Jun 05, 2018 at 12:26:04PM -0500, Peter Bergner wrote: > >> -%{mcpu=power8: %(asm_cpu_power8)} \ > >> +%{mcpu=power8: %{!mpower9-vector: %(asm_cpu_power8)}} \ > >> %{mcpu=power9: %(asm_cpu_power9)} \ > >> %{mcpu=a2: -ma2} \ > >> %{mcpu=powerpc: -mppc} \ > >> @@ -169,6 +169,7 @@ > >> %{m

PING [PATCH] restore -Wstringop-overflow for checked strcpy/strcat (PR 85259)

2018-06-05 Thread Martin Sebor
Ping: https://gcc.gnu.org/ml/gcc-patches/2018-05/msg00652.html On 05/14/2018 04:41 PM, Martin Sebor wrote: r256683 committed to GCC 8 to avoiding duplicate instances of -Wstringop-overflow warnings on some targets has the unintended side-effect of suppressing even singleton instances of the warn

Re: [PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Richard Sandiford
Paul Koning writes: > This fixes an ICE if post-reload compare elimination is done and the > target supports post_inc and similar modes, as pdp11 does. The ICE is > caused by a generated PARALLEL that has the side effect twice, which > is not legal. > > (Ideally it would generate a similar RTL su

Re: [Patch][Aarch64][PR target/79924] Cannot translate diagnostics

2018-06-05 Thread Steve Ellcey
On Tue, 2018-06-05 at 20:18 +0100, Richard Sandiford wrote: >  > This still has the change from "floating-point" to "floating point", > but "floating-point" preferred. > > Thanks, > Richard You are right, I missed that.  I haven't checked in the patch yet though so I will fix that before doing th

Re: [PATCH, rs6000 8/9] enable gimple folding for vec_xl, vec_xst

2018-06-05 Thread Segher Boessenkool
On Thu, May 31, 2018 at 02:59:40PM -0500, Will Schmidt wrote: > 2018-05-31 Will Schmidt > > * config/rs6000/rs6000.c: (rs6000_builtin_valid_without_lhs) Add vec_xst > variants to the list. (rs6000_gimple_fold_builtin) Add support for > folding unaligned vector loads and sto

Re: [Patch][Aarch64][PR target/79924] Cannot translate diagnostics

2018-06-05 Thread Richard Sandiford
Steve Ellcey writes: > On Tue, 2018-06-05 at 13:23 +0100, Richard Sandiford wrote: >>  >> This regresses a couple of things: >> >> - before the patch, the option would be properly quoted, whereas now >>   it's unquoted.  Either keeping %qs or using %<...%> would fix that. >>   Using %qs is probab

Re: [PATCH, rs6000 9/9] Enable some additional combinations for builtin_vec_xst

2018-06-05 Thread Segher Boessenkool
On Thu, May 31, 2018 at 02:59:52PM -0500, Will Schmidt wrote: > 2018-05-31 Will Schmidt > > * config/rs6000/rs6000-c.c: Add BUILTIN_VEC_XST entries for *double > and *long long. * config/rs6000/rs6000-c.c (altivec_overloaded_builtins): Add BUILTIN_VEC_XST entrie

Re: [PATCH, rs6000] Clean up implementation of built-in functions

2018-06-05 Thread Segher Boessenkool
Hi Kelvin, On Fri, Jun 01, 2018 at 01:22:09PM -0500, Kelvin Nilsen wrote: > 2018-06-01 Kelvin Nilsen > > * config/rs6000/rs6000-builtin.def (VSX_BUILTIN_VEC_LD, > VSX_BUILTIN_VEC_ST): Add comment to explain non-traditional uses. > * config/rs6000/rs6000-c.c (altivec_overloade

Re: [Patch][Aarch64][PR target/79924] Cannot translate diagnostics

2018-06-05 Thread James Greenhalgh
On Tue, Jun 05, 2018 at 12:40:01PM -0500, Steve Ellcey wrote: > On Tue, 2018-06-05 at 13:23 +0100, Richard Sandiford wrote: > >  > > This regresses a couple of things: > > > > - before the patch, the option would be properly quoted, whereas now > >   it's unquoted.  Either keeping %qs or using %<.

[PATCH, i386]: Fix several "operand missing mode" build warnings in i386.md

2018-06-05 Thread Uros Bizjak
No functional changes. 2018-06-05 Uros Bizjak * config/i386/i386.md (simple_return_indirect_internal): New expander. (*simple_return_indirect_internal): Rename from simple_return_indirect_internal. Use W mode iterator. (rstorssp): New expander. (*rstorssp): Rename from rst

Re: [PATCH v2, #1 of 3], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Michael Meissner
This patch breaks up the previous patch. It renames the boolean, and eliminates the switch. This is patch #1 of 3. It generates a weak reference for the mangled name generated in GCC 8.1 to point to the new name that will be in future releases (i.e. U10__float128 -> u9__ieee128). Patch #2 will

[PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Paul Koning
* compare-elim.c (addr_side_effect_check): New function. (addr_side_effect_p): Ditto. (try_merge_compare): Don't merge compare if address contains a side effect. * gcc.c-torture/compile/20180605-1.c: New test case. Index: compare-e

Re: [PATCH][AArch64] Support for LDP/STP of Q-registers

2018-06-05 Thread Dr. Philipp Tomsich
> On 5 Jun 2018, at 19:28, James Greenhalgh wrote: > > On Tue, Jun 05, 2018 at 11:32:06AM -0500, Kyrill Tkachov wrote: >> >> On 04/06/18 18:40, Kyrill Tkachov wrote: >>> Hi all, >>> >>> This patch adds support for generating LDPs and STPs of Q-registers. >>> This allows for more compact code g

Re: [Patch][Aarch64][PR target/79924] Cannot translate diagnostics

2018-06-05 Thread Steve Ellcey
On Tue, 2018-06-05 at 13:23 +0100, Richard Sandiford wrote: >  > This regresses a couple of things: > > - before the patch, the option would be properly quoted, whereas now >   it's unquoted.  Either keeping %qs or using %<...%> would fix that. >   Using %qs is probably nicer since we can reuse th

Re: [PATCH] PR target/85358: Add target hook to prevent default widening

2018-06-05 Thread Joseph Myers
On Tue, 5 Jun 2018, Richard Biener wrote: > Are there other targets with similar kind of (FP) modes or is ppc the > only one with this 'legacy'? Also keep in mind that you intended to > backport this... Well, ia64 RFmode probably isn't a sensible mode to include in widening either (it's specif

Re: [PATCH (for gdb)] enum-flags.h: Add trailing semicolon to example in comment

2018-06-05 Thread Pedro Alves
[adding gdb-patches] On 06/05/2018 06:56 PM, David Malcolm wrote: > On Tue, 2018-06-05 at 17:13 +0100, Pedro Alves wrote: >> On 06/05/2018 03:49 PM, David Malcolm wrote: >>> On Tue, 2018-06-05 at 04:40 -0400, Trevor Saunders wrote: You may want to look at gdb's enum-flags.h which I think alre

Re: [PATCH][AArch64] Support for LDP/STP of Q-registers

2018-06-05 Thread James Greenhalgh
On Tue, Jun 05, 2018 at 11:32:06AM -0500, Kyrill Tkachov wrote: > > On 04/06/18 18:40, Kyrill Tkachov wrote: > > Hi all, > > > > This patch adds support for generating LDPs and STPs of Q-registers. > > This allows for more compact code generation and makes better use of the > > ISA. > > > > It's

Re: [PATCH] PR target/85358: Add target hook to prevent default widening

2018-06-05 Thread Richard Biener
On June 5, 2018 6:50:22 PM GMT+02:00, Segher Boessenkool wrote: >On Fri, Jun 01, 2018 at 04:36:54PM -0400, Michael Meissner wrote: >> As I mentioned previously, the initialization process needs to go >through all >> of the widening tables in order to initialize all FP types, so we >can't just >>

Re: [PATCH, rs6000] Fix PR target/63177: Powerpc no-vfa-vect-depend-2.c and no-vfa-vect-depend-3.c failures

2018-06-05 Thread Peter Bergner
On 6/5/18 11:23 AM, Segher Boessenkool wrote: > On Mon, Jun 04, 2018 at 08:57:24PM -0500, Peter Bergner wrote: >> The fix used here is to catch the special case when we use -mpower9-vector >> and >> -mcpu=power8 together and then force ourselves to use the -mpower9 gas >> option. > > Ideally -mp

Re: [PATCH][AArch64] Support for LDP/STP of Q-registers

2018-06-05 Thread Siddhesh Poyarekar
On 06/05/2018 10:02 PM, Kyrill Tkachov wrote: Adding some folks who know more about other CPUs as well. Are you okay with enabling these instructions in AArch64? If you could give this a spin on some benchmarks you care about on your platforms it would be really useful data. Sameera had writte

[PATCH (for gdb)] enum-flags.h: Add trailing semicolon to example in comment

2018-06-05 Thread David Malcolm
On Tue, 2018-06-05 at 17:13 +0100, Pedro Alves wrote: > On 06/05/2018 03:49 PM, David Malcolm wrote: > > On Tue, 2018-06-05 at 04:40 -0400, Trevor Saunders wrote: > > > You may want to look at gdb's enum-flags.h which I think already > > > implements what your doing here. > > > > Aha! Thanks! > >

[PATCH] Avoid optimizing memory references with side effects in compare-elim.c

2018-06-05 Thread Paul Koning
This fixes an ICE if post-reload compare elimination is done and the target supports post_inc and similar modes, as pdp11 does. The ICE is caused by a generated PARALLEL that has the side effect twice, which is not legal. (Ideally it would generate a similar RTL suitable for a matching constrai

Re: [PATCH] PR target/85358: Add target hook to prevent default widening

2018-06-05 Thread Segher Boessenkool
On Fri, Jun 01, 2018 at 04:36:54PM -0400, Michael Meissner wrote: > As I mentioned previously, the initialization process needs to go through all > of the widening tables in order to initialize all FP types, so we can't just > arbitrarily eliminate IFmode from the widening table. Or change the ini

Re: [PATCH][AArch64] Support for LDP/STP of Q-registers

2018-06-05 Thread Andrew Pinski
On Tue, Jun 5, 2018 at 9:32 AM Kyrill Tkachov wrote: > > > On 04/06/18 18:40, Kyrill Tkachov wrote: > > Hi all, > > > > This patch adds support for generating LDPs and STPs of Q-registers. > > This allows for more compact code generation and makes better use of the > > ISA. > > > > It's implement

Re: [PATCH][AArch64] Support for LDP/STP of Q-registers

2018-06-05 Thread Kyrill Tkachov
On 04/06/18 18:40, Kyrill Tkachov wrote: Hi all, This patch adds support for generating LDPs and STPs of Q-registers. This allows for more compact code generation and makes better use of the ISA. It's implemented in a straightforward way by allowing 16-byte modes in the sched-fusion machinery

Re: [PATCH], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Joseph Myers
On Tue, 5 Jun 2018, Jakub Jelinek wrote: > On Tue, Jun 05, 2018 at 04:08:13PM +, Joseph Myers wrote: > > On Tue, 5 Jun 2018, Jakub Jelinek wrote: > > > > > If we need the C separate _FloatNN types in C++, they can mangle the way > > > we > > > (and others) agree on. Would _Float64 be distin

Re: [PATCH, rs6000] Fix PR target/63177: Powerpc no-vfa-vect-depend-2.c and no-vfa-vect-depend-3.c failures

2018-06-05 Thread Segher Boessenkool
On Mon, Jun 04, 2018 at 08:57:24PM -0500, Peter Bergner wrote: > PR63177 shows a bug in how we determine which gas options we decide to pass > to the > assembler. Normally, we pass the -m option to the assembler if we used > the > -mcpu= option. However, if we don't compile with -mcpu=, then we

Re: Sharing gdb's enum-flags.h with gcc? (was Re: [PATCH 01/10] Convert dump and optgroup flags to enums)

2018-06-05 Thread Richard Biener
On June 5, 2018 4:49:21 PM GMT+02:00, David Malcolm wrote: >On Tue, 2018-06-05 at 04:40 -0400, Trevor Saunders wrote: >> On Fri, Jun 01, 2018 at 12:00:09PM +0200, Richard Biener wrote: >> > On Tue, May 29, 2018 at 10:32 PM David Malcolm > > > wrote: >> > > >> > > The dump machinery uses "int" in

Re: [PATCH], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Jakub Jelinek
On Tue, Jun 05, 2018 at 04:08:13PM +, Joseph Myers wrote: > On Tue, 5 Jun 2018, Jakub Jelinek wrote: > > > If we need the C separate _FloatNN types in C++, they can mangle the way we > > (and others) agree on. Would _Float64 be distinct from double and thus > > mangle differently from "d"? >

Re: Sharing gdb's enum-flags.h with gcc? (was Re: [PATCH 01/10] Convert dump and optgroup flags to enums)

2018-06-05 Thread Pedro Alves
On 06/05/2018 03:49 PM, David Malcolm wrote: > On Tue, 2018-06-05 at 04:40 -0400, Trevor Saunders wrote: >> You may want to look at gdb's enum-flags.h which I think already >> implements what your doing here. > > Aha! Thanks! > > Browsing the git web UI, that gdb header was introduced by Pedro

Re: [PATCH], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Joseph Myers
On Tue, 5 Jun 2018, Jakub Jelinek wrote: > If we need the C separate _FloatNN types in C++, they can mangle the way we > (and others) agree on. Would _Float64 be distinct from double and thus > mangle differently from "d"? Right now it (as accessed via __typeof on built-in functions since the _

Sharing gdb's enum-flags.h with gcc? (was Re: [PATCH 01/10] Convert dump and optgroup flags to enums)

2018-06-05 Thread David Malcolm
On Tue, 2018-06-05 at 04:40 -0400, Trevor Saunders wrote: > On Fri, Jun 01, 2018 at 12:00:09PM +0200, Richard Biener wrote: > > On Tue, May 29, 2018 at 10:32 PM David Malcolm > > wrote: > > > > > > The dump machinery uses "int" in a few places, for two different > > > sets of bitmasks. > > > > >

[PATCH][RFC] Drop GENERIC conds in [VEC_]COND_EXPRs

2018-06-05 Thread Richard Biener
This is a prototype enforcing is_gimple_val conditions in [VEC_]COND_EXPRs. It shows quite some fallout - some likely due to necessary testsuite adjustments like gcc.dg/tree-ssa/ssa-lim-12.c others because some code simply doesn't handle conditions visible only via following SSA defs like vect_

Re: [PATCH, S390] Change mtune default

2018-06-05 Thread Andreas Krebbel
On 06/04/2018 02:26 PM, Robin Dapp wrote: > Hi, > > when building with --with-tune=zEC12 and calling the resulting gcc with > --march=z13 (no extra -mtune), the binary would unexpectedly be compiled > with -march=z13 -mtune=zEC12. This patch avoids using the default tune > parameter if -march is

[patch] Improve specs processing to allow %* in function arguments

2018-06-05 Thread Olivier Hainque
Hello, The attached patch is a proposal to improve specs processing so %* works in spec function arguments (it doesn't as of today). The immediate motivation is to allow a cleaner implementation of the -mmacosx-version-min support on darwin. I'll send a followup patch for that if the preliminary

[Committed] S/390: Fix __builtin_tbeginc signature

2018-06-05 Thread Andreas Krebbel
The builtin was accidentally defined to have an integer return value. Fixed with the attached patch. Bootstrapped and regression tested on s390x. Applied to mainline, 8, 7, and 6 branches. gcc/ChangeLog: 2018-06-05 Andreas Krebbel * config/s390/s390-builtin-types.def: Add void functi

Re: [Patch, Fortran] PR25829: Asynchronous I/O (v2)

2018-06-05 Thread Rainer Orth
Hi Nicolas, > Because they were originally intended for the gfortran test suite, but I > couldn't run it there because of libpthread. I will change the numbering > scheme. the way that libpthread dependency is currently handled seems weird to me: right now it is only dragged in via -fopenmp, alth

Re: [PATCH] Reject function and subroutine arguments in OpenACC declare data clauses (PR85701)

2018-06-05 Thread Jakub Jelinek
On Tue, Jun 05, 2018 at 06:53:43AM -0700, Cesar Philippidis wrote: > OpenACC declare is implemented somewhat special in the Fortran FE in > that it preforms its own data clause error handling. As PR85701 > demonstrated, one situation that lacked test coverage was passing in > function and subroutin

[PATCH] Reject function and subroutine arguments in OpenACC declare data clauses (PR85701)

2018-06-05 Thread Cesar Philippidis
OpenACC declare is implemented somewhat special in the Fortran FE in that it preforms its own data clause error handling. As PR85701 demonstrated, one situation that lacked test coverage was passing in function and subroutine identifiers as data clause arguments to acc declare. This patch rectifies

[PATCH] Optimize location expansion in lto type location compare

2018-06-05 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-06-05 Richard Biener * lto.c (cmp_type_location): Expand locations only once. Index: gcc/lto/lto.c === --- gcc/lto/lto.c (revision 26119

Re: [PATCH], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Jakub Jelinek
On Tue, Jun 05, 2018 at 03:01:23PM +0200, Jason Merrill wrote: > > Now that we only have two types, you can't have explicit overloading of the > > same type. You can have templates, because only one version of the > > template is > > created for the two types. > > I thought that the two long dou

Re: [PATCH], Add weak references to bridge old/new PowerPC ieee 128-bit name mangling

2018-06-05 Thread Jason Merrill
On Tue, Jun 5, 2018 at 12:05 AM, Michael Meissner wrote: > On Mon, Jun 04, 2018 at 12:46:42PM -0500, Segher Boessenkool wrote: >> Hi Mike, >> >> On Fri, Jun 01, 2018 at 07:28:40PM -0400, Michael Meissner wrote: >> > This patch also makes __ibm128 or __float128 use the long double mode if >> > lon

Re: C++ PATCH for c++/85976, ICE with USING_DECL in cp_tree_equal

2018-06-05 Thread Jason Merrill
On Mon, Jun 4, 2018 at 11:03 PM, Marek Polacek wrote: > On Mon, Jun 04, 2018 at 01:28:01PM -0400, Jason Merrill wrote: >> On Mon, Jun 4, 2018 at 11:44 AM, Marek Polacek wrote: >> > I've had no luck in reducing the testcase in this PR, creduce won't get >> > even >> > past the initial passes, and

Re: [Patch][Aarch64][PR target/79924] Cannot translate diagnostics

2018-06-05 Thread Richard Sandiford
Steve Ellcey writes: > @@ -1029,13 +1029,18 @@ aarch64_gen_far_branch (rtx * operands, int > pos_label, const char * dest, > } > > void > -aarch64_err_no_fpadvsimd (machine_mode mode, const char *msg) > +aarch64_err_no_fpadvsimd (machine_mode mode) > { > - const char *mc = FLOAT_MODE_P (mod

Re: [PATCH] Support variables in expansion of -fprofile-generate option (PR gcov-profile/47618).

2018-06-05 Thread Martin Liška
Installed as r261199. Martin

[gomp5] const qualified vars no longer predetermined shared

2018-06-05 Thread Jakub Jelinek
Hi! This changed in OpenMP 4.0 already by mistake and I've kept the 3.1 and earlier behavior in the hopes that future versions will undo that mistake, but after discussions it has been determined we are going to keep the 4.0+ behavior. In most cases it is just about slightly different diagnostics

C++ PATCH for c++/85731, error with qualified-id in template

2018-06-05 Thread Jason Merrill
The problem here was that finish_qualified_id_expr was throwing away the nested-name-specifier, which is necessary to avoid this error. Tested x86_64-pc-linux-gnu, applying to trunk and 8. commit b96ede757f83783d388623b26312a0514bfec8a6 Author: Jason Merrill Date: Tue Jun 5 01:05:09 2018 +0200

Re: [PATCH] Print working directory only in intermediate format (PR gcov-profile/84846).

2018-06-05 Thread Eric Botcazou
> I've just installed the patch as r261188. Great, thanks! -- Eric Botcazou

RE: [patch][i386] Tremont -march/-mtune options

2018-06-05 Thread Makhotina, Olga
No, it's not needed. Thanks. -Original Message- From: Uros Bizjak [mailto:ubiz...@gmail.com] Sent: Tuesday, June 5, 2018 11:52 AM To: Makhotina, Olga Cc: gcc-patches@gcc.gnu.org Subject: Re: [patch][i386] Tremont -march/-mtune options On Mon, Jun 4, 2018 at 1:48 PM, Makhotina, Olga w

Re: [ARM][PATCH] Make arm_cmse.h C99 compatible

2018-06-05 Thread Kyrill Tkachov
On 05/06/18 11:03, Andre Vieira (lists) wrote: On 05/06/18 10:42, Kyrill Tkachov wrote: Hi Andre, On 05/06/18 10:40, Andre Vieira (lists) wrote: Hi all, This patch makes the arm_cmse.h header file ISO C compliant. We were using 'typeof' before when we should be using '__typeof__'. Duplicate

Re: [ARM][PATCH] Make arm_cmse.h C99 compatible

2018-06-05 Thread Andre Vieira (lists)
On 05/06/18 10:42, Kyrill Tkachov wrote: > Hi Andre, > > On 05/06/18 10:40, Andre Vieira (lists) wrote: >> Hi all, >> >> This patch makes the arm_cmse.h header file ISO C compliant. We were >> using 'typeof' before when we should be using '__typeof__'. >> Duplicated cmse-1.c with -std=c99 on the c

[PATCH] Fix PR86047

2018-06-05 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. >From 44d460d4e2737d89592d70475908000adf4775d6 Mon Sep 17 00:00:00 2001 From: Richard Guenther Date: Tue, 5 Jun 2018 10:27:25 +0200 Subject: [PATCH] fix-pr86047 PR tree-optimization/86047 * tree-ssa-loop.c

[PATCH] Fix PR86046

2018-06-05 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. >From 0001b7c690860b4bcd643c4431918bc5aed08d41 Mon Sep 17 00:00:00 2001 From: Richard Guenther Date: Tue, 5 Jun 2018 10:20:51 +0200 Subject: [PATCH] fix-pr86046 2018-06-05 Richard Biener PR tree-optimization/86

[PATCH] Avoid excessive location expansion in assign_discriminators

2018-06-05 Thread Richard Biener
On testcases like that from PR60243 CFG build is dominated by assign_discriminators because it expands locations again and again and this got more expensive over the time. Cary - can you explain the overall logic of assign_discriminators, specifically why if the last stmt of a block has UNKNOWN_

Re: [patch][i386] Tremont -march/-mtune options

2018-06-05 Thread Uros Bizjak
On Mon, Jun 4, 2018 at 1:48 PM, Makhotina, Olga wrote: > > Hi, > > This patch implements Tremont -march/-mtune. > > 2018-06-04 Olga Makhotina > > gcc/ > > * config.gcc: Support "tremont". > * config/i386/driver-i386.c (host_detect_local_cpu): Detect "tremont". > * config

[PATCH][arm] PR target/81497: Fix arm_acle.h for C++

2018-06-05 Thread Kyrill Tkachov
Hi all, When trying to compile something with arm_acle.h using G++ we get a number of nasty errors: arm_acle.h:48:49: error: invalid conversion from ‘const void*’ to ‘const int*’ [-fpermissive] return __builtin_arm_ldc (__coproc, __CRd, __p); This is because the intrinsics that are supposed

Re: [ARM][PATCH] Make arm_cmse.h C99 compatible

2018-06-05 Thread Kyrill Tkachov
Hi Andre, On 05/06/18 10:40, Andre Vieira (lists) wrote: Hi all, This patch makes the arm_cmse.h header file ISO C compliant. We were using 'typeof' before when we should be using '__typeof__'. Duplicated cmse-1.c with -std=c99 on the command line to test this. Ran the cmse tests for arm-none-

[ARM][PATCH] Make arm_cmse.h C99 compatible

2018-06-05 Thread Andre Vieira (lists)
Hi all, This patch makes the arm_cmse.h header file ISO C compliant. We were using 'typeof' before when we should be using '__typeof__'. Duplicated cmse-1.c with -std=c99 on the command line to test this. Ran the cmse tests for arm-none-eabi. Is this OK for trunk? Cheers, Andre gcc 2018-06-xx

Re: [PATCH 01/10] Convert dump and optgroup flags to enums

2018-06-05 Thread Trevor Saunders
On Fri, Jun 01, 2018 at 12:00:09PM +0200, Richard Biener wrote: > On Tue, May 29, 2018 at 10:32 PM David Malcolm wrote: > > > > The dump machinery uses "int" in a few places, for two different > > sets of bitmasks. > > > > This patch makes things more self-documenting and type-safe by using > > a

Re: [PATCH] Simplify gcov_histogram as it's used only for ARCS counters.

2018-06-05 Thread Martin Liška
Installed as r261189. Martin

Re: [PATCH] Print working directory only in intermediate format (PR gcov-profile/84846).

2018-06-05 Thread Martin Liška
On 06/04/2018 11:01 AM, Martin Liška wrote: > PING^1 > > On 05/25/2018 01:36 PM, Martin Liška wrote: >> Hi. >> >> As requested by Eric, let's print working directory just in intermediate >> format: >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84846#c8 >> >> gcov.exp tests works with the patch.

Re: [PATCH] PR libstdc++/85951 for make_signed/make_unsigned for character types

2018-06-05 Thread Jonathan Wakely
On 31/05/18 13:17 +0100, Jonathan Wakely wrote: Because the wide character types are neither signed integer types nor unsigned integer types they need to be transformed to an integral type of the correct size and the lowest rank (which is not necessarily the underlying type). Reuse the helpers fo

Re: [PATCH] PR libstdc++/78870 support std::filesystem on Windows

2018-06-05 Thread Jonathan Wakely
On 31/05/18 20:19 +0100, Jonathan Wakely wrote: This adds incomplete but functional support for std::filesystem and std::experimental::filesystem on MinGW. In theory there should be no changes to the existing behaviour for POSIX targets from this patch, as all the various bugs I found while worki