mbox

[PULL,0/6] Use tcg_malloc more; tcg_cond_always fix

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

Pull-request

git://github.com/rth7680/qemu.git tags/tcg-pull-20150313

Message

Richard Henderson March 13, 2015, 8:23 p.m. UTC
I should have sent the pull somewhat before now, I imagine, but if
there's still room in the schedule before the rc0 hardfreeze I'd
like to clear my backlog of 5-6 patchsets which all depend on this.


r~


The following changes since commit 7ccfb495c64e1eef5886dcc4d48523ed6d1d22a4:

  user-exec.c: fix build on NetBSD/sparc64 and NetBSD/arm (2015-03-13 15:57:00 +0000)

are available in the git repository at:

  git://github.com/rth7680/qemu.git tags/tcg-pull-20150313

for you to fetch changes up to 37ed3bf1ee07bb1a26adca0df8718f601f231c0b:

  tcg: Complete handling of ALWAYS and NEVER (2015-03-13 13:08:05 -0700)

----------------------------------------------------------------
Pool TCG data, and ALWAYS/NEVER fix

----------------------------------------------------------------
Richard Henderson (6):
      tcg: Use tcg_malloc to allocate TCGLabelQemuLdst
      tcg-ia64: Use tcg_malloc to allocate TCGLabelQemuLdst
      tcg: Change translator-side labels to a pointer
      tcg: Change generator-side labels to a pointer
      tcg: Use tcg_malloc to allocate TCGLabel
      tcg: Complete handling of ALWAYS and NEVER

 include/exec/gen-icount.h     |   4 +-
 target-alpha/translate.c      |   4 +-
 target-arm/translate-a64.c    |  26 ++++-----
 target-arm/translate.c        |   8 +--
 target-arm/translate.h        |   4 +-
 target-cris/translate.c       |  23 +++-----
 target-cris/translate_v10.c   |   6 +--
 target-i386/translate.c       |  33 ++++++------
 target-lm32/translate.c       |  20 ++++---
 target-m68k/translate.c       |   8 +--
 target-microblaze/translate.c |  12 ++---
 target-mips/translate.c       |  94 ++++++++++++++++----------------
 target-moxie/translate.c      |   2 +-
 target-openrisc/translate.c   |  34 ++++++------
 target-ppc/translate.c        | 123 ++++++++++++++++++++----------------------
 target-s390x/translate.c      |   7 +--
 target-sh4/translate.c        |  22 ++++----
 target-sparc/translate.c      |  11 ++--
 target-tricore/translate.c    |   6 +--
 target-unicore32/translate.c  |   6 +--
 target-xtensa/translate.c     |  18 +++----
 tcg/aarch64/tcg-target.c      |  15 +++---
 tcg/arm/tcg-target.c          |  14 ++---
 tcg/i386/tcg-target.c         |  72 ++++++++++++-------------
 tcg/ia64/tcg-target.c         |  35 ++++++------
 tcg/mips/tcg-target.c         |  17 +++---
 tcg/ppc/tcg-target.c          |  20 ++++---
 tcg/s390/tcg-target.c         |  31 +++++------
 tcg/sparc/tcg-target.c        |  22 ++++----
 tcg/tcg-be-ldst.h             |  26 +++++----
 tcg/tcg-op.c                  |  45 ++++++++++------
 tcg/tcg-op.h                  |  19 +++----
 tcg/tcg.c                     |  45 ++++++++--------
 tcg/tcg.h                     |  34 ++++++++++--
 tcg/tci/tcg-target.c          |  13 +++--
 35 files changed, 433 insertions(+), 446 deletions(-)

Comments

Patchew Tool March 13, 2015, 8:50 p.m. UTC | #1
This series passed Patchew automatic testing, but there are some warnings.

Find the log fragments below, or open the following URL to see the full log:

http://qemu.patchew.org/testing/log/<1426278193-15317-1-git-send-email-rth@twiddle.net>

----------8<---------

total: 0 errors, 0 warnings, 1596 lines checked

0003-tcg--Change-translator-side-labels-to-a-pointer.patch has no obvious style problems and is ready for submission.

Checking 0004-tcg--Change-generator-side-labels-to-a-pointer.patch
total: 0 errors, 0 warnings, 753 lines checked

0004-tcg--Change-generator-side-labels-to-a-pointer.patch has no obvious style problems and is ready for submission.

Checking 0005-tcg--Use-tcg_malloc-to-allocate-TCGLabel.patch
command failed with exit code 0
$@
ERROR: spaces prohibited around that ':' (ctx:WxW)
#69: FILE: tcg/tcg.h:170:
+    unsigned has_value : 1;
                        ^

ERROR: spaces prohibited around that ':' (ctx:WxW)
#70: FILE: tcg/tcg.h:171:
+    unsigned id : 31;
                 ^

total: 2 errors, 0 warnings, 91 lines checked

0005-tcg--Use-tcg_malloc-to-allocate-TCGLabel.patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

Checking 0006-tcg--Complete-handling-of-ALWAYS-and-NEVER.patch
total: 0 errors, 0 warnings, 40 lines checked

0006-tcg--Complete-handling-of-ALWAYS-and-NEVER.patch has no obvious style problems and is ready for submission.


*** Testing 'configure' ***

Install prefix    /usr/local
Peter Maydell March 16, 2015, 9:53 a.m. UTC | #2
On 13 March 2015 at 20:23, Richard Henderson <rth@twiddle.net> wrote:
>
> I should have sent the pull somewhat before now, I imagine, but if
> there's still room in the schedule before the rc0 hardfreeze I'd
> like to clear my backlog of 5-6 patchsets which all depend on this.

Well, we can probably put this patchset in, but I think it's
getting too late to put the dependent patchsets in (hardfreeze
is tomorrow).

-- PMM
Peter Maydell March 16, 2015, 10:57 a.m. UTC | #3
On 13 March 2015 at 20:23, Richard Henderson <rth@twiddle.net> wrote:
>
> I should have sent the pull somewhat before now, I imagine, but if
> there's still room in the schedule before the rc0 hardfreeze I'd
> like to clear my backlog of 5-6 patchsets which all depend on this.
>
>
> r~
>
>
> The following changes since commit 7ccfb495c64e1eef5886dcc4d48523ed6d1d22a4:
>
>   user-exec.c: fix build on NetBSD/sparc64 and NetBSD/arm (2015-03-13 15:57:00 +0000)
>
> are available in the git repository at:
>
>   git://github.com/rth7680/qemu.git tags/tcg-pull-20150313
>
> for you to fetch changes up to 37ed3bf1ee07bb1a26adca0df8718f601f231c0b:
>
>   tcg: Complete handling of ALWAYS and NEVER (2015-03-13 13:08:05 -0700)
>
> ----------------------------------------------------------------
> Pool TCG data, and ALWAYS/NEVER fix

Applied, thanks.

-- PMM