This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch for MIPS profiling in mips.h
- To: gavin at cygnus dot com
- Subject: Re: patch for MIPS profiling in mips.h
- From: Ethan Solomita <ethan at cs dot columbia dot edu>
- Date: Tue, 27 Jun 2000 19:24:48 -0400 (EDT)
- CC: gcc-patches at gcc dot gnu dot org
Sorry that *I* took so long to reply. I've now tested it
and can confirm that it seems to work on my system. I give it my
blessing, for whatever that's worth. 8)
-- Ethan
> Ethan, thank you very much for the patch. It took longer to
> look at the patch than I expected, sorry. (I've been hung up
> on the mistaken believe that jal was _not_ a macro. Doh! Of
> course it sometimes is a macro.)
>
> I've made a small change to your patch, and wondered if you'd
> review it, and be sure it works for your favorite target(s).
> I've removed the (no)reorder directives in the belief that this
> is safe to do and allows the assembler to fill the jal delay
> slot with the (d)subu in those cases where jal is not expanded
> as a macro.