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