RE: [Patch,microblaze]: Optimized usage of fint instruction.

[Ajit Kumar Agarwal <ajit dot kumar dot agarwal at xilinx dot com>]

-----Original Message-----
From: Michael Eager [mailto:eager@eagerm.com] 
Sent: Thursday, February 26, 2015 4:33 AM
To: Ajit Kumar Agarwal; GCC Patches
Cc: Vinod Kathail; Shail Aditya Gupta; Vidhumouli Hunsigida; Nagaraju Mekala
Subject: Re: [Patch,microblaze]: Optimized usage of fint instruction.

On 02/25/15 02:20, Ajit Kumar Agarwal wrote:
> Hello All:
>
> Please find the patch for the optimized usage of fint instruction 
> changes. No regression is seen in the deja GNU tests.
>
> commit ed4dc0b96bf43c200cacad97f73a98ab7048e51b
> Author: Ajit Kumar Agarwal <ajitkum@xhdspdgnu.(none)>
> Date:   Wed Feb 25 15:36:29 2015 +0530
>
>      [Patch,microblaze]: Optimized usage of fint instruction.
>
>      The changes are made in the patch for optimized usage of fint instruction.
>      The sequence of fint/cond_branch is replaced with fcmp/cond_branch. The
>      fint instruction takes 6/7 cycles as compared to fcmp instruction which
>      takes 1 cycles. The conversion from float to int with fint instruction
>      is not required and can directly compared with fcmp instruction which
>      takes 1 cycle as compared to 6/7 cycles with fint instruction.
>
>      ChangeLog:
>      2015-02-25  Ajit Agarwal  <ajitkum@xilinx.com>
>
>          * config/microblaze/microblaze.md (peephole2): New.

>
+    emit_insn (gen_cstoresf4 (comp_reg, operands[2],
+                              gen_rtx_REG(SFmode,REGNO(cmp_op0)),
+                              gen_rtx_REG(SFmode,REGNO(cmp_op1))));

>>Spaces before left parens and after comma in last two lines.

Changes are incorporated. Please find the log for updated patch.

commit 492b0d0b67a5b12d2dc239de3215630c8838edea
Author: Ajit Kumar Agarwal <ajitkum@xhdspdgnu.(none)>
Date:   Wed Mar 4 17:15:16 2015 +0530

    [Patch,microblaze]: Optimized usage of fint instruction.
    
    The changes are made in the patch for optimized usage of fint instruction.
    The sequence of fint/cond_branch is replaced with fcmp/cond_branch. The
    fint instruction takes 6/7 cycles as compared to fcmp instruction which
    takes 1 cycles. The conversion from float to int with fint instruction
    is not required and can directly compared with fcmp instruction which
    takes 1 cycle as compared to 6/7 cycles with fint instruction.
    
    ChangeLog:
    2015-03-04  Ajit Agarwal  <ajitkum@xilinx.com>
    
        * config/microblaze/microblaze.md (peephole2): New.
    
    Signed-off-by:Ajit Agarwal ajitkum@xilinx.com

Thanks & Regards
Ajit

Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077

Attachment: 0001-Patch-microblaze-Optimized-usage-of-fint-instruction.patch
Description: 0001-Patch-microblaze-Optimized-usage-of-fint-instruction.patch