| commit | 3ff91d7e85176f8b4b131163d7fd801757a2c949 | |
| author | Joseph Myers <joseph@codesourcery.com> | |
| Thu, 27 Oct 2016 23:35:04 +0000 (27 23:35 +0000) | ||
| committer | Richard Henderson <rth@twiddle.net> | |
| Tue, 1 Nov 2016 16:30:45 +0000 (1 10:30 -0600) | ||
| tree | 9b58e8217513ed2375850b9831762d01aea2ad80 | treesnapshot (tar.gz zip) |
| parent | a40d4701bc9f6e6a3bbfb7b4fbe756a5b72b5df1 | commitdiff |
tcg: correct 32-bit tcg_gen_ld8s_i64 sign-extension
The version of tcg_gen_ld8s_i64 for 32-bit systems does a load into
the low part of the return value - then attempts a sign extension into
the high part, but wrongly sets the high part to a sign extension of
itself rather than of the low part. This results in TCG internal
errors from the use of the uninitialized high part (in some GCC tests
of AArch64 NEON shift intrinsics, in particular). This patch corrects
the sign-extension logic, making it match other functions such as
tcg_gen_ld16s_i64.
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Joseph Myers <joseph@codesourcery.com>
Message-Id: <alpine.DEB.2.20.1610272333560.22353@digraph.polyomino.org.uk>
Signed-off-by: Richard Henderson <rth@twiddle.net>
The version of tcg_gen_ld8s_i64 for 32-bit systems does a load into
the low part of the return value - then attempts a sign extension into
the high part, but wrongly sets the high part to a sign extension of
itself rather than of the low part. This results in TCG internal
errors from the use of the uninitialized high part (in some GCC tests
of AArch64 NEON shift intrinsics, in particular). This patch corrects
the sign-extension logic, making it match other functions such as
tcg_gen_ld16s_i64.
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Joseph Myers <joseph@codesourcery.com>
Message-Id: <alpine.DEB.2.20.1610272333560.22353@digraph.polyomino.org.uk>
Signed-off-by: Richard Henderson <rth@twiddle.net>
| tcg/tcg-op.c | diffblobblamehistory |