]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit - sim/sh/ChangeLog
Correct fabs and fneg insns in simulator
authorOleg Endo <oleg.endo@t-online.de>
Fri, 28 Nov 2014 15:39:39 +0000 (19:39 +0400)
committerJoel Brobecker <brobecker@adacore.com>
Fri, 28 Nov 2014 15:44:03 +0000 (19:44 +0400)
commit57df9adf2d437e3c7f17a77c3e0f3c0d8e56aa40
tree73a55ed62e5e2bde5d03fd54513570594f98b050
parentf7ca3fcfccd144c234370aa939e4f5f15f3b2a88
Correct fabs and fneg insns in simulator

It seems that the implementation of the SH fabs and fneg insns in the
simulator is not correct.  They use the FP_UNARY macro which checks the
FPSCR.PR setting and raises an exception if PR = 1 (double precision)
and the register number is not even (i.e. a valid DF reg number).
For normal unary FP insns this is fine.  However, fneg and fabs perform
the same (integer) operations regardless of the FPSCR.PR setting.

This issue initially popped up here
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63260

I've checked some of the failing tests mentioned in GCC PR 63260 above
with the patch applied and the failures go away.

sim/sh/ChangeLog (tiny patch):

* gencode.c (fabs, fneg): Implement as integer operation
instead of using the FP_UNARY macro.
sim/sh/ChangeLog
sim/sh/gencode.c