diff mbox series

[v6,07/50] tcg: Return NULL temp for TCG_CALL_DUMMY_ARG

Message ID	20171016172609.23422-8-richard.henderson@linaro.org
State	Superseded
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 2001:4830:134:3::11 as permitted sender) client-ip=2001:4830:134:3::11; From: Richard Henderson <richard.henderson@linaro.org> To: qemu-devel@nongnu.org Date: Mon, 16 Oct 2017 10:25:26 -0700 Message-Id: <20171016172609.23422-8-richard.henderson@linaro.org> In-Reply-To: <20171016172609.23422-1-richard.henderson@linaro.org> References: <20171016172609.23422-1-richard.henderson@linaro.org> Subject: [Qemu-devel] [PATCH v6 07/50] tcg: Return NULL temp for TCG_CALL_DUMMY_ARG Precedence: list Cc: cota@braap.org, Richard Henderson <rth@twiddle.net> Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+patch=linaro.org@nongnu.org>
Series	tcg tb_lock removal \| expand [v6,00/50] tcg tb_lock removal [v6,01/50] tcg: Merge opcode arguments into TCGOp [v6,02/50] tcg: Propagate args to op->args in optimizer [v6,03/50] tcg: Propagate args to op->args in tcg.c [v6,04/50] tcg: Propagate TCGOp down to allocators [v6,05/50] tcg: Introduce arg_temp [v6,06/50] tcg: Add temp_global bit to TCGTemp [v6,07/50] tcg: Return NULL temp for TCG_CALL_DUMMY_ARG [v6,08/50] tcg: Introduce temp_arg [v6,09/50] tcg: Use per-temp state data in liveness [v6,10/50] tcg: Avoid loops against variable bounds [v6,11/50] tcg: Change temp_allocate_frame arg to TCGTemp [v6,12/50] tcg: Remove unused TCG_CALL_DUMMY_TCGV [v6,13/50] tcg: Export temp_idx [v6,14/50] tcg: Use per-temp state data in optimize [v6,15/50] tcg: Push tcg_ctx into generator functions [v6,16/50] tcg: Push tcg_ctx into tcg_gen_callN [v6,17/50] tcg: Introduce index_arg [v6,18/50] tcg: Reserve temporary index 0 [v6,19/50] target/alpha: Avoid translate_init unless tcg_enabled [v6,20/50] qom: Introduce CPUClass.tcg_initialize [v6,21/50] tcg: Use pointers in TCGOp->args [v6,22/50] tcg: define CF_PARALLEL and use it for TB hashing along with CF_COUNT_MASK [v6,23/50] hack dump tb->flags and tb->cflags [v6,24/50] tcg: Add CPUState step_next_tb [v6,25/50] tcg: Include CF_COUNT_MASK in CF_HASH_MASK [v6,26/50] tcg: convert tb->cflags reads to tb_cflags(tb) [v6,27/50] target/arm: check CF_PARALLEL instead of parallel_cpus [v6,28/50] target/hppa: check CF_PARALLEL instead of parallel_cpus [v6,29/50] target/i386: check CF_PARALLEL instead of parallel_cpus [v6,30/50] target/m68k: check CF_PARALLEL instead of parallel_cpus [v6,31/50] target/s390x: check CF_PARALLEL instead of parallel_cpus [v6,32/50] target/sh4: check CF_PARALLEL instead of parallel_cpus [v6,33/50] target/sparc: check CF_PARALLEL instead of parallel_cpus [v6,34/50] tcg: check CF_PARALLEL instead of parallel_cpus [v6,35/50] cpu-exec: lookup/generate TB outside exclusive region during step_atomic [v6,36/50] tcg: Add CF_LAST_IO + CF_USE_ICOUNT to CF_HASH_MASK [v6,37/50] tcg: Remove CF_IGNORE_ICOUNT [v6,38/50] translate-all: use a binary search tree to track TBs in TBContext [v6,39/50] exec-all: rename tb_free to tb_remove [v6,40/50] translate-all: report correct avg host TB size [v6,41/50] tcg: take tb_ctx out of TCGContext [v6,42/50] tcg: define tcg_init_ctx and make tcg_ctx a pointer [v6,43/50] gen-icount: fold exitreq_label into TCGContext [v6,44/50] tcg: introduce **tcg_ctxs to keep track of all TCGContext's [v6,45/50] tcg: distribute profiling counters across TCGContext's [v6,46/50] tcg: allocate optimizer temps with tcg_malloc [v6,47/50] osdep: introduce qemu_mprotect_rwx/none [v6,48/50] translate-all: use qemu_protect_rwx/none helpers [v6,49/50] tcg: introduce regions to split code_gen_buffer [v6,50/50] tcg: enable multiple TCG contexts in softmmu

Commit Message

Richard Henderson Oct. 16, 2017, 5:25 p.m. UTC

From: Richard Henderson <rth@twiddle.net>


Signed-off-by: Richard Henderson <rth@twiddle.net>

---
 tcg/tcg.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.13.6

Comments

Emilio Cota Oct. 17, 2017, 8:56 p.m. UTC | #1

On Mon, Oct 16, 2017 at 10:25:26 -0700, Richard Henderson wrote:
> From: Richard Henderson <rth@twiddle.net>

> 

> Signed-off-by: Richard Henderson <rth@twiddle.net>

> ---

>  tcg/tcg.h | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/tcg/tcg.h b/tcg/tcg.h

> index fc4d1ed58b..5fcdec1fc5 100644

> --- a/tcg/tcg.h

> +++ b/tcg/tcg.h

> @@ -731,7 +731,7 @@ extern bool parallel_cpus;

>  

>  static inline TCGTemp *arg_temp(TCGArg a)

>  {

> -    return &tcg_ctx.temps[a];

> +    return a == TCG_CALL_DUMMY_ARG ? NULL : &tcg_ctx.temps[a];

>  }


Reviewed-by: Emilio G. Cota <cota@braap.org>


		E.

diff mbox series

Patch

diff --git a/tcg/tcg.h b/tcg/tcg.h
index fc4d1ed58b..5fcdec1fc5 100644
--- a/tcg/tcg.h
+++ b/tcg/tcg.h
@@ -731,7 +731,7 @@  extern bool parallel_cpus;
 
 static inline TCGTemp *arg_temp(TCGArg a)
 {
-    return &tcg_ctx.temps[a];
+    return a == TCG_CALL_DUMMY_ARG ? NULL : &tcg_ctx.temps[a];
 }
 
 static inline void tcg_set_insn_param(int op_idx, int arg, TCGArg v)