[PATCH] Small reg-stack improvement (PR target/70465)

2017-02-21 Thread Jakub Jelinek
Hi! As reported by Uros, the fld a fld b fxchg %st(1) optimization to fld b fld a misses several important cases, one is FLOAT_EXTEND memory loads where the memory is SFmode or DFmode but we extend it to a wider mode, and the other is when we load a known i?87 constant like 0.0, 1.0, PI etc. Boot

Re: PR79286, ira combine_and_move_insns in loops

2017-02-21 Thread Alan Modra
On Sat, Feb 18, 2017 at 12:39:08PM +0100, Dominique d'Humières wrote: > > I'm slightly concerned about the test and how it'll behave on targets with > > small address spaces. If it's a problem we can fault in adjustments. > > The test fails on x86_64-apple-darwin16 with -m32 and -O1 and above. H

RE: [PATCH][MIPS] MSA machine description fixes

2017-02-21 Thread Moore, Catherine
> The patch fixes some bugs as mentioned below. > > 1. mips_gen_const_int_vector(): Change type for argument VAL from > int to HOST_WIDE_INT to allow const vector of type doubleword. It in > turn enables generation of BCLRI.d instead of AND.d for immediate > const vector operand with only one bit

RE: [PATCH, MIPS] Calling convention differs depending on the presence of MSA

2017-02-21 Thread Moore, Catherine
> -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Tuesday, February 21, 2017 5:35 AM > To: Sameera Deshpande ; Moore, > Catherine > Cc: gcc-patches@gcc.gnu.org > Subject: RE: [PATCH, MIPS] Calling convention differs depending on the > presence of MS

Re: [PATCH] Small reg-stack improvement (PR target/70465)

2017-02-21 Thread Jakub Jelinek
On Tue, Feb 21, 2017 at 10:23:47PM +0100, Jakub Jelinek wrote: > As reported by Uros, the > fld a > fld b > fxchg %st(1) > optimization to > fld b > fld a > misses several important cases, one is FLOAT_EXTEND memory loads where > the memory is SFmode or DFmode but we extend it to a wider mode, and

Re: [PATCH] restore -Wunused-variable on a typedef'd variable in a function template (PR 79548)

2017-02-21 Thread Martin Sebor
Ah, I see, your patch changes attribute unused handling for local variables from tracking TREE_USED to lookup_attribute. I'm not opposed to this change, but I'd like to understand why the TREE_USED handling wasn't working. In the test case in the bug: template void g () { T t; //

Rename the "openmp" group of optimizations to "omp" (was: [PATCH 3/4] OpenMP lowering changes from the hsa branch)

2017-02-21 Thread Thomas Schwinge
Hi! On Tue, 22 Nov 2016 14:43:02 +0100, Martin Jambor wrote: > On Fri, Nov 18, 2016 at 11:38:56AM +0100, Jakub Jelinek wrote: > > On Sun, Nov 13, 2016 at 10:42:01PM +0100, Martin Jambor wrote: > > > @@ -14133,7 +14183,7 @@ const pass_data pass_data_expand_omp = > > > { > > >GIMPLE_PASS, /* t

Miscellaneous optimization group fixes (was: Rename the "openmp" group of optimizations to "omp")

2017-02-21 Thread Thomas Schwinge
Hi! On Wed, 22 Feb 2017 08:48:40 +0100, I wrote: > On Tue, 22 Nov 2016 14:43:02 +0100, Martin Jambor wrote: > > On Fri, Nov 18, 2016 at 11:38:56AM +0100, Jakub Jelinek wrote: > > > On Sun, Nov 13, 2016 at 10:42:01PM +0100, Martin Jambor wrote: > > > > @@ -14133,7 +14183,7 @@ const pass_data pass_

<    1   2