RFR: 8202640: Small C1 cleanups for BarrierSetC1
Erik Österlund
erik.osterlund at oracle.com
Fri May 4 09:52:44 UTC 2018
Hi Nils,
Thanks for the review!
/Erik
On 2018-05-04 11:31, Nils Eliasson wrote:
> Looks great!
>
> // Nils
>
>
> On 2018-05-04 11:33, Erik Österlund wrote:
>> Hi,
>>
>> After rebasing ZGC to BarrierSetC1, we noticed a few small cleanups
>> that would be great.
>>
>> * restore_live_registers_except_rax should be exposed so that barrier
>> set backends may use it
>> * save_live_registers_no_oop_map has an unused argument that should
>> be removed
>> * LIR_OpVisitState::visit should allow visiting LIR with a virtual
>> call (for GC barrier LIR_Ops).
>> * LIR_List::append ought to be public so that barrier set backends
>> can add barrier LIR_Ops.
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8202640
>>
>> Webrev:
>> http://cr.openjdk.java.net/~eosterlund/8202640/webrev.00/
>>
>> Thanks,
>> /Erik
>
More information about the hotspot-compiler-dev
mailing list