Re: [PR47785] COLLECT_AS_OPTIONS

2020-02-23 Thread Prathamesh Kulkarni
On Wed, 19 Feb 2020 at 19:54, Richard Biener wrote: > > On Wed, Feb 19, 2020 at 12:19 AM Prathamesh Kulkarni > wrote: > > > > On Tue, 18 Feb 2020 at 19:40, Richard Biener > > wrote: > > > > > > On Tue, Feb 18, 2020 at 1:52 PM Prathamesh Kulkarni > > > wrote: > > > > > > > > On Mon, 17 Feb 2020

Re: [PATCH] Fix PR66552, Missed optimization when shift amount is result of signed modulus

2020-02-23 Thread Li Jia He
Hi, On 2020/2/22 11:12 PM, Marc Glisse wrote: On Tue, 18 Feb 2020, Li Jia He wrote: Also the pattern doing the standalone transform does /* Optimize TRUNC_MOD_EXPR by a power of two into a BIT_AND_EXPR,     i.e. "X % C" into "X & (C - 1)", if X and C are positive.     Also optimize A % (C <<

Re: [PATCH] rs6000: Fix infinite loop building ghostscript and icu [PR93658]

2020-02-23 Thread Peter Bergner
On 2/22/20 6:49 PM, Segher Boessenkool wrote: > On Sat, Feb 22, 2020 at 06:25:43PM -0600, Peter Bergner wrote: >> Is this still ok for GCC 8 & 9? > > Please do each of those backports as separate commits (so that if there > is a problem with them, we can bisect it; it also should be easier to do >

Re: [PATCH v2] RISC-V: Adjust floating point code gen for LTGT compare

2020-02-23 Thread Kito Cheng
Thanks, committed :) On Sat, Feb 22, 2020 at 6:10 AM Jim Wilson wrote: > > On Fri, Feb 21, 2020 at 1:04 AM Kito Cheng wrote: > > * config/riscv/riscv.c (riscv_emit_float_compare): Change the code > > gen > > for LTGT. > > (riscv_rtx_costs): Update cost model for LTGT. >

Re: [patch, committed] Fix trailing space in diagnostic

2020-02-23 Thread Thomas Koenig
Am 23.02.20 um 17:11 schrieb Thomas Koenig: Committed as simple and obvious, r10-6802-g7260547dbffd8e6442f99da8adf98ab0ce294e4e. Better with the patch, of course. diff --git a/gcc/fortran/interface.c b/gcc/fortran/interface.c index 429abc79ca2..4106fe5831b 100644 --- a/gcc/fortran/interface.c +

Re: One more patch for PR91333

2020-02-23 Thread Vladimir Makarov
On 2020-02-18 10:39 a.m., Maxim Kuvyrkov wrote: On Feb 18, 2020, at 6:30 PM, Vladimir Makarov wrote: On 2/17/20 10:08 AM, Maxim Kuvyrkov wrote: Hi Vladimir, This patch increases code size at -Os on arm-linux-gnueabihf by 1% (with no code-size reductions) on several SPEC CPU2006 benchmarks:

patch to fix PR93564

2020-02-23 Thread Vladimir Makarov
The following patch is for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93564 The patch was successfully bootstrapped on x86-64 and benchmarked on SPEC2000. commit 3133bed5d0327e8a9cd0a601b7ecdb9de4fc825d Author: Vladimir N. Makarov Date: Sun Feb 23 16:20:05 2020 -0500 Changing cost

Re: [RFC PATCH v0] PPC64: Implement POWER Architecure Vector Function ABI.

2020-02-23 Thread Segher Boessenkool
On Sun, Feb 23, 2020 at 10:55:53AM -0600, Bill Schmidt wrote: > Though I'm usually uncomfortable with kicking the can down the road on these > sorts of things, I can probably be convinced in this case. Tulio and I were > wondering why the libmvec interface doesn't make use of ifunc capability for

Re: [RFC PATCH v0] PPC64: Implement POWER Architecure Vector Function ABI.

2020-02-23 Thread Jakub Jelinek
On Sun, Feb 23, 2020 at 10:42:17AM -0600, Bill Schmidt wrote: > > > Have I missed something crucial? > > I haven't seen anything in the patch that would only enable it for ELFv2, > > and while powerpc64le-linux probably assumes TARGET_VSX unconditionally > > (haven't verified), powerpc64-linux or p

Re: [RFC PATCH v0] PPC64: Implement POWER Architecure Vector Function ABI.

2020-02-23 Thread Jakub Jelinek
On Sun, Feb 23, 2020 at 10:55:53AM -0600, Bill Schmidt wrote: > > If/when it becomes necessary to have 'c' variants of functions, then a new > > version of > > the Vector Function ABI document will be created. And GLIBC and GCC > > modifications to > > comply with that new ABI will be made then.

Re: [RFC PATCH v0] PPC64: Implement POWER Architecure Vector Function ABI.

2020-02-23 Thread Bill Schmidt
On 2/20/20 1:14 PM, GT wrote: ‐‐‐ Original Message ‐‐‐ On Wednesday, February 19, 2020 12:33 PM, Bill Schmidt wrote: The reason 'c' was added to the ABI is this mailing list discussion: https://sourceware.org/ml/libc-alpha/2019-11/msg00765.html As long as 'b' specifies that the VSX fu

Re: [RFC PATCH v0] PPC64: Implement POWER Architecure Vector Function ABI.

2020-02-23 Thread Bill Schmidt
On 2/14/20 4:09 PM, Jakub Jelinek wrote: On Fri, Feb 14, 2020 at 10:02:39PM +, GT wrote: Function rs6000_simd_clone_adjust, even though it's body is empty, cannot simply be removed. I tried it. It resulted in ICE. In my view, leaving it empty is preferable to modifying other files unrelate

[patch, fortran, committed] Add missing closing parenthises in error message.

2020-02-23 Thread Thomas Koenig
Another trival patch. Regards Thomas Add missing closing parenthises in error message. 2020-02-23 Thomas Koenig PR fortran/93889 * interface.c (compare_parameter): Fix error message. diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 5f889fb31

[patch, committed] Fix trailing space in diagnostic

2020-02-23 Thread Thomas Koenig
Committed as simple and obvious, r10-6802-g7260547dbffd8e6442f99da8adf98ab0ce294e4e. Regards Thomas Fix error message. 2020-02-23 Thomas Koenig PR fortran/93890 * interface.c: Replace "can not" by "cannot" and remove trailing space. 2020-02-23

Re: [Patch, fortran] PR57710 - [OOP] [F08] _vptr not set for allocatable CLASS component inside BLOCK

2020-02-23 Thread Paul Richard Thomas
Committed as r10-6801-g61c8d9e4e5f540501eaa98aae1d6c74bde7d4299 Thanks Paul On Sun, 23 Feb 2020 at 14:25, Thomas Koenig wrote: > > Hi Paul, > > > Regtested on FC31/x86_64 - OK for head? > > Looks good. > > Best of luck committing! > > Regards > > Thomas -- "If you can't explain it s

Re: [Patch, fortran] PR57710 - [OOP] [F08] _vptr not set for allocatable CLASS component inside BLOCK

2020-02-23 Thread Thomas Koenig
Hi Paul, Regtested on FC31/x86_64 - OK for head? Looks good. Best of luck committing! Regards Thomas

[PATCH] libstd++: Library-side tests for parenthesized aggregate init (c++/92878, c++/92947)

2020-02-23 Thread Ville Voutilainen
This shebang adds library tests for all cases of parenthesized aggregate initialization that I could find. Tested locally on Linux-x64, going to test with full suite on Linux-PPC64, OK for trunk if tests pass? 2020-02-23 Ville Voutilainen Library-side tests for parenthesized aggregate init

[Patch, fortran] PR57710 - [OOP] [F08] _vptr not set for allocatable CLASS component inside BLOCK

2020-02-23 Thread Paul Richard Thomas
This patch is relatively trivial and represents my first foray into gitland. Thus far, it has been... well, "interesting" compared with svn. Class components of derived types are initialized by calls to trans-array.c(gfc_trans_deferred_array) from trans-decl.c(gfc_trans_deferred_vars). The compone