RFR: JDK-8241503: C2: Share MacroAssembler between mach nodes during code emission [v6]

Cesar Soares Lucas cslucas at openjdk.org
Tue Jan 2 23:15:52 UTC 2024


On Wed, 20 Dec 2023 19:23:47 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Cesar Soares Lucas has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
>> 
>>  - Merge with origin/master
>>  - Fix build, copyright dates, m4 files.
>>  - Fix merge
>>  - Catch up with master branch.
>>    
>>    Merge remote-tracking branch 'origin/master' into reuse-macroasm
>>  - Some inst_mark fixes; Catch up with master.
>>  - Catch up with changes on master
>>  - Reuse same C2_MacroAssembler object to emit instructions.
>
> src/hotspot/cpu/x86/x86_32.ad line 1541:
> 
>> 1539:   // in the MacroAssembler. Should go away once all "instruct" are
>> 1540:   // patched to emit bytes only using methods in MacroAssembler.
>> 1541:   enc_class SetInstMark %{
> 
> Do you have separate RFE for that?

Created this one: https://bugs.openjdk.org/browse/JDK-8322876

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16484#discussion_r1439937293


More information about the shenandoah-dev mailing list