summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorNilay Vaish <nilay@cs.wisc.edu>2012-12-30 12:45:45 -0600
committerNilay Vaish <nilay@cs.wisc.edu>2012-12-30 12:45:45 -0600
commit23ba6fc5fbaf55b016b5f0c0a852fa135d3f5f55 (patch)
tree4e5bf0c68a330d74f5019cc73690f78953924373 /src
parent3b01edd7fa5a78565aa911c9e33f8fd3a6b0478e (diff)
downloadgem5-23ba6fc5fbaf55b016b5f0c0a852fa135d3f5f55.tar.xz
x86: implement x87 fp instruction fsincos
This patch implements the fsincos instruction. The code was originally written by Vince Weaver. Gabe had made some comments about the code, but those were never addressed. This patch addresses those comments.
Diffstat (limited to 'src')
-rw-r--r--src/arch/x86/isa/decoder/x87.isa2
-rw-r--r--src/arch/x86/isa/insts/x87/transcendental_functions/trigonometric_functions.py9
-rw-r--r--src/arch/x86/isa/microops/fpop.isa63
3 files changed, 58 insertions, 16 deletions
diff --git a/src/arch/x86/isa/decoder/x87.isa b/src/arch/x86/isa/decoder/x87.isa
index ace96fbf6..4ed902192 100644
--- a/src/arch/x86/isa/decoder/x87.isa
+++ b/src/arch/x86/isa/decoder/x87.isa
@@ -106,7 +106,7 @@ format WarnUnimpl {
0x0: fprem();
0x1: fyl2xp1();
0x2: fsqrt();
- 0x3: fsincos();
+ 0x3: Inst::FSINCOS();
0x4: frndint();
0x5: fscale();
0x6: fsin();
diff --git a/src/arch/x86/isa/insts/x87/transcendental_functions/trigonometric_functions.py b/src/arch/x86/isa/insts/x87/transcendental_functions/trigonometric_functions.py
index 437b47d70..05fd20829 100644
--- a/src/arch/x86/isa/insts/x87/transcendental_functions/trigonometric_functions.py
+++ b/src/arch/x86/isa/insts/x87/transcendental_functions/trigonometric_functions.py
@@ -38,7 +38,14 @@
microcode = '''
# FSIN
# FCOS
-# FSINCOS
+
+def macroop FSINCOS {
+ sinfp ufp1, st(0)
+ cosfp ufp2, st(0)
+ movfp st(0), ufp1
+ movfp st(-1), ufp2, spm=-1
+};
+
# FPTAN
# FPATAN
'''
diff --git a/src/arch/x86/isa/microops/fpop.isa b/src/arch/x86/isa/microops/fpop.isa
index 7acbe04ea..f6cbd2036 100644
--- a/src/arch/x86/isa/microops/fpop.isa
+++ b/src/arch/x86/isa/microops/fpop.isa
@@ -174,8 +174,40 @@ let {{
return cls
+ class FpUnaryOp(X86Microop):
+ __metaclass__ = FpOpMeta
+ # This class itself doesn't act as a microop
+ abstract = True
- class FpOp(X86Microop):
+ # Default template parameter values
+ flag_code = ""
+ cond_check = "true"
+ else_code = ";"
+
+ def __init__(self, dest, src1, spm=0, \
+ SetStatus=False, dataSize="env.dataSize"):
+ self.dest = dest
+ self.src1 = src1
+ self.src2 = "InstRegIndex(0)"
+ self.spm = spm
+ self.dataSize = dataSize
+ if SetStatus:
+ self.className += "Flags"
+ if spm:
+ self.className += "Top"
+
+ def getAllocator(self, microFlags):
+ return '''new %(class_name)s(machInst, macrocodeBlock,
+ %(flags)s, %(src1)s, %(src2)s, %(dest)s,
+ %(dataSize)s, %(spm)d)''' % {
+ "class_name" : self.className,
+ "flags" : self.microFlagsText(microFlags),
+ "src1" : self.src1, "src2" : self.src2,
+ "dest" : self.dest,
+ "dataSize" : self.dataSize,
+ "spm" : self.spm}
+
+ class FpBinaryOp(X86Microop):
__metaclass__ = FpOpMeta
# This class itself doesn't act as a microop
abstract = True
@@ -208,24 +240,27 @@ let {{
"dataSize" : self.dataSize,
"spm" : self.spm}
- class Movfp(FpOp):
- def __init__(self, dest, src1, spm=0, \
- SetStatus=False, dataSize="env.dataSize"):
- super(Movfp, self).__init__(dest, src1, "InstRegIndex(0)", \
- spm, SetStatus, dataSize)
+ class Movfp(FpUnaryOp):
code = 'FpDestReg_uqw = FpSrcReg1_uqw;'
else_code = 'FpDestReg_uqw = FpDestReg_uqw;'
cond_check = "checkCondition(ccFlagBits | cfofBits | dfBit | \
ecfBit | ezfBit, src2)"
- class Xorfp(FpOp):
+ class Xorfp(FpBinaryOp):
code = 'FpDestReg_uqw = FpSrcReg1_uqw ^ FpSrcReg2_uqw;'
- class Sqrtfp(FpOp):
+ class Sqrtfp(FpBinaryOp):
code = 'FpDestReg = sqrt(FpSrcReg2);'
+ class Cosfp(FpUnaryOp):
+ code = 'FpDestReg = cos(FpSrcReg1);'
+
+ class Sinfp(FpUnaryOp):
+ code = 'FpDestReg = sin(FpSrcReg1);'
+
+
# Conversion microops
- class ConvOp(FpOp):
+ class ConvOp(FpBinaryOp):
abstract = True
def __init__(self, dest, src1):
super(ConvOp, self).__init__(dest, src1, \
@@ -258,19 +293,19 @@ let {{
# These need to consider size at some point. They'll always use doubles
# for the moment.
- class addfp(FpOp):
+ class addfp(FpBinaryOp):
code = 'FpDestReg = FpSrcReg1 + FpSrcReg2;'
- class mulfp(FpOp):
+ class mulfp(FpBinaryOp):
code = 'FpDestReg = FpSrcReg1 * FpSrcReg2;'
- class divfp(FpOp):
+ class divfp(FpBinaryOp):
code = 'FpDestReg = FpSrcReg1 / FpSrcReg2;'
- class subfp(FpOp):
+ class subfp(FpBinaryOp):
code = 'FpDestReg = FpSrcReg1 - FpSrcReg2;'
- class Compfp(FpOp):
+ class Compfp(FpBinaryOp):
def __init__(self, src1, src2, spm=0, setStatus=False, \
dataSize="env.dataSize"):
super(Compfp, self).__init__("InstRegIndex(FLOATREG_MICROFP0)", \