qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] d83820: tcg: Remove redundant pointer from TC

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] d83820: tcg: Remove redundant pointer from TCGContext
Date: Sun, 07 Oct 2012 12:00:13 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d838201111894b17ca9123d5d2a74ab4cf6376d6
      
https://github.com/qemu/qemu/commit/d838201111894b17ca9123d5d2a74ab4cf6376d6
  Author: Stefan Weil <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M tcg/tcg.c
    M tcg/tcg.h

  Log Message:
  -----------
  tcg: Remove redundant pointer from TCGContext

The pointer entry 'temps' always refers to the array entry 'static_temps'.
Removing the pointer and renaming 'static_temps' to 'temps' reduces the
size of TCGContext (4 or 8 byte) and allows better code generation.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 76a23ca099084ce72f5c30a3c5dc18e331384c59
      
https://github.com/qemu/qemu/commit/76a23ca099084ce72f5c30a3c5dc18e331384c59
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/cpu.c

  Log Message:
  -----------
  target-sparc: Tidy cpu_dump_state

We don't really need to be told that %g are general register, etc.
Issue a trailing newline to separate blocks.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: b884fc5ecaacd241829f225282356d59eb186645
      
https://github.com/qemu/qemu/commit/b884fc5ecaacd241829f225282356d59eb186645
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/int32_helper.c
    M target-sparc/int64_helper.c

  Log Message:
  -----------
  target-sparc: Make CPU_LOG_INT useful by default

No need for ifdefs when the log mask does just as well.
No need to print pc/npc when we're dumping the whole cpu state.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: d4a288ef9c2b432307961429bdcacb2416ad8a99
      
https://github.com/qemu/qemu/commit/d4a288ef9c2b432307961429bdcacb2416ad8a99
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy do_branch interfaces

We always pass cpu_cond to the r_cond parameter.  Use that global
register directly instead of passing it down.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: dee8913cbf82a03fed661ac6c461491045dac7f5
      
https://github.com/qemu/qemu/commit/dee8913cbf82a03fed661ac6c461491045dac7f5
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy flush_cond interface

We always pass cpu_cond to the cond parameter.  Use that global
register directly instead of passing it down.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 5b12f1e8a1aad8030c3ec5790cb5f8f370a1df1f
      
https://github.com/qemu/qemu/commit/5b12f1e8a1aad8030c3ec5790cb5f8f370a1df1f
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy gen_trap_ifnofpu interface

We always pass cpu_cond to the cond parameter.  Use that global
register directly instead of passing it down.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 66442b07ae37a23c727ecd8c30a25054a7f21886
      
https://github.com/qemu/qemu/commit/66442b07ae37a23c727ecd8c30a25054a7f21886
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy save_state interface

Use the cpu_cond global register directly instead of passing it down.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 13a6dd00433005831c2bc671b0e085cb5b04b424
      
https://github.com/qemu/qemu/commit/13a6dd00433005831c2bc671b0e085cb5b04b424
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy gen_mov_pc_npc interface

Use the cpu_cond global register directly instead of passing it down.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 934da7ee0840f582c7720d0ce4507456d9955feb
      
https://github.com/qemu/qemu/commit/934da7ee0840f582c7720d0ce4507456d9955feb
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy save_npc interface

Use the cpu_cond global register directly instead of passing it down.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 2e655fe768d9873d07be3be264c042c35720694b
      
https://github.com/qemu/qemu/commit/2e655fe768d9873d07be3be264c042c35720694b
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy gen_generic_branch interface

The arguments passed are always the same.
Pass down just DisasContext instead.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 416fcaea1ef7e9a71ccb0a968de594618ed7675e
      
https://github.com/qemu/qemu/commit/416fcaea1ef7e9a71ccb0a968de594618ed7675e
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Introduce DisasCompare and functions to generate it

For the moment gen_cond et al retain their existing interface,
using setcond to turn a (potential) comparison back into a boolean.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 3a49e7598b78bca019e35c42590914faab2a817a
      
https://github.com/qemu/qemu/commit/3a49e7598b78bca019e35c42590914faab2a817a
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use DisasCompare in Tcc

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 7e480893cc3eb9f307199517f0e1d5c8e72a030f
      
https://github.com/qemu/qemu/commit/7e480893cc3eb9f307199517f0e1d5c8e72a030f
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use DisasCompare and movcond in FMOVR, FMOVCC

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: f52879b4410aa511e7c2baccc6a2fe37096b471e
      
https://github.com/qemu/qemu/commit/f52879b4410aa511e7c2baccc6a2fe37096b471e
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use DisasCompare and movcond in MOVCC

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: c33f80f52a4621e6ed2d021adb08c7de8bec09d8
      
https://github.com/qemu/qemu/commit/c33f80f52a4621e6ed2d021adb08c7de8bec09d8
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use DisasCompare and movcond in MOVR

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 61316742e2bc7b5b5257198f2248c42a9d238c84
      
https://github.com/qemu/qemu/commit/61316742e2bc7b5b5257198f2248c42a9d238c84
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use movcond in gen_generic_branch

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: c28ae41ecd3bec70f1db8545e0800b9023891057
      
https://github.com/qemu/qemu/commit/c28ae41ecd3bec70f1db8545e0800b9023891057
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/cpu.h
    M target-sparc/helper.c
    M target-sparc/helper.h
    M target-sparc/ldst_helper.c
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Move sdivx and udivx out of line

The branches around the exception are maintaining an otherwise
unnecessary use of local temps for the cpu destination.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: bd49ed41ebe518c79bd52e46ce5b9cf278f8a2af
      
https://github.com/qemu/qemu/commit/bd49ed41ebe518c79bd52e46ce5b9cf278f8a2af
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Tidy Tcc

Share more code between unconditional and conditional paths.

Move the computation of the trap number into the conditional BB;
avoid using temporaries that have gone out of scope (cpu_tmp32)
or rely on local temps (cpu_dst).

Fully fold the exception number when the trap number is %g0+imm.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: a2ea4aa9898086c1e45e1db9b5f94d16dbf0762e
      
https://github.com/qemu/qemu/commit/a2ea4aa9898086c1e45e1db9b5f94d16dbf0762e
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/helper.c
    M target-sparc/helper.h
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Move taddcctv and tsubcctv out of line

The branches around the exception are maintaining an otherwise
unnecessary use of local temps for the cpu destination.

Note that gen_op_t{add,sub}_cc were identical to gen_op_{add,sub}_cc.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 6cb675b0f89131dcb7bc4f9e0801ac7620374185
      
https://github.com/qemu/qemu/commit/6cb675b0f89131dcb7bc4f9e0801ac7620374185
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use movcond in mulscc

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 690995a615ed584a4d494a7b61ede4f849d83e4a
      
https://github.com/qemu/qemu/commit/690995a615ed584a4d494a7b61ede4f849d83e4a
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Use movcond for FMOV*R

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: b09b2fd30ce3079fbca1bbd7c1a87459378c5bd7
      
https://github.com/qemu/qemu/commit/b09b2fd30ce3079fbca1bbd7c1a87459378c5bd7
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Cleanup "global" temporary allocation

There are 6 temporaries that disas_sparc_insn relies on having been
allocated.  Now that they are no longer referenced across branches,
they need not be allocated as local temps.

Move the allocation/free of these temporaries to make it clear that
they are local to the translation of a single insn.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: fe1755cbb21c34a40234e2c380d049b9b031c4e2
      
https://github.com/qemu/qemu/commit/fe1755cbb21c34a40234e2c380d049b9b031c4e2
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Fall through from not-taken trap

Now that we've cleaned up global temporary allocation, we can
continue translating the fallthru path of a conditional trap.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 2a484ecf8283814c92b5743845e285f0efc23247
      
https://github.com/qemu/qemu/commit/2a484ecf8283814c92b5743845e285f0efc23247
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M target-sparc/translate.c

  Log Message:
  -----------
  target-sparc: Optimize conditionals using SUBCC

Aka "normal" comparisons.  We now have the infrastructure to
pass back non-boolean results from gen_compare.  This will
automatically get used by both branches and conditional moves.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Blue Swirl <address@hidden>


  Commit: 4bb26682f70a5f626cad3e0ac82bf4b6252ea7a4
      
https://github.com/qemu/qemu/commit/4bb26682f70a5f626cad3e0ac82bf4b6252ea7a4
  Author: Blue Swirl <address@hidden>
  Date:   2012-10-07 (Sun, 07 Oct 2012)

  Changed paths:
    M qemu-barrier.h

  Log Message:
  -----------
  Merge branch 'master' of git.qemu.org:/pub/git/qemu

* 'master' of git.qemu.org:/pub/git/qemu:
  qemu-barrier: Fix compilation on i386 hosts


Compare: https://github.com/qemu/qemu/compare/f1829782d6cb...4bb26682f70a
reply via email to
[Prev in Thread] Current Thread [Next in Thread]