diff options
| author | David Hildenbrand | 2019-08-14 17:12:42 +0200 |
|---|---|---|
| committer | Cornelia Huck | 2019-08-22 14:53:49 +0200 |
| commit | 25bcb45d1b81d22634daa2b1a2d8bee746ac129b (patch) | |
| tree | 4a8a19c71f9f79080b533988c1da91135f4ea6ba /target | |
| parent | Merge remote-tracking branch 'remotes/armbru/tags/pull-monitor-2019-08-21' in... (diff) | |
| download | qemu-25bcb45d1b81d22634daa2b1a2d8bee746ac129b.tar.gz qemu-25bcb45d1b81d22634daa2b1a2d8bee746ac129b.tar.xz qemu-25bcb45d1b81d22634daa2b1a2d8bee746ac129b.zip | |
s390x/tcg: Fix VERIM with 32/64 bit elements
Wrong order of operands. The constant always comes last. Makes QEMU crash
reliably on specific git fetch invocations.
Reported-by: Stefano Brivio <sbrivio@redhat.com>
Signed-off-by: David Hildenbrand <david@redhat.com>
Message-Id: <20190814151242.27199-1-david@redhat.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Fixes: 5c4b0ab460ef ("s390x/tcg: Implement VECTOR ELEMENT ROTATE AND INSERT UNDER MASK")
Cc: qemu-stable@nongnu.org
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
Diffstat (limited to 'target')
| -rw-r--r-- | target/s390x/translate_vx.inc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target/s390x/translate_vx.inc.c b/target/s390x/translate_vx.inc.c index 41d5cf869f..0caddb3958 100644 --- a/target/s390x/translate_vx.inc.c +++ b/target/s390x/translate_vx.inc.c @@ -213,7 +213,7 @@ static void get_vec_element_ptr_i64(TCGv_ptr ptr, uint8_t reg, TCGv_i64 enr, vec_full_reg_offset(v3), ptr, 16, 16, data, fn) #define gen_gvec_3i(v1, v2, v3, c, gen) \ tcg_gen_gvec_3i(vec_full_reg_offset(v1), vec_full_reg_offset(v2), \ - vec_full_reg_offset(v3), c, 16, 16, gen) + vec_full_reg_offset(v3), 16, 16, c, gen) #define gen_gvec_4(v1, v2, v3, v4, gen) \ tcg_gen_gvec_4(vec_full_reg_offset(v1), vec_full_reg_offset(v2), \ vec_full_reg_offset(v3), vec_full_reg_offset(v4), \ |