From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugs-return-482783-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 118012 invoked by alias); 6 Apr 2015 17:11:14 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Received: (qmail 117577 invoked by uid 55); 6 Apr 2015 17:11:11 -0000
From: "meissner at linux dot vnet.ibm.com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/65614] [5 Regression] PowerPC VSX systems should use XSCPSGNDP to copy scalar fp data to/from Altivec registers
Date: Mon, 06 Apr 2015 17:11:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: target
X-Bugzilla-Version: 5.0
X-Bugzilla-Keywords: wrong-code
X-Bugzilla-Severity: major
X-Bugzilla-Who: meissner at linux dot vnet.ibm.com
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: meissner at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 5.0
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-65614-4-IxSOd948xH@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-65614-4@http.gcc.gnu.org/bugzilla/>
References: <bug-65614-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2015-04/txt/msg00335.txt.bz2

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65614

--- Comment #5 from Michael Meissner <meissner at linux dot vnet.ibm.com> ---
On Fri, Apr 03, 2015 at 07:19:24PM +0000, dje at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65614
> 
> --- Comment #4 from David Edelsohn <dje at gcc dot gnu.org> ---
> Comment on attachment 35227
>   --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=35227
> Proposed patch to fix the problem
> 
> The FLOAT_EXTEND cost should be based on the processor tuning, not the ISA.

This patch moves the decision to the cost structure based on the processor
tuning.  Is it ok to install?

[gcc]
2015-04-06  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/65614
    * config/rs6000/rs6000.c (struct processor_costs): Add cost field
    for SF->DF conversions to make FLOAT_EXTEND more expensive, so
    that LFD is used to load double constants instead of LFS.  Add
    defaults for all costs structures.  Add comments for missing
    initialization fields.
    (size32_cost): Likewise.
    (size64_cost): Likewise.
    (rs64a_cost): Likewise.
    (mpccore_cost): Likewise.
    (ppc403_cost): Likewise.
    (ppc405_cost): Likewise.
    (ppc440_cost): Likewise.
    (ppc476_cost): Likewise.
    (ppc601_cost): Likewise.
    (ppc603_cost): Likewise.
    (ppc604_cost): Likewise.
    (ppc604e_cost): Likewise.
    (ppc620_cost): Likewise.
    (ppc630_cost): Likewise.
    (ppccell_cost): Likewise.
    (ppc750_cost): Likewise.
    (ppc7450_cost): Likewise.
    (ppc8540_cost): Likewise.
    (ppce300c2c3_cost): Likewise.
    (ppce500mc_cost): Likewise.
    (ppce500mc64_cost): Likewise.
    (ppce5500_cost): Likewise.
    (ppce6500_cost): Likewise.
    (titan_cost): Likewise.
    (power4_cost): Likewise.
    (power6_cost): Likewise.
    (power7_cost): Likewise.
    (power8_cost): Likewise.
    (ppca2_cost): Likewise.
    (rs6000_rtx_costs): Make FLOAT_EXTEND use SFDF_convert field.

    * config/rs6000/rs6000.md (extendsfdf2_fpr): Generate XSCPSGNDP
    instead of XXLOR to copy SFmode to clear out dirty bits created
    when SFmode denormals are generated.
    (mov<mode>_hardfloat, FMOVE32 case): Likewise.
    (truncdfsf2_fpr): Add support for ISA 2.07 XSRSP instruction.

[gcc/testsuite]
2015-04-06  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/65614
    * gcc.target/powerpc/compress-float-ppc-pic.c: Run test on power5
    to get floating point compression.
    * gcc.target/powerpc/compress-foat-ppc.c: Likewise.