Re: [Valgrind-developers] lazy FPU state save/restore

[Jeremy F. <je...@go...>]

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