diff options
| author | Alex Bennée <alex.bennee@linaro.org> | 2022-02-09 11:21:42 +0000 |
|---|---|---|
| committer | Richard Henderson <richard.henderson@linaro.org> | 2022-03-04 08:50:41 -1000 |
| commit | 9becc36f0285da206150bc4f0c774f3298aaa439 (patch) | |
| tree | 0ca2bc8990c6fc0d712c2bbfc1cfeee5589feea4 | |
| parent | 3d1fbc59665ff8a5d74b0fd30583044fe99e1117 (diff) | |
| download | qemu-9becc36f0285da206150bc4f0c774f3298aaa439.zip qemu-9becc36f0285da206150bc4f0c774f3298aaa439.tar.gz qemu-9becc36f0285da206150bc4f0c774f3298aaa439.tar.bz2 | |
tcg/optimize: only read val after const check
valgrind pointed out that arg_info()->val can be undefined which will
be the case if the arguments are not constant. The ordering of the
checks will have ensured we never relied on an undefined value but for
the sake of completeness re-order the code to be clear.
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20220209112142.3367525-1-alex.bennee@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
| -rw-r--r-- | tcg/optimize.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tcg/optimize.c b/tcg/optimize.c index e573000..06213fd 100644 --- a/tcg/optimize.c +++ b/tcg/optimize.c @@ -552,10 +552,10 @@ static bool do_constant_folding_cond_eq(TCGCond c) static int do_constant_folding_cond(TCGType type, TCGArg x, TCGArg y, TCGCond c) { - uint64_t xv = arg_info(x)->val; - uint64_t yv = arg_info(y)->val; - if (arg_is_const(x) && arg_is_const(y)) { + uint64_t xv = arg_info(x)->val; + uint64_t yv = arg_info(y)->val; + switch (type) { case TCG_TYPE_I32: return do_constant_folding_cond_32(xv, yv, c); @@ -567,7 +567,7 @@ static int do_constant_folding_cond(TCGType type, TCGArg x, } } else if (args_are_copies(x, y)) { return do_constant_folding_cond_eq(c); - } else if (arg_is_const(y) && yv == 0) { + } else if (arg_is_const(y) && arg_info(y)->val == 0) { switch (c) { case TCG_COND_LTU: return 0; |