From 5b620fb698e69a5386f2f02c7c455bdbdd59a52b Mon Sep 17 00:00:00 2001 From: Peter Maydell Date: Wed, 22 Jun 2011 15:16:32 +0100 Subject: [PATCH] exec-all.h: Make MAX_OP_PER_INSTR large enough for target-arm's uses The target-arm frontend's worst-case TCG ops per instr is 194 (and in general many of the "load multiple registers" ARM instructions generate more than 100 TCG ops). Raise MAX_OP_PER_INSTR accordingly to avoid possible buffer overruns. Since it doesn't make any sense for the "64 bit guest on 32 bit host" case to have a smaller limit than the normal case, we collapse the two cases back into each other again. (This increase costs us about 14K in extra static buffer space and 21K of extra margin at the end of a 32MB codegen buffer.) Signed-off-by: Peter Maydell Signed-off-by: Blue Swirl --- exec-all.h | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/exec-all.h b/exec-all.h index 21a69d68a..69acf3b97 100644 --- a/exec-all.h +++ b/exec-all.h @@ -44,11 +44,7 @@ struct TranslationBlock; typedef struct TranslationBlock TranslationBlock; /* XXX: make safe guess about sizes */ -#if (HOST_LONG_BITS == 32) && (TARGET_LONG_BITS == 64) -#define MAX_OP_PER_INSTR 128 -#else -#define MAX_OP_PER_INSTR 96 -#endif +#define MAX_OP_PER_INSTR 208 #if HOST_LONG_BITS == 32 #define MAX_OPC_PARAM_PER_ARG 2