mbox

[PULL,00/18] target-alpha fpu improvments

Message ID 1432148728-11985-1-git-send-email-rth@twiddle.net
State New
Headers show

Pull-request

git://github.com/rth7680/qemu.git tags/pull-axp-20150520

Message

Richard Henderson May 20, 2015, 7:05 p.m. UTC
No change since the patch set of May 12.

Thanks,


r~


The following changes since commit 385057cbec9b4a0eb6150330c572e875ed714965:

  Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-05-15' into staging (2015-05-15 17:51:20 +0100)

are available in the git repository at:

  git://github.com/rth7680/qemu.git tags/pull-axp-20150520

for you to fetch changes up to 7e66205cd6ae0fc015960af8d907fdf2147cadb8:

  target-alpha: Add vector implementation for CMPBGE (2015-05-18 13:03:47 -0700)

----------------------------------------------------------------
Rewrite fp exceptions

----------------------------------------------------------------
Richard Henderson (18):
      target-alpha: Move VAX helpers to a new file
      target-alpha: Rename floating-point subroutines
      target-alpha: Forget installed round mode after MT_FPCR
      target-alpha: Set PC correctly for floating-point exceptions
      target-alpha: Tidy FPCR representation
      target-alpha: Set fpcr_exc_status even for disabled exceptions
      target-alpha: Set EXC_M_SWC for exceptions from /S insns
      target-alpha: Raise IOV from CVTTQ
      target-alpha: Fix cvttq vs large integers
      target-alpha: Fix cvttq vs inf
      target-alpha: Fix integer overflow checking insns
      target-alpha: Implement WH64EN
      target-alpha: Disallow literal operand to 1C.30 to 1C.37
      target-alpha: Raise EXC_M_INV properly for fp inputs
      target-alpha: Suppress underflow from CVTTQ if DNZ
      target-alpha: Raise IOV from CVTQL
      target-alpha: Rewrite helper_zapnot
      target-alpha: Add vector implementation for CMPBGE

 target-alpha/Makefile.objs |   2 +-
 target-alpha/cpu.h         |  95 ++++----
 target-alpha/fpu_helper.c  | 530 +++++++++++----------------------------------
 target-alpha/helper.c      | 132 ++---------
 target-alpha/helper.h      |  14 +-
 target-alpha/int_helper.c  | 126 +++++------
 target-alpha/mem_helper.c  |   9 +-
 target-alpha/translate.c   | 265 ++++++++++++-----------
 target-alpha/vax_helper.c  | 353 ++++++++++++++++++++++++++++++
 9 files changed, 752 insertions(+), 774 deletions(-)
 create mode 100644 target-alpha/vax_helper.c

Comments

Peter Maydell May 21, 2015, 11:38 a.m. UTC | #1
On 20 May 2015 at 20:05, Richard Henderson <rth@twiddle.net> wrote:
> No change since the patch set of May 12.
>
> Thanks,
>
>
> r~
>
>
> The following changes since commit 385057cbec9b4a0eb6150330c572e875ed714965:
>
>   Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-05-15' into staging (2015-05-15 17:51:20 +0100)
>
> are available in the git repository at:
>
>   git://github.com/rth7680/qemu.git tags/pull-axp-20150520
>
> for you to fetch changes up to 7e66205cd6ae0fc015960af8d907fdf2147cadb8:
>
>   target-alpha: Add vector implementation for CMPBGE (2015-05-18 13:03:47 -0700)
>
> ----------------------------------------------------------------
> Rewrite fp exceptions

Fails to build for win32:

/home/petmay01/linaro/qemu-for-merges/target-alpha/int_helper.c: In
function ‘helper_zapnnot’:
/home/petmay01/linaro/qemu-for-merges/target-alpha/int_helper.c:48:
warning: integer constant is too large for ‘unsigned long’ type
/home/petmay01/linaro/qemu-for-merges/target-alpha/int_helper.c:49:
warning: integer constant is too large for ‘unsigned long’ type
/home/petmay01/linaro/qemu-for-merges/target-alpha/int_helper.c:50:
warning: integer constant is too large for ‘unsigned long’ type
/home/petmay01/linaro/qemu-for-merges/target-alpha/int_helper.c:51:
warning: integer constant is too large for ‘unsigned long’ type

64-bit constants should always have the 'ULL' or 'LL' suffixes.
'UL' and 'L' suffixes are almost always a bug, because they
don't work for 64-bit constants and are overkill for 32-bit
constants (where no suffix or just 'U' would do).

-- PMM
Richard Henderson May 21, 2015, 3:34 p.m. UTC | #2
On 05/21/2015 04:38 AM, Peter Maydell wrote:
> 64-bit constants should always have the 'ULL' or 'LL' suffixes.

Oops, yes indeed.


r~