[patch] Add DFA-based pipeline descriptions for MIPS 24K core.

David Ung davidu@mips.com
Fri May 6 17:49:00 GMT 2005


On Fri, 2005-05-06 at 18:18 +0100, Richard Sandiford wrote:
> David Ung <davidu@mips.com> writes:
> > because of all the bypasses involving r24k_int_store will call the check
> > on store_data_bypass_p which will fail on the above rtl.
> > The above rtl don't qualify as a set.
> 
> OK, that's fine.  But please always give this sort of detail in comments!
> Don't leave folks to guess.
> 
> The patch is OK provided you replace the comment:
> 
> ;; Unknown stores assumed to transfer integers.
> (define_insn_reservation "r24k_unknown_store" 1 
>   (and (eq_attr "cpu" "24k,24kx")
>        (and (eq_attr "type" "store")
> 	    (eq_attr "mode" "unknown")))
>   "r24k_iss+r24k_ixu_arith")
> 
> with one which says that this reservation is specifically for cprestore
> and is needed because the cprestore pattern does not have a normal
> SET-of-MEM pattern.

thanks.  I am attaching the revised 24k.md file.

David.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 24k.md
Type: application/x-genesis-rom
Size: 15155 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20050506/c4e39e9d/attachment.bin>


More information about the Gcc-patches mailing list