RFR: 8202640: Small C1 cleanups for BarrierSetC1
erik.osterlund at oracle.com
Fri May 4 19:42:57 UTC 2018
Thank you for reviewing.
This change does not change anything for any other platforms. So I do not think RH and SAP needs to try this change.
> On 4 May 2018, at 19:18, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> Hi Erik,
> changes look reasonable. Should you change other platforms too since you changed API? At least asks RH and SAP test these changes.
>> On 5/4/18 2:33 AM, Erik Österlund wrote:
>> 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.
More information about the hotspot-compiler-dev