RFR  8217467 (XS): Access barriers are missing in C2 intrinsic for Base64
vladimir.kozlov at oracle.com
Tue Jan 22 19:56:03 UTC 2019
Yes, changes are good. I approved it for push into JDK 12.
On 1/22/19 2:48 AM, Roman Kennke wrote:
> Looks good. Thanks!
>> (correct title)
>> On 1/22/19 11:27 AM, Aleksey Shipilev wrote:
>>> This is found and verified by Shenandoah CTW tests that verifies barrier placement. Base64 intrinsic
>>> is new, and only enabled on modern hardware (I think you need AVX512). I'd like to push this fix to
>>> Testing: Shenandoah CTW tests, hotspot tier1 (includes compiler/intrinsics/base64), jdk-submit12
More information about the hotspot-compiler-dev