diff mbox

[MIPS] P5600 scheduler fix

Message ID B5E67142681B53468FAF6B7C313565624F4EDCDE@hhmail02.hh.imgtec.org
State New
Headers show

Commit Message

Robert Suchanek May 24, 2016, 2:48 p.m. UTC
Hi,

The below is a fix for the P5600 scheduler.  Ok to commit?

Regards,
Robert

2016-05-24  Simon Dardis  <simon.dardis@imgtec.com>
            Prachi Godbole  <prachi.godbole@imgtec.com>

gcc/
	* config/mips/p5600.md (p5600_fpu_fadd): Remove checking for
	`fabs' and `fneg' type attributes.
	(p5600_fpu_fabs): Add `fmove' to the comment.
---
 gcc/config/mips/p5600.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Matthew Fortune May 24, 2016, 2:52 p.m. UTC | #1
Robert Suchanek <Robert.Suchanek@imgtec.com> writes:
> gcc/
> 	* config/mips/p5600.md (p5600_fpu_fadd): Remove checking for
> 	`fabs' and `fneg' type attributes.
> 	(p5600_fpu_fabs): Add `fmove' to the comment.

OK.

Thanks,
Matthew
Robert Suchanek June 7, 2016, 11:57 a.m. UTC | #2
Hi,

> > gcc/
> > 	* config/mips/p5600.md (p5600_fpu_fadd): Remove checking for
> > 	`fabs' and `fneg' type attributes.
> > 	(p5600_fpu_fabs): Add `fmove' to the comment.
> 
> OK.
> 
> Thanks,
> Matthew

Committed as r237173.

Regards,
Robert
diff mbox

Patch

diff --git a/gcc/config/mips/p5600.md b/gcc/config/mips/p5600.md
index 694a745..4500ceb 100644
--- a/gcc/config/mips/p5600.md
+++ b/gcc/config/mips/p5600.md
@@ -163,10 +163,10 @@  (define_insn_reservation "msa_long_div" 10
 ;; fadd, fsub
 (define_insn_reservation "p5600_fpu_fadd" 4
   (and (eq_attr "cpu" "p5600")
-       (eq_attr "type" "fadd,fabs,fneg"))
+       (eq_attr "type" "fadd"))
   "p5600_fpu_long, p5600_fpu_apu")
 
-;; fabs, fneg, fcmp
+;; fabs, fneg, fcmp, fmove
 (define_insn_reservation "p5600_fpu_fabs" 2
   (and (eq_attr "cpu" "p5600")
        (eq_attr "type" "fabs,fneg,fcmp,fmove"))