[Ada] Fix comment typo in exp_ch9.adb

2018-09-19 Thread Oliver Kellogg
Index: gcc/ada/ChangeLog === --- gcc/ada/ChangeLog (revision 264438) +++ gcc/ada/ChangeLog (working copy) @@ -1,3 +1,7 @@ +2018-09-20 Oliver Kellogg + + * exp_ch9.adb: Fix typo 'geenrated' to 'generated'. + 2018-09-13

Re: [PATCH] Remove arc profile histogram in non-LTO mode.

2018-09-19 Thread Bin.Cheng
On Thu, Sep 20, 2018 at 2:11 AM Martin Liška wrote: > > Hello. > > I've been working for some time on a patch that simplifies how we set > the hotness threshold of basic blocks. Currently, we calculate so called > arc profile histograms that should identify edges that cover 99.9% of all >

C++ PATCH to refine c++/87109 patch

2018-09-19 Thread Marek Polacek
Aaaand this addresses , as I promised earlier. I hope I got it right. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2018-09-19 Marek Polacek PR c++/87109 - wrong ctor with maybe-rvalue semantics. * call.c

[PATCH] hppa: Remove TARGET_SCHED_ADJUST_PRIORITY hook

2018-09-19 Thread John David Anglin
On 2018-09-06 1:05 PM, Jeff Law wrote: Anyway, the patch is fine. Your call if you want to just kill the code in the target file, it's just not terribly important anymore. The attached patch removes the TARGET_SCHED_ADJUST_PRIORITY hook. Tested on hppa-unknown-linux-gnu.  Committed to trunk.

Re: [committed] hppa: Revise atomic support to use sync barrier

2018-09-19 Thread John David Anglin
On 2018-08-11 6:17 PM, John David Anglin wrote: It recently came to my attention that PA 2.0 supports out-of-order execution for loads and stores.  Loads and stores are strongly ordered on PA 1.x.  This has caused no end of confusion. This out-of-order execution is discussed in the following

Re: C++ PATCH to add -Wclass-conversion

2018-09-19 Thread Jason Merrill
On Wed, Sep 19, 2018 at 4:25 PM, Marek Polacek wrote: > On Wed, Sep 19, 2018 at 04:14:25PM -0400, Jason Merrill wrote: >> On Wed, Sep 19, 2018 at 3:44 PM, Marek Polacek wrote: >> > On Wed, Sep 19, 2018 at 09:40:05PM +0200, Florian Weimer wrote: >> >> Marek Polacek writes: >> >> >> >> > +@item

Re: [PATCH 08/25] Fix co-array allocation

2018-09-19 Thread Damian Rouson
On Wed, Sep 19, 2018 at 3:30 PM Andrew Stubbs wrote: > > If you want to port your tool to GCN that would be cool, but I suspect > non-trivial. > To clarify, OpenCoarrays is not a tool. It is the parallel ABI required to create executable programs capable of executing in multiple images as

[PATCH,FORTRAN v2] Use stringpool on loading module symbols

2018-09-19 Thread Bernhard Reutner-Fischer
From: Bernhard Reutner-Fischer gcc/fortran/ChangeLog: 2018-09-19 Bernhard Reutner-Fischer * class.c (generate_finalization_wrapper, gfc_find_derived_vtab, find_intrinsic_vtab): Set module if in module context. * decl.c (gfc_match_decl_type_spec): Likewise.

Re: [PATCH 08/25] Fix co-array allocation

2018-09-19 Thread Andrew Stubbs
On 19/09/18 22:18, Damian Rouson wrote: Has this been tested in multi-image execution using OpenCoarrays?   If not, I would be glad to assist with installing OpenCoarrays so that it can be part of the testing process. It's been tested with the GCC testsuite -- the same suite that found the

Re: [openacc] Teach gfortran to lower OpenACC routine dims

2018-09-19 Thread Bernhard Reutner-Fischer
On Wed, 5 Sep 2018 12:52:03 -0700 Cesar Philippidis wrote: > At present, gfortran does not encode the gang, worker or vector > parallelism clauses when it creates acc routines dim attribute for > subroutines and functions. While support for acc routine is lacking in > other areas in gfortran

Re: [PATCH 08/25] Fix co-array allocation

2018-09-19 Thread Damian Rouson
Has this been tested in multi-image execution using OpenCoarrays? If not, I would be glad to assist with installing OpenCoarrays so that it can be part of the testing process. On a related note, two Sourcery Institute developers have attempted to edit the GCC build system to make the

Re: [PATCH] PR libstdc++/78179 run long double tests separately

2018-09-19 Thread Rainer Orth
Hi Christophe, > I have noticed failures on hypot-long-double.cc on arm, so I suggest we add: > > diff --git > a/libstdc++-v3/testsuite/26_numerics/headers/cmath/hypot-long-double.cc > b/libstdc++-v3/testsuite/26_numerics/headers/cmath/hypot-long-double.cc > index 8a05473..4c2e33b 100644 > ---

Re: [PATCH] PR libstdc++/78179 run long double tests separately

2018-09-19 Thread Christophe Lyon
On Mon, 3 Sep 2018 at 16:54, Jonathan Wakely wrote: > > Split the long double testing into a separate file, so that we can XFAIL > targets where the long double precision doesn't meet the expected > tolerances. The float and double tests are still expefted to PASS for > all targets. > >

Re: [PATCH 13/14] Add D Phobos standard library and license.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:38, Iain Buclaw wrote: > This patch add the Phobos runtime library and license (Boost) files. > Phobos is the standard runtime library that comes with the D language > compiler. The bulk of which is comprised mostly of generic algorithms > and high level primitives for

Re: [PATCH 12/14] Add GDC/GCC builtins and runtime support (part of D runtime)

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:37, Iain Buclaw wrote: > This patch adds GCC builtins and runtime support for GDC compiled code. > > - module __entrypoint defines the C main function. Its contents are > parsed and compiled in during compilation, but only if needed. > - module gcc.attribute

Re: [PATCH 11/14] D runtime library and license.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:37, Iain Buclaw wrote: > This patch adds the D runtime library and license (Boost) files. D > runtime is a low level that implements the building blocks of the > runtime environment, as well as C and C++ platform bindings. Many > high level operations are lowered to

Re: [PATCH 09/14] Add D2 Testsuite Dejagnu files.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:36, Iain Buclaw wrote: > > This patch adds D language support to the GCC testsuite. > > As well as generating the DejaGNU options for compile and link tests, > handles the conversion from DMD-style compiler options to GDC.

Re: [PATCH 07/14] Add patches for D language support in GCC targets.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:35, Iain Buclaw wrote: > > This patch add D language support to targets of GCC itself. > > These are used to declare pre-defined version identifiers in the D > language that describe something about the target that the front-end > itself is unable to obtain. Version

Re: C++ PATCH to add -Wclass-conversion

2018-09-19 Thread Marek Polacek
On Wed, Sep 19, 2018 at 04:14:25PM -0400, Jason Merrill wrote: > On Wed, Sep 19, 2018 at 3:44 PM, Marek Polacek wrote: > > On Wed, Sep 19, 2018 at 09:40:05PM +0200, Florian Weimer wrote: > >> Marek Polacek writes: > >> > >> > +@item -Wno-class-conversion @r{(C++ and Objective-C++ only)} > >> >

Re: [PATCH 06/14] Add patches for D language support in GCC proper.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:35, Iain Buclaw wrote: > > This patch adds D language support to GCC itself. > Likewise, this was approved so long as all prerequisites have been approved also. https://gcc.gnu.org/ml/gcc-patches/2017-09/msg00609.html Iain.

Re: [PATCH 05/14] Add GCC configuration file changes and documentation.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:35, Iain Buclaw wrote: > > This patch adds the D language front-end to GCC documentation and > configuration files, as described on the anatomy of a language front-end. > It looks like this was previously ok'd, with reason as it's mostly auto-generated boilerplate.

Re: C++ PATCH to add -Wclass-conversion

2018-09-19 Thread Jason Merrill
On Wed, Sep 19, 2018 at 3:44 PM, Marek Polacek wrote: > On Wed, Sep 19, 2018 at 09:40:05PM +0200, Florian Weimer wrote: >> Marek Polacek writes: >> >> > +@item -Wno-class-conversion @r{(C++ and Objective-C++ only)} >> > +@opindex Wno-class-conversion >> > +@opindex Wclass-conversion >> >

Re: [PATCH 04/14] Add D front-end (GDC) config, Makefile, and manpages.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:34, Iain Buclaw wrote: > > This patch adds the D frontend language configure make files, as described on > the anatomy of a language front-end. > This is where the last conversation ended up. https://gcc.gnu.org/ml/gcc-patches/2017-09/msg00612.html I've just noticed

Re: [PATCH 03/14] Add D frontend (GDC) changelogs.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:34, Iain Buclaw wrote: > This patch just includes all changelogs for the D front-end (GDC), > going back to the dawn of time itself. > https://gcc.gnu.org/ml/gcc-patches/2017-09/msg00734.html This was previously ok'd, if I understand right. Iain.

Re: [PATCH 02/14] Add D frontend (GDC) implementation.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:33, Iain Buclaw wrote: > This patch adds the D front-end implementation, the only part of the > compiler that interacts with GCC directly, and being the parts that I > maintain, is something that I can talk about more directly. > > For the actual code generation pass,

Re: C++ PATCH to add -Wclass-conversion

2018-09-19 Thread Marek Polacek
On Wed, Sep 19, 2018 at 09:40:05PM +0200, Florian Weimer wrote: > Marek Polacek writes: > > > +@item -Wno-class-conversion @r{(C++ and Objective-C++ only)} > > +@opindex Wno-class-conversion > > +@opindex Wclass-conversion > > +Disable the warning about user-defined conversions converting a

Re: C++ PATCH to add -Wclass-conversion

2018-09-19 Thread Florian Weimer
Marek Polacek writes: > +@item -Wno-class-conversion @r{(C++ and Objective-C++ only)} > +@opindex Wno-class-conversion > +@opindex Wclass-conversion > +Disable the warning about user-defined conversions converting a class to > +the wrong type, such as having a conversion function converting an >

C++ PATCH to add -Wclass-conversion

2018-09-19 Thread Marek Polacek
This patch adds a new warning -Wclass-conversion that is on by default, and warns about conversion functions converting a class to the wrong type. This was under the -Wconversion umbrella, but we should warn about this by default. It also makes the warning print the types in question.

Re: [PATCH] Add a dwarf unit type to represent 24 bit values.

2018-09-19 Thread John Darrington
Thank you for your insight. I will investigate further and see if I can find out what is going on. J' On Thu, Sep 13, 2018 at 01:33:34PM -0400, Jason Merrill wrote: Well, that's curious, given that GCC doesn't have that address type either. Ah, looking closer, I see that we

Re: [PATCH 01/14] Add D front-end (DMD) language implementation and license.

2018-09-19 Thread Iain Buclaw
On 18 September 2018 at 02:31, Iain Buclaw wrote: > This patch adds the DMD front-end proper and license (Boost) files, > comprised of a lexer, parser, and semantic analyzer. > > ftp://ftp.gdcproject.org/patches/v4/01-v4-d-frontend-dmd.patch > I've dug up the history of what was reviewed before,

[Patch, fortran] PR84109 - ICE in adjustl on allocatable array of strings

2018-09-19 Thread Paul Richard Thomas
As Dominique pointed out (and I denied initially - sorry!) this PR is essentially the same as PR87239 except that an intrinsic, rather than an extrinsic, function is involved. I have treated it in the same way but have added a slight improvement to keep scalar elemental calls outside of the

[PATCH v2] S/390: Fix conditional returns on z196+

2018-09-19 Thread Ilya Leoshkevich
S/390 epilogue ends with (parallel [(return) (use %r14)]) instead of the more usual (return) or (simple_return). This sequence is not recognized by the conditional return logic in try_optimize_cfg (). This was introduced for processors older than z196, where it is sometimes profitable to use

[PATCH] Remove arc profile histogram in non-LTO mode.

2018-09-19 Thread Martin Liška
Hello. I've been working for some time on a patch that simplifies how we set the hotness threshold of basic blocks. Currently, we calculate so called arc profile histograms that should identify edges that cover 99.9% of all branching. These edges are then identified as hot. Disadvantage of the

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

2018-09-19 Thread Martin Liška
PING^3 On 8/23/18 1:00 PM, Martin Liška wrote: PING^2 On 08/01/2018 03:56 PM, Martin Liška wrote: PING^1 On 07/18/2018 05:49 PM, Martin Liška wrote: Hi. This patch improves aarch64 feature modifier hints. May I please ask ARM folks to test the patch? Thanks, Martin gcc/ChangeLog:

[PATCH] combine: Use correct mode in new comparison (PR86902)

2018-09-19 Thread Segher Boessenkool
This code in try_combine uses the wrong mode. This fails (with RTL checking) in trunk, but not in any released branches. Committing to trunk. Segher 2018-09-19 Segher Boessenkool * combine.c (try_combine): When changing the CC mode used, don't change an unrelated mode in

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

2018-09-19 Thread Jason Merrill
On Wed, Sep 19, 2018 at 11:10 AM, Marek Polacek wrote: > On Wed, Sep 19, 2018 at 05:07:34PM +0200, Andreas Schwab wrote: >> On Sep 19 2018, Marek Polacek wrote: >> >> > 2018-09-19 Marek Polacek >> > >> > * class.c (build_vtbl_initializer): Don't mess with *inits for >> >

Re: [PATCH 09/25] Elide repeated RTL elements.

2018-09-19 Thread Andrew Stubbs
On 11/09/18 23:45, Jeff Law wrote: On 9/5/18 5:49 AM, a...@codesourcery.com wrote: GCN's 64-lane vectors tend to make RTL dumps very long. This patch makes them far more bearable by eliding long sequences of the same element into "repeated" messages. 2018-09-05 Andrew Stubbs

Re: [PATCH 08/25] Fix co-array allocation

2018-09-19 Thread Andrew Stubbs
On 05/09/18 19:07, Janne Blomqvist wrote: The argument must be of type size_type_node, not sizetype. Please instead use size = build_zero_cst (size_type_node); * trans-intrinsic.c (conv_intrinsic_event_query): Convert computed index to a size_t type. Using

Re: [PATCH] PR libstdc++/87135 Rehash only when necessary (LWG2156)

2018-09-19 Thread François Dumont
On 09/19/2018 01:07 PM, Jonathan Wakely wrote: On 18/09/18 22:39 +0200, François Dumont wrote: On 09/18/2018 10:41 AM, Jonathan Wakely wrote: On 13/09/18 07:49 +0200, François Dumont wrote: All changes limited to hashtable_c++0x.cc file. _Prime_rehash_policy::_M_next_bkt now really does what

Re: C++ PATCH for c++/87357, missing -Wconversion warning

2018-09-19 Thread Martin Sebor
On 09/19/2018 10:02 AM, Marek Polacek wrote: On Wed, Sep 19, 2018 at 09:51:44AM -0600, Martin Sebor wrote: On 09/18/2018 09:47 PM, Marek Polacek wrote: As I mentioned in the PR, [class.conv.fct] says "A conversion function is never used to convert a (possibly cv-qualified) object to the

Re: C++ PATCH for c++/87357, missing -Wconversion warning

2018-09-19 Thread Jason Merrill
On Tue, Sep 18, 2018 at 11:47 PM, Marek Polacek wrote: > As I mentioned in the PR, [class.conv.fct] says "A conversion function is > never > used to convert a (possibly cv-qualified) object to the (possibly > cv-qualified) > same object type (or a reference to it), to a (possibly cv-qualified)

Re: [C++ Patch] PR 87324 ("g++ ICE with overriding initializers: Segmentation fault")

2018-09-19 Thread Jason Merrill
OK. On Wed, Sep 19, 2018 at 10:42 AM, Paolo Carlini wrote: > Hi, > > in this simple error-recovery issue we ICE in get_class_binding_direct when > we try to use IDENTIFIER_CONV_OP_P on the index which we nulled in > cp_parser_initializer_list. If, instead, we assign error_mark_node to it we >

Re: C++ PATCH for c++/87357, missing -Wconversion warning

2018-09-19 Thread Marek Polacek
On Wed, Sep 19, 2018 at 09:51:44AM -0600, Martin Sebor wrote: > On 09/18/2018 09:47 PM, Marek Polacek wrote: > > As I mentioned in the PR, [class.conv.fct] says "A conversion function is > > never > > used to convert a (possibly cv-qualified) object to the (possibly > > cv-qualified) > > same

Re: C++ PATCH for c++/87357, missing -Wconversion warning

2018-09-19 Thread Martin Sebor
On 09/18/2018 09:47 PM, Marek Polacek wrote: As I mentioned in the PR, [class.conv.fct] says "A conversion function is never used to convert a (possibly cv-qualified) object to the (possibly cv-qualified) same object type (or a reference to it), to a (possibly cv-qualified) base class of that

[PATCH] Change EQ_ATTR_ALT to support up to 64 alternatives

2018-09-19 Thread Ilya Leoshkevich
On S/390 there is a need to support more than 32 instruction alternatives per define_insn. Currently this is not explicitly prohibited or unsupported: MAX_RECOG_ALTERNATIVES is equal 35, and, futhermore, the related code uses uint64_t for bitmaps in most places. However, genattrtab contains the

Re: [PATCH 03/25] Improve TARGET_MANGLE_DECL_ASSEMBLER_NAME.

2018-09-19 Thread Julian Brown
On Fri, 14 Sep 2018 22:49:35 -0400 Julian Brown wrote: > > > On 12/09/18 16:16, Richard Biener wrote: > > > It may well be that there's a better way to solve the problem, or > > > at least to do the lookups. > > > > > > It may also be that there are some unintended consequences, such > > >

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

2018-09-19 Thread Marek Polacek
On Wed, Sep 19, 2018 at 05:07:34PM +0200, Andreas Schwab wrote: > On Sep 19 2018, Marek Polacek wrote: > > > 2018-09-19 Marek Polacek > > > > * class.c (build_vtbl_initializer): Don't mess with *inits for > > TARGET_VTABLE_USES_DESCRIPTORS. > > > > diff --git gcc/cp/class.c

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

2018-09-19 Thread Andreas Schwab
On Sep 19 2018, Marek Polacek wrote: > 2018-09-19 Marek Polacek > > * class.c (build_vtbl_initializer): Don't mess with *inits for > TARGET_VTABLE_USES_DESCRIPTORS. > > diff --git gcc/cp/class.c gcc/cp/class.c > index 9ca46441871..0239f6ae045 100644 > --- gcc/cp/class.c > +++

Re: [PATCH][Middle-end][Version 3]Add a new option to control inlining only on static functions

2018-09-19 Thread Qing Zhao
thanks, Martin. > On Sep 18, 2018, at 5:26 PM, Martin Sebor wrote: >> >> gcc/ChangeLog >> >> +2018-09-18 Qing Zhao mailto:qing.z...@oracle.com>> >> + >> +* cif-code.def (FUNCTION_EXTERN): New CIFCODE. >> +* common.opt (-finline-only-static): New option. >> +* doc/invoke.texi:

Re: [Patch, Fortran, OOP] PR 46313: OOP-ABI issue, ALLOCATE issue, CLASS renaming issue

2018-09-19 Thread Bernhard Reutner-Fischer
On Mon, 17 Sep 2018 at 22:25, Janus Weil wrote: > The regtest was successful. I don't think the off-by-two error for the > vtab/vtype comparisons is a big problem in practice, since the number > of internal symbols with leading underscores is very limited, but of > course it should still be

[C++ Patch] PR 87324 ("g++ ICE with overriding initializers: Segmentation fault")

2018-09-19 Thread Paolo Carlini
Hi, in this simple error-recovery issue we ICE in get_class_binding_direct when we try to use IDENTIFIER_CONV_OP_P on the index which we nulled in cp_parser_initializer_list. If, instead, we assign error_mark_node to it we catch it later in reshape_init_class and error-recovery goes well.

Re: [PATCH,FORTRAN 00/29] Move towards stringpool, part 1

2018-09-19 Thread Bernhard Reutner-Fischer
On Fri, 7 Sep 2018 at 10:07, Bernhard Reutner-Fischer wrote: > > On Wed, 5 Sep 2018 at 20:57, Janne Blomqvist > wrote: > > > > On Wed, Sep 5, 2018 at 5:58 PM Bernhard Reutner-Fischer > > wrote: > > >> Bootstrapped and regtested on x86_64-foo-linux. > >> > >> I'd appreciate if someone could

Re: [PATCH][RFC] Fix PR63155 (some more)

2018-09-19 Thread Steven Bosscher
On Wed, Sep 19, 2018 at 3:06 PM Richard Biener wrote: > If we'd only had a O(log n) search sparse bitmap implementation ... > (Steven posted patches to switch bitmap from/to such one but IIRC > that at least lacked bitmap_first_set_bit). But bitmap_first_set_bit would be easy to implement. Just

Re: [PATCH] look harder for MEM_REF operand equality to avoid -Wstringop-truncation (PR 84561)

2018-09-19 Thread Martin Sebor
On 09/18/2018 10:23 PM, Jeff Law wrote: On 9/18/18 1:46 PM, Martin Sebor wrote: On 09/18/2018 12:58 PM, Jeff Law wrote: On 9/18/18 11:12 AM, Martin Sebor wrote: My bad. Sigh. CCP doesn't track copies, just constants, so there's not going to be any data structure you can exploit. And I

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

2018-09-19 Thread Marek Polacek
On Wed, Sep 19, 2018 at 03:25:02PM +0200, Andreas Schwab wrote: > I'm getting this ICE on ia64: > > libtool: compile: /usr/local/gcc/gcc-20180919/Build/./gcc/xgcc > -shared-libgcc -B/usr/local/gcc/gcc-20180919/Build/./gcc -nostdinc++ > -L/usr/local/gcc/gcc-20180919/Build/

Re: [PATCH][RFC] Fix PR63155 (some more)

2018-09-19 Thread Richard Biener
On Wed, 19 Sep 2018, Jakub Jelinek wrote: > On Wed, Sep 19, 2018 at 03:06:30PM +0200, Richard Biener wrote: > > > > The second testcase in the above PR runs into our O(N) bitmap element > > search limitation and spends 8s (60%) of the compile-time in the SSA > > propagator > > engine (when

Re: [PATCH] dumpfile.c: fix stray dump_loc output (PR tree-optimization/87309)

2018-09-19 Thread Richard Biener
On Wed, Sep 19, 2018 at 12:42 PM David Malcolm wrote: > > In r262891 I reimplemented this call: > dump_printf_loc (MSG_NOTE, loc, "=== %s ===\n", name); > in dump_begin_scope to use direct calls to dump_loc: > if (dump_file) > { > dump_loc (MSG_NOTE, dump_file, loc.get_location_t

Re: [PATCH] look harder for MEM_REF operand equality to avoid -Wstringop-truncation (PR 84561)

2018-09-19 Thread Richard Biener
On Tue, Sep 18, 2018 at 8:58 PM Jeff Law wrote: > > On 9/18/18 11:12 AM, Martin Sebor wrote: > > >> My bad. Sigh. CCP doesn't track copies, just constants, so there's not > >> going to be any data structure you can exploit. And I don't think > >> there's a value number you can use to determine

Re: [PATCH] dump_printf: use %T and %G throughout

2018-09-19 Thread Richard Biener
On Wed, Sep 19, 2018 at 1:18 AM David Malcolm wrote: > > As promised at Cauldron, this patch uses %T and %G with dump_printf and > dump_printf_loc calls to eliminate calls to > > dump_generic_expr (MSG_*, arg, TDF_SLIM) (via %T) > > and > > dump_gimple_stmt (MSG_*, TDF_SLIM, stmt, 0) (via

Re: [PATCH 13/25] Create TARGET_DISABLE_CURRENT_VECTOR_SIZE

2018-09-19 Thread Richard Biener
On Tue, Sep 18, 2018 at 10:22 PM Andrew Stubbs wrote: > > On 18/09/18 12:21, Richard Sandiford wrote: > > Would the same be useful for GCN, or do you basically always > > want a VF of 64? > > Always 64; the vector size varies between 512-bit and 4096-bit, as needed. > > > None of this is a

Re: [PATCH][RFC] Fix PR63155 (some more)

2018-09-19 Thread Jakub Jelinek
On Wed, Sep 19, 2018 at 03:06:30PM +0200, Richard Biener wrote: > > The second testcase in the above PR runs into our O(N) bitmap element > search limitation and spends 8s (60%) of the compile-time in the SSA > propagator > engine (when optimizing). The patch improves that to 0.9s (15%). For

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

2018-09-19 Thread Andreas Schwab
I'm getting this ICE on ia64: libtool: compile: /usr/local/gcc/gcc-20180919/Build/./gcc/xgcc -shared-libgcc -B/usr/local/gcc/gcc-20180919/Build/./gcc -nostdinc++ -L/usr/local/gcc/gcc-20180919/Build/ia64-suse-linux/libstdc++-v3/src -L/usr/local/gcc/gcc-20180919/Build/ia64-suse-linux/libstdc

[PATCH][RFC] Fix PR63155 (some more)

2018-09-19 Thread Richard Biener
The second testcase in the above PR runs into our O(N) bitmap element search limitation and spends 8s (60%) of the compile-time in the SSA propagator engine (when optimizing). The patch improves that to 0.9s (15%). For the first testcase it isn't that bad but still the patch improves CCP from

Re: [PATCH] S/390: Fix conditional returns

2018-09-19 Thread Segher Boessenkool
On Tue, Sep 18, 2018 at 10:15:00PM -0600, Jeff Law wrote: > On 9/18/18 8:04 AM, Segher Boessenkool wrote: > > On Wed, Sep 05, 2018 at 10:34:48AM +0200, Ilya Leoshkevich wrote: > >> S/390 epilogue ends with (parallel [(return) (use %r14)]) instead of > >> the more usual (return) or (simple_return).

Re: Transform assertion into optimization hints

2018-09-19 Thread Jonathan Wakely
On 19/09/18 10:42 +0100, Jonathan Wakely wrote: On 19/09/18 10:16 +0100, Jonathan Wakely wrote: On 17/09/18 23:15 +0200, Marc Glisse wrote: On Mon, 17 Sep 2018, François Dumont wrote: We talk about it a while back. I've run testsuite several times since I have this patch on my local copy.

Re: [PATCH] PR libstdc++/87135 Rehash only when necessary (LWG2156)

2018-09-19 Thread Jonathan Wakely
On 18/09/18 22:39 +0200, François Dumont wrote: On 09/18/2018 10:41 AM, Jonathan Wakely wrote: On 13/09/18 07:49 +0200, François Dumont wrote: All changes limited to hashtable_c++0x.cc file. _Prime_rehash_policy::_M_next_bkt now really does what its comment was declaring that is to say:  

[PATCH] dumpfile.c: fix stray dump_loc output (PR tree-optimization/87309)

2018-09-19 Thread David Malcolm
In r262891 I reimplemented this call: dump_printf_loc (MSG_NOTE, loc, "=== %s ===\n", name); in dump_begin_scope to use direct calls to dump_loc: if (dump_file) { dump_loc (MSG_NOTE, dump_file, loc.get_location_t ()); fprintf (dump_file, "=== %s ===\n", name); } if

[PATCH v4] [aarch64] Add HiSilicon tsv110 CPU support

2018-09-19 Thread Shaokun Zhang
This patch adds HiSilicon's an mcpu: tsv110, which supports v8_4A. It has been tested on aarch64 and no regressions from this patch. --- gcc/ChangeLog| 9 +++ gcc/config/aarch64/aarch64-cores.def | 3 + gcc/config/aarch64/aarch64-cost-tables.h | 104

Re: Transform assertion into optimization hints

2018-09-19 Thread Jonathan Wakely
On 19/09/18 10:16 +0100, Jonathan Wakely wrote: On 17/09/18 23:15 +0200, Marc Glisse wrote: On Mon, 17 Sep 2018, François Dumont wrote: We talk about it a while back. I've run testsuite several times since I have this patch on my local copy. Note that when I implemented it the wrong way

Re: [GCC][PATCH v2][Aarch64] Exploiting BFXIL when OR-ing two AND-operations with appropriate bitmasks

2018-09-19 Thread Kyrill Tkachov
Hi Christophe, On 18/09/18 23:00, Christophe Lyon wrote: On Thu, 13 Sep 2018 at 11:49, Kyrill Tkachov wrote: On 13/09/18 10:25, Sam Tebbs wrote: On 09/11/2018 04:20 PM, James Greenhalgh wrote: On Tue, Sep 04, 2018 at 10:13:43AM -0500, Sam Tebbs wrote: Hi James, Thanks for the feedback.

Re: Transform assertion into optimization hints

2018-09-19 Thread Jonathan Wakely
On 17/09/18 23:15 +0200, Marc Glisse wrote: On Mon, 17 Sep 2018, François Dumont wrote: We talk about it a while back. I've run testsuite several times since I have this patch on my local copy. Note that when I implemented it the wrong way tests started to fail so it is clearly having an

Re: [PATCH] PR libstdc++/87135 Rehash only when necessary (LWG2156)

2018-09-19 Thread Jonathan Wakely
On 18/09/18 22:39 +0200, François Dumont wrote: On 09/18/2018 10:41 AM, Jonathan Wakely wrote: On 13/09/18 07:49 +0200, François Dumont wrote: All changes limited to hashtable_c++0x.cc file. _Prime_rehash_policy::_M_next_bkt now really does what its comment was declaring that is to say:  

Re: [PATCH] S/390: Fix conditional returns

2018-09-19 Thread Ilya Leoshkevich
> Am 19.09.2018 um 06:15 schrieb Jeff Law : > > On 9/18/18 8:04 AM, Segher Boessenkool wrote: >> On Wed, Sep 05, 2018 at 10:34:48AM +0200, Ilya Leoshkevich wrote: >>> S/390 epilogue ends with (parallel [(return) (use %r14)]) instead of >>> the more usual (return) or (simple_return). This

Re: [Patch, fortran] PR87239 - ICE in deferred-length string

2018-09-19 Thread Janne Blomqvist
On Tue, Sep 18, 2018 at 10:41 PM Paul Richard Thomas < paul.richard.tho...@gmail.com> wrote: > This is sufficiently 'obvious' that I have committed the patch to > trunk as revision 264409. > > My inclination is to commit it to 8-branch as well since deferred > character bugs were one of the

[PATCH] Fix PR87349/87342

2018-09-19 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-09-19 Richard Biener PR tree-optimization/87349 PR tree-optimization/87342 * tree-ssa-sccvn.c (do_rpo_vn): Iterate max_rpo computation. * gcc.dg/torture/pr87349-1.c: New testcase.