Integrated: 8241503: C2: Share MacroAssembler between mach nodes during code emission
Cesar Soares Lucas
cslucas at openjdk.org
Thu Apr 11 15:47:53 UTC 2024
On Thu, 2 Nov 2023 22:17:43 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:
> # Description
>
> Please review this PR with a patch to re-use the same C2_MacroAssembler object to emit all instructions in the same compilation unit.
>
> Overall, the change is pretty simple. However, due to the renaming of the variable to access C2_MacroAssembler, from `_masm.` to `masm->`, and also some method prototype changes, the patch became quite large.
>
> # Help Needed for Testing
>
> I don't have access to all platforms necessary to test this. I hope some other folks can help with testing on `S390`, `RISC-V` and `PPC`.
This pull request has now been integrated.
Changeset: 31ee5108
Author: Cesar Soares Lucas <cslucas at openjdk.org>
Committer: Martin Doerr <mdoerr at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/31ee5108e059afae0a3809947adb7b91e19baec6
Stats: 2144 lines in 60 files changed: 118 ins; 431 del; 1595 mod
8241503: C2: Share MacroAssembler between mach nodes during code emission
Reviewed-by: kvn, mdoerr, amitkumar, lucy
-------------
PR: https://git.openjdk.org/jdk/pull/16484
More information about the shenandoah-dev
mailing list