On 08.01.2014, at 19:32, Richard Henderson <r...@twiddle.net> wrote:
> On 01/07/2014 08:06 AM, Tom Musta wrote: >> The fri* series of instructions was introduced prior to ISA 2.06 and >> is supported on Power7 and Power8 hardware. However, the instruction >> is still considered illegal in the P7 and P8 QEMU emulation models. >> This patch enables these instructions for the P7 and P8 machines. >> >> Also, the existing helper is modified to correctly handle some of >> the boundary cases (NaNs and the inexact flag). >> >> Signed-off-by: Tom Musta <tommu...@gmail.com> >> --- >> V4: frin changed to use "ties away" rounding mode per Richard Henderson's >> review. Modified NaN handling. Proper handling of stickiness of >> the inexact flag. Added to P7+ model. >> >> target-ppc/fpu_helper.c | 18 +++++++++++------- >> target-ppc/translate_init.c | 3 +++ >> 2 files changed, 14 insertions(+), 7 deletions(-) > > Reviewed-by: Richard Henderson <r...@twiddle.net> This patch (among others) spawn warnings about constants that are defined "ul" on 32bit hosts: http://award.ath.cx/results/89-alex/ibook/virt.qemu.build/debug/virt.qemu.build.DEBUG Tom, please follow up with a patch fixing all of these in target-ppc. Alex
