public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
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	[thread overview]
Message-ID: <bug-65614-4-IxSOd948xH@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-65614-4@http.gcc.gnu.org/bugzilla/>

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.


  parent reply	other threads:[~2015-04-06 17:11 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-27 19:43 [Bug target/65614] New: " meissner at gcc dot gnu.org
2015-03-27 20:22 ` [Bug target/65614] " meissner at gcc dot gnu.org
2015-03-27 20:29 ` meissner at gcc dot gnu.org
2015-03-27 21:22 ` meissner at gcc dot gnu.org
2015-03-30  9:22 ` [Bug target/65614] [5 Regression] " rguenth at gcc dot gnu.org
2015-04-03 19:19 ` dje at gcc dot gnu.org
2015-04-06 17:11 ` meissner at linux dot vnet.ibm.com [this message]
2015-04-07  3:36 ` meissner at gcc dot gnu.org
2015-04-07  7:52 ` jakub at gcc dot gnu.org
2015-04-07 15:39 ` meissner at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-65614-4-IxSOd948xH@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).