[lworld] RFR: 8367156: [lworld] MacroAssembler::remove_frame hits "Field too big for insn" [v2]

Christian Hagedorn chagedorn at openjdk.org
Fri Sep 12 16:23:52 UTC 2025


On Fri, 12 Sep 2025 16:17:39 GMT, Marc Chevalier <mchevalier at openjdk.org> wrote:

>> I thought so too, and I tried without and it doesn't work. I'm using actually `MyValue1` that is defined in the same package, as we do in a lot of valhalla tests. And this one include a lot of things that needs both these paths (I've tried with each alone).
>
> Without that, I would also not need `@modules java.base/jdk.internal.vm.annotation`. Alas, it seems needed...

I see! Then let's keep that.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1575#discussion_r2344770849


More information about the valhalla-dev mailing list