[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v7 05/13] target/hexagon: introduce new helper functions
From: |
Taylor Simpson |
Subject: |
RE: [PATCH v7 05/13] target/hexagon: introduce new helper functions |
Date: |
Tue, 21 Dec 2021 18:51:19 +0000 |
> -----Original Message-----
> From: Anton Johansson <anjo@rev.ng>
> Sent: Friday, December 17, 2021 2:01 AM
> To: qemu-devel@nongnu.org
> Cc: ale@rev.ng; Taylor Simpson <tsimpson@quicinc.com>; Brian Cain
> <bcain@quicinc.com>; babush@rev.ng; nizzo@rev.ng;
> richard.henderson@linaro.org
> Subject: [PATCH v7 05/13] target/hexagon: introduce new helper functions
>
> From: Niccolò Izzo <nizzo@rev.ng>
>
> These helpers will be employed by the idef-parser generated code.
> "Helper" can here mean two things, a helper in the QEMU sense added to
> `helper.h` and `op_helper.c`, but also helper functions providing a manual
> TCG implementation of a certain features.
>
> Signed-off-by: Alessandro Di Federico <ale@rev.ng>
> Signed-off-by: Niccolò Izzo <nizzo@rev.ng>
> Signed-off-by: Anton Johansson <anjo@rev.ng>
> ---
> target/hexagon/genptr.c | 166
> +++++++++++++++++++++++++++++++++++--
> target/hexagon/genptr.h | 16 +++-
> target/hexagon/helper.h | 2 +
> target/hexagon/macros.h | 9 ++
> target/hexagon/op_helper.c | 10 +++
> 5 files changed, 195 insertions(+), 8 deletions(-)
>
> diff --git a/target/hexagon/genptr.c b/target/hexagon/genptr.c index
> ae798e921e..44a1ec9ccf 100644
> +void gen_satu_i64_ovfl(TCGv ovfl, TCGv_i64 dest, TCGv_i64 source, int
> +width) {
> + gen_sat_i64(dest, source, width);
Should be gen_satu_i64 (unsigned)
> + TCGv_i64 ovfl_64 = tcg_temp_new_i64();
> + tcg_gen_setcond_i64(TCG_COND_NE, ovfl_64, dest, source);
> + tcg_gen_trunc_i64_tl(ovfl, ovfl_64);
> + tcg_temp_free_i64(ovfl_64);
> +}
- [PATCH v7 08/13] target/hexagon: import flex/bison to docker files, (continued)
- [PATCH v7 08/13] target/hexagon: import flex/bison to docker files, Anton Johansson, 2021/12/17
- [PATCH v7 09/13] target/hexagon: import lexer for idef-parser, Anton Johansson, 2021/12/17
- [PATCH v7 04/13] target/hexagon: make helper functions non-static, Anton Johansson, 2021/12/17
- [PATCH v7 03/13] target/hexagon: make slot number an unsigned, Anton Johansson, 2021/12/17
- [PATCH v7 11/13] target/hexagon: call idef-parser functions, Anton Johansson, 2021/12/17
- [PATCH v7 06/13] target/hexagon: expose next PC in DisasContext, Anton Johansson, 2021/12/17
- [PATCH v7 05/13] target/hexagon: introduce new helper functions, Anton Johansson, 2021/12/17
- [PATCH v7 07/13] target/hexagon: prepare input for the idef-parser, Anton Johansson, 2021/12/17
- [PATCH v7 01/13] target/hexagon: update MAINTAINERS for idef-parser, Anton Johansson, 2021/12/17
- [PATCH v7 12/13] target/hexagon: import additional tests, Anton Johansson, 2021/12/17
- [PATCH v7 13/13] gitlab-ci: do not use qemu-project Docker registry, Anton Johansson, 2021/12/17
- [PATCH v7 10/13] target/hexagon: import parser for idef-parser, Anton Johansson, 2021/12/17