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