-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1918/
-----------------------------------------------------------

(Updated June 13, 2013, 4:26 a.m.)


Review request for Default and Gabe Black.


Description
-------

Changeset 9760:5c7f9a77bfdd
---------------------------
x86: Add support for FXSAVE, FXSAVE64, FXRSTOR, and FXRSTOR64


Diffs (updated)
-----

  
src/arch/x86/isa/insts/simd128/integer/save_and_restore_state/save_and_restore_state.py
 9df73385c878 
  src/arch/x86/isa/decoder/two_byte_opcodes.isa 9df73385c878 
  src/arch/x86/isa/microops/regop.isa 9df73385c878 
  src/arch/x86/utility.hh 9df73385c878 
  src/arch/x86/utility.cc 9df73385c878 

Diff: http://reviews.gem5.org/r/1918/diff/


Testing (updated)
-------

Quick regressions pass. Minor stat differences in full regressions (to be 
expected since some instructions aren't no-ops anymore). Solves a bug triggered 
by switching between a simulated CPU that tries to save the x87 state and a 
virtualized CPU that restores the state (which is completely bogus unless 
FXSAVE is implemented).


Thanks,

Andreas Sandberg

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to