RE: [EXTERNAL] Re: [PATCH v2 2/3] [sim/riscv] Add support for compressed integer instruction set

Jaydeep Patil
2023-12-19T06:11:40Z

Mike Frysinger <vapier@gentoo.org> writes:

> i haven't been keeping up with riscv specs. is the compressed extension finalized ? so you're only implementing official insns in the spec ? i don't think it's appropriate for the sim to implement vendor-specific stuff at this point in time.

The compressed instruction set ("c") is not vendor specific. It has been ratified.

> afaict, there is no relationship between the compression & semi-hosting work.
> these are just two things you're working on ? so they don't really need to be in the same patch series.

Yes, there is no relation between compression & semi-hosting work. The patches are independent and I can re-submit them if needed.

> i'm missing something ... why does there need to be tests in gdb at all here ?

Purpose of riscv-insn-simulation.exp is to test both C extension and semi-hosting. However, I have also added sim specific tests in patch v3.

Regards,
Jaydeep