Re: [PATCHv2] Handle overlength string literals in the fortan FE

On 03/09/2018, 21:25 Janne Blomqvist wrote: > On Fri, Aug 24, 2018 at 11:06 PM Bernd Edlinger > wrote: > >> Hi! >> >> >> This is an alternative approach to handle overlength strings in the >> Fortran FE. >> > > Hi, > > can you explain a little more what the problem that this patch tries to > solve

[PATCH] gcc: xtensa: fix NAND code in xtensa_expand_atomic

NAND is ~(a1 & a2), but xtensa_expand_atomic does ~a1 & a2. That fixes libatomic tests atomic-op-{1,2}. gcc/ 2018-09-04 Max Filippov * config/xtensa/xtensa.c (xtensa_expand_atomic): Reorder AND and XOR operations in NAND case. --- gcc/config/xtensa/xtensa.c | 6 +++--- 1 file

Re: [PATCH] genmatch: isolate reporting into a function

On Mon, Sep 3, 2018 at 4:19 PM Martin Liška wrote: > > On 09/03/2018 03:31 PM, Richard Biener wrote: > > On Mon, Sep 3, 2018 at 2:09 PM Martin Liška wrote: > >> > >> On 09/03/2018 10:19 AM, Richard Biener wrote: > >>> On Mon, Sep 3, 2018 at 9:56 AM Martin Liška wrote: > > Hi. > >

Re: [PATCH v2 1/6] [MIPS] Split Loongson (MMI) from loongson3a

On Tue, Sep 4, 2018 at 11:53 AM, Paul Hua wrote: > On Mon, Sep 3, 2018 at 8:29 PM Paul Hua wrote: >> >> > > Hi: > > The v2 patch add: > * gcc/doc/invoke.texi (-mloongson-mmi): Document. > > Thanks > Paul Hua Hi Paul, For the new files, I think the copyright year should be just 2018. diff --git

Re: [PATCH] Add -Wabsolute-value

Hi, On Fri, Aug 31 2018, Joseph Myers wrote: > On Fri, 31 Aug 2018, Martin Jambor wrote: > >> diff --git a/gcc/common.opt b/gcc/common.opt >> index ebc3ef43ce2..2950760fb2a 100644 >> --- a/gcc/common.opt >> +++ b/gcc/common.opt >> @@ -815,6 +815,10 @@ Wvector-operation-performance >> Common Var(w

[PATCH] Update C Extensions docs for support in latest C++

The docs on 'long long' and hexadecimal floating-point literals are outdated w.r.t what C++ supports. OK for trunk? diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 981e7063f97..8e31ec6b610 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2018-09-04 Jonathan Wakely + + * doc/e

Fix a minor copyright year typo

Hi, According to commit log, this file is created in the year 2018, hence the copyright year should be corrected to 2018. Is below patch ok to fix this problem? Thanks Wei Xiao diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 62e32c6..416dbd1 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,

Re: [PATCH] Update C Extensions docs for support in latest C++

On Tue, 4 Sep 2018, Jonathan Wakely wrote: > The docs on 'long long' and hexadecimal floating-point literals are > outdated w.r.t what C++ supports. > > OK for trunk? OK. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Fix PR87176

On Mon, 3 Sep 2018, Richard Biener wrote: > > The following fixes a wrong-code issue similar to that in PR87132 where > the alias walk reaches into code parts that are only reachable over > backedges. This time not via PHI processing but by value-numbering > a virtual PHI to its backedge value.

Re: Relocation (= move+destroy)

On 01/09/18 17:00 +0200, Marc Glisse wrote: _GLIBCXX_ASAN_ANNOTATE_REINIT: I am not familiar with those annotations. It was convenient in one function to move this annotation after _Destroy, to reduce code duplication. For consistency, I did the same in the whole file. As far as I understand, t

Re: VRP: abstract out bitwise AND/OR optimizations

On Wed, Aug 29, 2018 at 12:55 PM Aldy Hernandez wrote: > > I thought I was done with this one as well, but it looks like we can > push more things out to wide-int-range.*. > > I also pushed the optimization itself into wide_int_range_bit_{and,or}, > for maximal sharing. > > And finally, I split ou

Re: [Patch][Aarch64] Implement Aarch64 SIMD ABI and aarch64_vector_pcs attribute

Hi Steve, On 20/08/18 18:37, Steve Ellcey wrote: On Tue, 2018-08-07 at 12:15 -0500, Segher Boessenkool wrote: > > +/* { dg-final { scan-assembler-not "\[ \t\]stp\tq\[01234567\]" } } > > */ > That's [0-7] but maybe you find [01234567] more readable here. Segher, I fixed all the issues you poin

Re: VRP: abstract out POINTER_TYPE_P handling

On Thu, Aug 30, 2018 at 9:39 AM Aldy Hernandez wrote: > > > > On 08/29/2018 12:32 PM, David Malcolm wrote: > > On Wed, 2018-08-29 at 06:54 -0400, Aldy Hernandez wrote: > >> Never say never. Just when I thought I was done... > >> > >> It looks like I need the special casing we do for pointer types

Re: [PATCH v2 1/6] [MIPS] Split Loongson (MMI) from loongson3a

Hi Terry, Thanks for your comments. > > For the new files, I think the copyright year should be just 2018. > The loongson-mmi.md is a renamed file from loongson.md, I think the copyright year should be include the old file. But in gs464e.md and gs264e.md, the copyright years will be just 2018, I

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On Mon, Sep 3, 2018 at 1:32 PM Aldy Hernandez wrote: > > > > On 08/28/2018 05:27 AM, Richard Biener wrote: > > On Mon, Aug 27, 2018 at 2:24 PM Aldy Hernandez wrote: > >> > >> Howdy! > >> > >> Phew, I think this is the last abstraction. This handles the unary > >> CONVERT_EXPR_P code. > >> > >> I

[PATCH] Fix PR87211

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2018-09-04 Richard Biener PR tree-optimization/87211 * tree-ssa-sccvn.c (visit_phi): When value-numbering to a backedge value we're supposed to treat as VARYING also number the PHI to VARY

Re: VRP: abstract out POINTER_TYPE_P handling

On 09/04/2018 07:42 AM, Richard Biener wrote: On Thu, Aug 30, 2018 at 9:39 AM Aldy Hernandez wrote: On 08/29/2018 12:32 PM, David Malcolm wrote: On Wed, 2018-08-29 at 06:54 -0400, Aldy Hernandez wrote: Never say never. Just when I thought I was done... It looks like I need the special

Building old GCC with new GCC: stage1 32-bit libstdc++ fails to build after building 64-bit libstdc++

Hello, I was recently trying to build an older multilib bootstrap GCC (version 4.8.3) using a newer GCC (version 8.1), and I ran into an issue where after building the 64-bit libstdc++ for the target, while building the 32-bit libstdc++, I would get the following error[1]: /builddir/build/BUILD/g

Re: [PATCH] Optimise sqrt reciprocal multiplications

Hi Richard, On 31/08/18 12:07, Richard Biener wrote: > On Thu, 30 Aug 2018, Kyrill Tkachov wrote: > >> Ping. >> >> https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01496.html >> >> Thanks, >> Kyrill >> >> On 23/08/18 18:09, Kyrill Tkachov wrote: >>> Hi Richard, >>> >>> On 23/08/18 11:13, Richard San

Re: VRP: abstract out POINTER_TYPE_P handling

On Tue, Sep 4, 2018 at 2:09 PM Aldy Hernandez wrote: > > > > On 09/04/2018 07:42 AM, Richard Biener wrote: > > On Thu, Aug 30, 2018 at 9:39 AM Aldy Hernandez wrote: > >> > >> > >> > >> On 08/29/2018 12:32 PM, David Malcolm wrote: > >>> On Wed, 2018-08-29 at 06:54 -0400, Aldy Hernandez wrote: > >>

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On 09/04/2018 07:58 AM, Richard Biener wrote: On Mon, Sep 3, 2018 at 1:32 PM Aldy Hernandez wrote: On 08/28/2018 05:27 AM, Richard Biener wrote: On Mon, Aug 27, 2018 at 2:24 PM Aldy Hernandez wrote: Howdy! Phew, I think this is the last abstraction. This handles the unary CONVERT_EX

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On Tue, Sep 4, 2018 at 2:41 PM Aldy Hernandez wrote: > > > > On 09/04/2018 07:58 AM, Richard Biener wrote: > > On Mon, Sep 3, 2018 at 1:32 PM Aldy Hernandez wrote: > >> > >> > >> > >> On 08/28/2018 05:27 AM, Richard Biener wrote: > >>> On Mon, Aug 27, 2018 at 2:24 PM Aldy Hernandez wrote: >

Re: Keep std::deque algos specializations in Debug mode

On 25/08/18 22:44 +0200, François Dumont wrote: The last optimizations that get disabled when Debug mode is enable are the algo specializations for std::deque iterators. This patch move those algos in std namespace as they should even when Debug mode is enable so that they get considered even

Re: VRP: abstract out POINTER_TYPE_P handling

On 09/04/2018 08:29 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 2:09 PM Aldy Hernandez wrote: On 09/04/2018 07:42 AM, Richard Biener wrote: On Thu, Aug 30, 2018 at 9:39 AM Aldy Hernandez wrote: On 08/29/2018 12:32 PM, David Malcolm wrote: On Wed, 2018-08-29 at 06:54 -0400, Ald

[PATCH] Add -Waligned-new to Option Summary

* doc/invoke.texi (Option Summary): Add -Waligned-new. Committing to trunk and branches as obvious. commit 55e8b8ff39837963039e6852fd9c7ed403e6eb49 Author: Jonathan Wakely Date: Tue Sep 4 14:22:09 2018 +0100 Add -Waligned-new to Option Summary * doc/invoke.texi

Re: [PATCH] Add -Waligned-new to Option Summary

On 04/09/18 14:25 +0100, Jonathan Wakely wrote: * doc/invoke.texi (Option Summary): Add -Waligned-new. Committing to trunk and branches as obvious. commit 55e8b8ff39837963039e6852fd9c7ed403e6eb49 Author: Jonathan Wakely Date: Tue Sep 4 14:22:09 2018 +0100 Add -Waligned-new to

Re: [2/2] Add AddressSanitizer annotations to std::string.

^^^ gentle ping. On 08/16/2018 02:28 PM, Mikhail Kashkarov wrote: > ^^ gentle ping. > > On 07/16/2018 07:16 PM, Mikhail Kashkarov wrote: >> Rebased and update patch (typos, add missing annotations), >> add ASan teststo verify string annotation. >> >> >> On 06/28/2018 11:09 AM, Mikhail Kashkarov w

Re: VRP: abstract out POINTER_TYPE_P handling

On Tue, Sep 4, 2018 at 3:21 PM Aldy Hernandez wrote: > > > > On 09/04/2018 08:29 AM, Richard Biener wrote: > > On Tue, Sep 4, 2018 at 2:09 PM Aldy Hernandez wrote: > >> > >> > >> > >> On 09/04/2018 07:42 AM, Richard Biener wrote: > >>> On Thu, Aug 30, 2018 at 9:39 AM Aldy Hernandez wrote: >

Re: VRP: abstract out POINTER_TYPE_P handling

On 09/04/2018 09:41 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 3:21 PM Aldy Hernandez wrote: On 09/04/2018 08:29 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 2:09 PM Aldy Hernandez wrote: On 09/04/2018 07:42 AM, Richard Biener wrote: On Thu, Aug 30, 2018 at 9:39 AM Aldy H

[PATCH] Ignore properly -mdirect-move (PR target/87164).

Hi. Option mdirect-move should be Deprecated, that means option value is ignored and user can't influence rs6000_isa_flags. Patch can bootstrap on ppc64le-redhat-linux (gcc110 and gcc112) and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2018-08-31 Martin Liska

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On 09/04/2018 08:49 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 2:41 PM Aldy Hernandez wrote: On 09/04/2018 07:58 AM, Richard Biener wrote: On Mon, Sep 3, 2018 at 1:32 PM Aldy Hernandez wrote: On 08/28/2018 05:27 AM, Richard Biener wrote: On Mon, Aug 27, 2018 at 2:24 PM Aldy He

Re: VRP: abstract out POINTER_TYPE_P handling

On Tue, Sep 4, 2018 at 3:51 PM Aldy Hernandez wrote: > > > > On 09/04/2018 09:41 AM, Richard Biener wrote: > > On Tue, Sep 4, 2018 at 3:21 PM Aldy Hernandez wrote: > >> > >> > >> > >> On 09/04/2018 08:29 AM, Richard Biener wrote: > >>> On Tue, Sep 4, 2018 at 2:09 PM Aldy Hernandez wrote: >

Re: VRP: abstract out POINTER_TYPE_P handling

On 09/04/2018 10:12 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 3:51 PM Aldy Hernandez wrote: Splitting this out makes my life a lot easier, with virtually no penalty to VRP. And we could always revisit it later. But if you feel strongly about it, I could inline the pointer handling

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On Tue, Sep 4, 2018 at 4:12 PM Aldy Hernandez wrote: > > > > On 09/04/2018 08:49 AM, Richard Biener wrote: > > On Tue, Sep 4, 2018 at 2:41 PM Aldy Hernandez wrote: > >> > >> > >> > >> On 09/04/2018 07:58 AM, Richard Biener wrote: > >>> On Mon, Sep 3, 2018 at 1:32 PM Aldy Hernandez wrote: >

[PATCH] Improve x % c1 == c2 expansion (PR middle-end/82853)

Hi! Improve expansion of x % c1 == c2 and x % c1 != c2 checks. As mentioned in Hacker's Delight book, section 10-{16,17}, we can improve generated code for modulo by constant, if we only use the result to equality compare against some other constant (0 for all divisors, other constants only in ce

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On 09/04/2018 10:20 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 4:12 PM Aldy Hernandez wrote: On 09/04/2018 08:49 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 2:41 PM Aldy Hernandez wrote: On 09/04/2018 07:58 AM, Richard Biener wrote: On Mon, Sep 3, 2018 at 1:32 PM Aldy Her

[PATCH] Fix up -mno-xsave handling (PR target/87198)

Hi! The -mxsave{opt,s,c} options turn on automatically -mxsave option and the patterns rely on TARGET_XSAVE{OPT,S,C} implying TARGET_XSAVE, but if somebody uses e.g. -mxsave{opt,s,c} -mno-xsave (or something that implies -mno-xsave), TARGET_XSAVE{OPT,S,C} remains set and TARGET_XSAVE is clear. Fi

Re: [PATCH] Use complete_array_type on flexible array member initializers

On 09/03/2018 06:35 AM, Bernd Edlinger wrote: [ Big snip, dropping lots of context ] >>> >>> No I don't think so, because at that time BRACE_ENCLOSED_INITIALIZER_P >>> property does no longer work, as I explained in the previous mail. >>> >>> And cp_complete_array_type does use that property:

Re: [PATCH] Optimise sqrt reciprocal multiplications

On Tue, 4 Sep 2018, Kyrill Tkachov wrote: > Hi Richard, > > On 31/08/18 12:07, Richard Biener wrote: > > On Thu, 30 Aug 2018, Kyrill Tkachov wrote: > > > >> Ping. > >> > >> https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01496.html > >> > >> Thanks, > >> Kyrill > >> > >> On 23/08/18 18:09, Kyrill T

Re: VRP: abstract out wide int CONVERT_EXPR_P code

On 09/04/2018 10:20 AM, Richard Biener wrote: On Tue, Sep 4, 2018 at 4:12 PM Aldy Hernandez wrote: I honestly don't remember exactly but I suppose precision mismatched somehow. Your code was OK with not using widest_ints. Ah, this got lost in the noise. Sorry. Will commit the original

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

On Tue, Aug 21, 2018 at 03:14:29PM +0200, Martin Liška wrote: > On 08/18/2018 12:24 AM, Segher Boessenkool wrote: > > Removing Init is *wrong* as far as I see; it changes things, anyway. > > Could you not have done this as a separate patch? > > It's already in, but it should be fine. Note that I a

[testsuite, i386] Don't xfail gcc.target/i386/addr-sel-1.c (PR target/86744)

gcc.target/i386/addr-sel-1.c had started to XPASS around 20180730. As suggested by Uros in the PR, I've just removed the xfail. Tested on i386-pc-solaris2.11 and x86_64-pc-linux-gnu (both 32 and 64-bit), installed on mainline. Rainer -- -

Re: [RFH] split {generic,gimple}-match.c files

On 09/03/2018 04:43 PM, Richard Biener wrote: > On Mon, 3 Sep 2018, Martin Liška wrote: > >> On 09/03/2018 04:00 PM, Richard Biener wrote: >>> On Mon, 3 Sep 2018, Martin Liška wrote: >>> On 09/03/2018 02:54 PM, Martin Liška wrote: > On 09/03/2018 02:41 PM, Richard Biener wrote: >> On

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

Hi James, Thanks for the feedback. Here is an update with the changes you proposed and an updated changelog. gcc/ 2018-09-04 Sam Tebbs PR target/85628 * config/aarch64/aarch64.md (*aarch64_bfxil): Define. * config/aarch64/constraints.md (Ulc): Define

[PATCH] DWARF: Allow hard frame pointer even if frame pointer isn't used

On Mon, Sep 3, 2018 at 10:01 AM, H.J. Lu wrote: > On Mon, Sep 3, 2018 at 9:44 AM, Michael Matz wrote: >> Hi, >> >> On Mon, 3 Sep 2018, H.J. Lu wrote: >> >>> > So, what's the testcase testing then? Before the patch it doesn't ICE, >>> > after the patch it doesn't ICE. What should I look out for

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

ping From: Wilco Dijkstra Sent: 18 June 2018 15:01 To: GCC Patches Cc: nd; Joseph Myers Subject: [PATCH v3] Change default to -fno-math-errno   GCC currently defaults to -fmath-errno.  This generates code assuming math functions set errno and the application checks errno.  Few applications

Re: [ARM/FDPIC v2 04/21] [ARM] FDPIC: Add support for FDPIC for arm architecture

On 29/08/18 11:46, Kyrill Tkachov wrote: > Hi Christophe, > > On 13/07/18 17:10, christophe.l...@st.com wrote: >> From: Christophe Lyon >> >> The FDPIC register is hard-coded to r9, as defined in the ABI. >> >> We have to disable tailcall optimizations if we don't know if the >> target function i

Re: [PATCH] DWARF: add DW_AT_count to zero-length arrays

[ Adding Jason as addressee ] On 08/28/2018 08:20 PM, Omar Sandoval wrote: > On Fri, Aug 17, 2018 at 12:16:07AM -0700, Omar Sandoval wrote: >> On Thu, Aug 16, 2018 at 11:54:53PM -0700, Omar Sandoval wrote: >>> On Thu, Aug 16, 2018 at 10:27:48PM -0700, Andrew Pinski wrote: On Thu, Aug 16, 2018

PING^1 [PATCH] i386: Add pass_remove_partial_avx_dependency

On Tue, Aug 28, 2018 at 11:04 AM, H.J. Lu wrote: > With -mavx, for > > [hjl@gnu-cfl-1 skx-2]$ cat foo.i > extern float f; > extern double d; > extern int i; > > void > foo (void) > { > d = f; > f = i; > } > > we need to generate > > vxorp[ds] %xmmN, %xmmN, %xmmN > ... >

Re: [PATCH] Fix up -mno-xsave handling (PR target/87198)

On Tue, Sep 4, 2018 at 4:28 PM, Jakub Jelinek wrote: > Hi! > > The -mxsave{opt,s,c} options turn on automatically -mxsave option and > the patterns rely on TARGET_XSAVE{OPT,S,C} implying TARGET_XSAVE, > but if somebody uses e.g. -mxsave{opt,s,c} -mno-xsave (or something that > implies > -mno-xsav

[PATCH v2] gcc: xtensa: fix NAND code in xtensa_expand_atomic

NAND is ~(a1 & a2), but xtensa_expand_atomic does ~a1 & a2. That fixes libatomic tests atomic-op-{1,2}. gcc/ 2018-09-04 Max Filippov * config/xtensa/xtensa.c (xtensa_expand_atomic): Reorder AND and XOR operations in NAND case. --- Changes v1->v2: - put final inversion result in

[patch, fortan] PR87103 - [OOP] ICE in gfc_new_symbol() due to overlong symbol name

As suggested by Janus, PR87103 is easily fixed by the attached patch which increases GFC_MAX_SYMBOL_LEN to 76 (sufficient to hold the maximum allowed F08 symbol length of 63, plus a null terminator, plus the "__tmp_class_" prefix). Bootstraps and regtests successfully. OK for trunk? -Andrew O

Re: [PATCH] Optimise sqrt reciprocal multiplications

On 04/09/18 15:31, Richard Biener wrote: On Tue, 4 Sep 2018, Kyrill Tkachov wrote: Hi Richard, On 31/08/18 12:07, Richard Biener wrote: On Thu, 30 Aug 2018, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01496.html Thanks, Kyrill On 23/08/18 18:09, Kyrill Tka

Re: [Patch][Aarch64] Implement Aarch64 SIMD ABI and aarch64_vector_pcs attribute

Hi Steve, The latest version compiles the examples I used correctly, so it looks fine from that perspective (but see comments below). However the key point of the ABI is to enable better code generation when calling a vector function, and that will likely require further changes that may conflict

Re: [PATCH v2] gcc: xtensa: fix NAND code in xtensa_expand_atomic

On Tue, Sep 4, 2018 at 9:42 AM Max Filippov wrote: > NAND is ~(a1 & a2), but xtensa_expand_atomic does ~a1 & a2. > That fixes libatomic tests atomic-op-{1,2}. > > gcc/ > 2018-09-04 Max Filippov > > * config/xtensa/xtensa.c (xtensa_expand_atomic): Reorder AND and > XOR operation

Re: [patch, fortan] PR87103 - [OOP] ICE in gfc_new_symbol() due to overlong symbol name

On Tue, 4 Sep 2018 at 18:43, Andrew Benson wrote: > > As suggested by Janus, PR87103 is easily fixed by the attached patch which > increases GFC_MAX_SYMBOL_LEN to 76 (sufficient to hold the maximum allowed F08 > symbol length of 63, plus a null terminator, plus the "__tmp_class_" prefix). This is

Re: [PATCH v2] gcc: xtensa: fix NAND code in xtensa_expand_atomic

On Tue, Sep 4, 2018 at 10:35 AM, augustine.sterl...@gmail.com wrote: > On Tue, Sep 4, 2018 at 9:42 AM Max Filippov wrote: >> >> NAND is ~(a1 & a2), but xtensa_expand_atomic does ~a1 & a2. >> That fixes libatomic tests atomic-op-{1,2}. >> >> gcc/ >> 2018-09-04 Max Filippov >> >> * confi

Re: C++ PATCH/RFC for c++/87109, wrong overload with ref-qualifiers

On Thu, Aug 30, 2018 at 09:22:26AM -0400, Jason Merrill wrote: > On Wed, Aug 29, 2018 at 8:03 PM, Marek Polacek wrote: > > I've now gotten to the point where I question the validity of this PR, so > > it's > > probably a good time to stop and ask for some advice. > > > > As discussed in

Re: [PATCH] Fix PR87177 (and dup)

On Mon, Sep 3, 2018 at 6:50 AM, Richard Biener wrote: > > This reverts vuse_ssa_val back to its previous state. I need to think > about how to limit alias walking for regions but will do so as followup. > > Bootstrap & regtest running on x86_64-unknown-linux-gnu. > > Richard. > > 2018-09-03 Rich

Re: [PATCH] Ignore properly -mdirect-move (PR target/87164).

Hi! On Tue, Sep 04, 2018 at 04:02:23PM +0200, Martin Liška wrote: > Option mdirect-move should be Deprecated, that means option value is ignored > and user can't influence rs6000_isa_flags. > > Patch can bootstrap on ppc64le-redhat-linux (gcc110 and gcc112) and survives > regression tests. > >

Re: [PATCH] DWARF: Allow hard frame pointer even if frame pointer isn't used

OK. On Tue, Sep 4, 2018 at 11:13 AM, H.J. Lu wrote: > On Mon, Sep 3, 2018 at 10:01 AM, H.J. Lu wrote: >> On Mon, Sep 3, 2018 at 9:44 AM, Michael Matz wrote: >>> Hi, >>> >>> On Mon, 3 Sep 2018, H.J. Lu wrote: >>> > So, what's the testcase testing then? Before the patch it doesn't ICE,

Re: [Patch][Aarch64] Implement Aarch64 SIMD ABI and aarch64_vector_pcs attribute

On Tue, 2018-09-04 at 17:20 +, Wilco Dijkstra wrote: > External Email > > Hi Steve, > > The latest version compiles the examples I used correctly, so it looks fine > from that perspective (but see comments below). However the key point of > the ABI is to enable better code generation when cal

Re: C++ PATCH/RFC for c++/87109, wrong overload with ref-qualifiers

On Tue, Sep 4, 2018 at 3:02 PM, Marek Polacek wrote: > On Thu, Aug 30, 2018 at 09:22:26AM -0400, Jason Merrill wrote: >> On Wed, Aug 29, 2018 at 8:03 PM, Marek Polacek wrote: >> > I've now gotten to the point where I question the validity of this PR, so >> > it's >> > probably a good time to sto

[wwwdocs] Fix main page

Something must have gone awry with our main page during the last stages of the HTML 5 conversion. I noticed this yesterday and fixed it on the spot with the patch below. More CSS, though not exactly beautiful. As a next step, I'll see to reduce the use of tables for our main page; it should not

Re: [PATCH][4/4][v2] RPO-style value-numbering for FRE/PRE

On Fri, 24 Aug 2018, Richard Biener wrote: > Comments are still welcome - I've re-bootstrapped and tested the series > on x86_64-unknown-linux-gnu for all languages and will talk about > this work at the Cauldron in more detail. Is there any chance you can test this on i586 as well? Since around

Re: [patch,nvptx] Basic -misa support for nvptx

On 09/02/2018 07:57 AM, Cesar Philippidis wrote: > On 09/01/2018 12:04 PM, Tom de Vries wrote: >> On 08/31/2018 04:14 PM, Cesar Philippidis wrote: > >>> Is this patch OK for trunk? >>> >> >> Well, how did you test this ( >> https://gcc.gnu.org/contribute.html#patches : "Bootstrapping and >> testin

Re: [PATCH][4/4][v2] RPO-style value-numbering for FRE/PRE

On 09/04/2018 04:12 PM, Gerald Pfeifer wrote: > On Fri, 24 Aug 2018, Richard Biener wrote: >> Comments are still welcome - I've re-bootstrapped and tested the series >> on x86_64-unknown-linux-gnu for all languages and will talk about >> this work at the Cauldron in more detail. > > Is there any c

C++ PATCH for build_throw

It occurred to me that we should be able to use treat_lvalue_as_rvalue_p in the second context where we're supposed to use a move operation as well. Except that for a throw-expression, the operand may not be a function parameter. While at it, also make use of CP_TYPE_VOLATILE_P. Bootstrapped/regt

Re: C++ PATCH for build_throw

OK. On Tue, Sep 4, 2018 at 7:01 PM, Marek Polacek wrote: > It occurred to me that we should be able to use treat_lvalue_as_rvalue_p in > the > second context where we're supposed to use a move operation as well. Except > that for a throw-expression, the operand may not be a function parameter.

Re: [PING v2][PATCH] Make function clone name numbering independent.

Hi Martin, Richard, Thanks for your responses. On 2018-09-03 09:15 AM, Martin Jambor wrote: > Hi, > > On Mon, Sep 03 2018, Richard Biener wrote: >> On Mon, Sep 3, 2018 at 12:02 PM Martin Jambor wrote: >>> >>> Hi, >>> >>> On Fri, Aug 31 2018, Michael Ploujnikov wrote: I've done some more di

Re: [PING v2][PATCH] Make function clone name numbering independent.

Hi Martin, On 2018-09-03 06:01 AM, Martin Jambor wrote: > Hi, > > On Fri, Aug 31 2018, Michael Ploujnikov wrote: >> I've done some more digging into the current uses of >> numbered_clone_function_name and checked if any tests fail if I change >> it to suffixed_function_name: >> >> - gcc/cgraphc