x86: Fix the flag handling code in FABS and FCHS
authorAndreas Sandberg <andreas@sandberg.pp.se>
Tue, 18 Jun 2013 14:10:21 +0000 (16:10 +0200)
committerAndreas Sandberg <andreas@sandberg.pp.se>
Tue, 18 Jun 2013 14:10:21 +0000 (16:10 +0200)
commitde89e133d8a3f92d9f2f99d0d8bcca50e9da6cb4
treec4a16317ee7385992707b4f10cf01ade4abed703
parent59befdb62824ee220e4ea91a045f0db5817e1cd7
x86: Fix the flag handling code in FABS and FCHS

This changeset fixes two problems in the FABS and FCHS
implementation. First, the ISA parser expects the assignment in
flag_code to be a pure assignment and not an and-assignment, which
leads to the isa_parser omitting the misc reg update. Second, the FCHS
and FABS macro-ops don't set the SetStatus flag, which means that the
default micro-op version, which doesn't update FSW, is executed.
src/arch/x86/isa/insts/x87/arithmetic/change_sign.py
src/arch/x86/isa/microops/fpop.isa