[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 12/23] target/loongarch: Scrutinise TCG bitops translation
From: |
Jiaxun Yang |
Subject: |
Re: [PATCH v2 12/23] target/loongarch: Scrutinise TCG bitops translation for 32 bit build |
Date: |
Thu, 26 Dec 2024 22:08:38 +0000 |
在2024年12月26日十二月 下午9:55,Richard Henderson写道:
[...]
> While this allows the code to compile, (1) the functions are unused and
> (2) they do not
> compute the required results. For me, the latter is concerning.
>
> I'd suggest moving GEN_FALSE_TRANS out of trans_privileged.c.inc, then
>
> #ifdef TARGET_LOONGARCH64
> // all gen_foo_d
> TRANS(bytepick_d, 64, gen_rrr_sa, EXT_NONE, EXT_NONE, gen_bytepick_d)
> // etc
> #else
> GEN_FALSE_TRANS(bytepick_d)
> // etc
> #endif
Thanks for the insight!
I'm actually thinking about introducing an TRANS64 which resolves to
GEN_FALSE_TRANS if not on TARGET_LOONGARCH64.
Will try in next version.
Thanks
>
>
> r~
--
- Jiaxun
- [PATCH v2 08/23] target/loongarch: Fix some modifiers for log formatting, (continued)
- [PATCH v2 08/23] target/loongarch: Fix some modifiers for log formatting, Jiaxun Yang, 2024/12/26
- [PATCH v2 09/23] target/loongarch: Use target_ulong for CSR helpers, Jiaxun Yang, 2024/12/26
- [PATCH v2 10/23] target/loongarch: Scrutinise TCG float translation for 32 bit build, Jiaxun Yang, 2024/12/26
- [PATCH v2 11/23] target/loongarch: Scrutinise TCG vector translation for 32 bit build, Jiaxun Yang, 2024/12/26
- [PATCH v2 12/23] target/loongarch: Scrutinise TCG bitops translation for 32 bit build, Jiaxun Yang, 2024/12/26
- [PATCH v2 13/23] target/loongarch: Fix rdtimer on 32bit build, Jiaxun Yang, 2024/12/26
- [PATCH v2 14/23] target/loongarch: Scrutinise TCG arithmetic translation for 32 bit build, Jiaxun Yang, 2024/12/26
- [PATCH v2 15/23] target/loongarch: Fix load type for gen_ll, Jiaxun Yang, 2024/12/26
- [PATCH v2 16/23] target/loongarch: Define address space information for LoongArch32, Jiaxun Yang, 2024/12/26
- [PATCH v2 17/23] target/loongarch: Refactoring is_la64/is_va32 for LoongArch32, Jiaxun Yang, 2024/12/26
- [PATCH v2 18/23] target/loongarch: ifdef out 64 bit CPUs on 32 bit builds, Jiaxun Yang, 2024/12/26