RFR: 8213795: Force explicit null check on patching placeholder offset
adinn at redhat.com
Tue Nov 13 16:14:21 UTC 2018
On 13/11/2018 15:16, Roman Kennke wrote:
> With current logic after JDK-8213199, we allow implicit null-checks in
> the offset range [-cell_header_size;vm_page_size). When using
> Shenandoah, cell_header_size is -8, so we allow [-8;vm_page_size).
> Unfortunately, this disables explicit null-checks on -1 which is used as
> placeholder for offsets to be patched in C1. This results in weird
> asserts later and may actually crash if offset is outside of legal range
> for implicit null-checks. We need to force explicit null checks on -1.
> Testing: tier1 and Shenandoah testing
> Can I please get a review?
Senior Principal Software Engineer
Red Hat UK Ltd
Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander
More information about the hotspot-compiler-dev