[Bug target/53512] New: SH Target: Allow fsca and fsrra for non-SH4A

[Bug target/53512] New: SH Target: Allow fsca and fsrra for non-SH4A

[olegendo at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53512

             Bug #: 53512
           Summary: SH Target: Allow fsca and fsrra for non-SH4A
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: olegendo@gcc.gnu.org
                CC: kkojima@gcc.gnu.org
            Target: sh4*-*-*


Currently the fsca and fsrra insns are available only for SH4A.  However, there
are some SH4 variants around (e.g. SH7091) that also support those insns.
My idea was to add the following options:
  -mfsca, -mno-fsca
  -mfsrra, -mno-fsrra
...to be able to manually enable/disable the respective insns.  On SH4A -mfsca
and -mfsrra would be enabled by default when setting 
-funsafe-math-optimizations, which would preserve the current behavior.
Any objections?

[Bug target/53512] SH Target: Allow fsca and fsrra for non-SH4A

[kkojima at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53512

--- Comment #1 from Kazumoto Kojima <kkojima at gcc dot gnu.org> 2012-05-29 02:39:28 UTC ---
(In reply to comment #0)
Sounds good.

[Bug target/53512] SH Target: Allow fsca and fsrra for non-SH4A

[olegendo at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53512

--- Comment #2 from Oleg Endo <olegendo at gcc dot gnu.org> 2012-06-03 11:26:58 UTC ---
Author: olegendo
Date: Sun Jun  3 11:26:54 2012
New Revision: 188149

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188149
Log:
    PR target/53512
    * sh.opt (mfsca, mfsrra): New options.
    * sh.md (rsqrtsf2): Use TARGET_FPU_ANY and TARGET_FSRRA    condition.
    (fsca): Use TARGET_FPU_ANY and TARGET_FSCA condition.
    (sinssf2, cossf2): Fold expanders to ...
    (sincossf3): ... this new expander.  Use TARGET_FPU_ANY and
    TARGET_FSCA condition.
    * sh.c (sh_option_override): Handle TARGET_FSRRA and TARGET_FSCA.
    * doc/invoke.texi (SH Options): Add descriptions for -mfsca,
    -mno-fsca, -mfsrra, -mno-fsrra.

    PR target/53512
    * gcc.target/sh/pr53512-1.c: New.
    * gcc.target/sh/pr53512-2.c: New.
    * gcc.target/sh/pr53512-3.c: New.
    * gcc.target/sh/pr53512-4.c: New.


Added:
    trunk/gcc/testsuite/gcc.target/sh/pr53512-1.c
    trunk/gcc/testsuite/gcc.target/sh/pr53512-2.c
    trunk/gcc/testsuite/gcc.target/sh/pr53512-3.c
    trunk/gcc/testsuite/gcc.target/sh/pr53512-4.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/sh/sh.c
    trunk/gcc/config/sh/sh.md
    trunk/gcc/config/sh/sh.opt
    trunk/gcc/doc/invoke.texi
    trunk/gcc/testsuite/ChangeLog

[Bug target/53512] SH Target: Allow fsca and fsrra for non-SH4A

[olegendo at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53512

Oleg Endo <olegendo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Oleg Endo <olegendo at gcc dot gnu.org> 2012-06-03 16:23:04 UTC ---
Done.

[Bug target/53512] SH Target: Allow fsca and fsrra for non-SH4A

[olegendo at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53512

--- Comment #4 from Oleg Endo <olegendo at gcc dot gnu.org> 2012-11-12 00:27:59 UTC ---
Author: olegendo
Date: Mon Nov 12 00:27:56 2012
New Revision: 193423

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=193423
Log:
    PR target/53512
    * config/sh/sh.md (sincossf3): Fix swapped sin and cos operands.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/sh/sh.md