This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Unreviewed patches
- From: Joern Rennecke <joern dot rennecke at superh dot com>
- To: ebotcazou at libertysurf dot fr (Eric Botcazou)
- Cc: zack at codesourcery dot com (Zack Weinberg), gcc-patches at gcc dot gnu dot org, joern dot rennecke at superh dot com
- Date: Mon, 19 Jul 2004 13:18:01 +0100 (BST)
- Subject: Re: Unreviewed patches
>
> > I believe that there is an SH emulator in the gdb tree, which would
> > allow you to test this.
>
> Ouch! I'm not sure the ratio benefit/cost is very appealing.
You can use the gdb simulator stand-alone, whne you build all-sim in
the unified tree, it is built as sim/sh/run.
This simulator has adequate performance; is actually used for the
automated SH regression tests.
However, I can tell you now just from looking at the code that
your proposed change is both unsafe and inappropriate.
print_slot is used in some places to actually emit a delay slot insn -
there using 0 would be inappropriate - and in other places it is used
to emit the delay slot insn before the jump, there it would be unsafe
to do peepholes.