[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 09/23] target/loongarch: Use target_ulong for CSR helpers
From: |
Jiaxun Yang |
Subject: |
Re: [PATCH v2 09/23] target/loongarch: Use target_ulong for CSR helpers |
Date: |
Thu, 26 Dec 2024 21:39:31 +0000 |
在2024年12月26日十二月 下午9:31,Richard Henderson写道:
[...]
>>
>> -target_ulong helper_crc32c(target_ulong val, target_ulong m, uint64_t sz)
>> +target_ulong helper_crc32c(target_ulong val, target_ulong m, target_ulong
>> sz)
>> {
>> uint8_t buf[8];
>> target_ulong mask = ((sz * 8) == 64) ? -1ULL : ((1ULL << (sz * 8)) -
>> 1);
>>
>
> Changes to helper definitions, but not to the declarations.
>
> Is this patch a victim of incorrect splitting, or incorrect rebasing?
Aha, those pairs of definitions and declarations are inconsistent in
existing code.
I should reflect this in commit message.
Thanks
>
>
> r~
--
- Jiaxun
- [PATCH v2 04/23] target/loongarch: Perform sign extension for IOCSR reads, (continued)
- [PATCH v2 04/23] target/loongarch: Perform sign extension for IOCSR reads, Jiaxun Yang, 2024/12/26
- [PATCH v2 05/23] target/loongarch: Use target_ulong for iocsrrd helper results, Jiaxun Yang, 2024/12/26
- [PATCH v2 06/23] target/loongarch: Store some uint64_t values as target_ulong, Jiaxun Yang, 2024/12/26
- [PATCH v2 07/23] target/loongarch: Cast address to 64bit before DMW_64_VSEG shift, Jiaxun Yang, 2024/12/26
- [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