RFR: 8285711: riscv: RVC: Support disassembler show-bytes option [v2]
Xiaolin Zheng
xlinzheng at openjdk.java.net
Fri Apr 29 04:42:45 UTC 2022
On Fri, 29 Apr 2022 04:30:53 GMT, Fei Yang <fyang at openjdk.org> wrote:
>> Thank you for the suggestion, Felix -- changed, and hope it looks good.
>>
>> One another thing is the `instructions are stored in memory as a sequence of 16-bit little-endian parcels, regardless of memory system endianness` -- I was wondering if the current `Assembler::emit() -> Assembler::emit_int32()` could match it. Also when loading a 32-bit instruction from the memory - I think maybe the 16-bit parcels should be loaded, considering the 16-bit little-endian order, and be combined into one 32-bit instruction? - I cannot find a big-endian simulator to test that, and feel glad to receive any input.
>
> I don't quite understand what you mean here. But let's discuss that as a separate issue when you have more details.
It seems I failed to say it clear, but I also feel glad putting it afterward, for it might have nothing to do with this patch.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8421
More information about the hotspot-compiler-dev
mailing list