|
From: Jeremy F. <je...@go...> - 2002-10-03 04:48:31
|
On Wed, 2002-10-02 at 21:42, Jeremy Fitzhardinge wrote:
> Hi,
>
> This patch makes FPU state changes lazy, so there should only be one
> save/restore pair per basic block.
Oh, for safety's sake, it should also probably have:
default:
if (VG_(needs).extended_UCode) {
+ if (fplive) {
+ emit_put_fpu_state();
+ fplive = False;
+ }
SK_(emit_XUInstr)(u, regs_live_before);
} else {
VG_(printf)("\nError:\n"
" unhandled opcode: %u. Perhaps "
" VG_(needs).extended_UCode should be set?\n",
u->opcode);
VG_(pp_UInstr)(0,u);
VG_(core_panic)("emitUInstr: unimplemented opcode");
}
J
|