RFR: 7903606: Move layout and function descriptor generation closer to code builders [v4]

Mon Dec 11 23:59:42 UTC 2023

On Mon, 11 Dec 2023 21:42:34 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> still don't get it :-) The `layout` variable is a local variable in this method?
>
> Ok, I guess you meant this as an observation: since it's a primitive layout, we just emit JAVA_XYZ inline. This is different from what we do for offset (and size). If you prefer to add a layout constant too, I can add (for readability).

Right sorry, it was just an observation/making sure I'm understanding the code correctly.

-------------

PR Review Comment: https://git.openjdk.org/jextract/pull/156#discussion_r1423250137