[9] RFR(S): 8138651: -XX:DisableIntrinsic matches intrinsics overly eagerly

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Oct 7 02:37:57 UTC 2015

To be precise DisableIntrinsic is ccstrlist option, not ccstr. Yes, the 
actual type is the same.

An other concern is separators since format could be different if option 
specified in file. Look how we do search in DeoptimizeOnlyAt string.


On 10/6/15 8:00 PM, Zoltán Majó wrote:
> Hi,
> please review the patch for JDK-8138651.
> https://bugs.openjdk.java.net/browse/JDK-8138651
> Problem: The DisableIntrinsic flag does not disable intrinsics
> accurately. For example, -XX:DisableIntrinsic=_copyOfRange disables both
> the intrinsic with the ID _copyOfRange and the intrinsic with the _copyOf.
> Solution: Change the processing of the DisableIntrinsic flag (both
> globally and on a per-method level).
> Webrev: http://cr.openjdk.java.net/~zmajo/8138651/webrev.00/
> Testing:
> - JPRT (testset hotspot);
> - executed the the newly added test
> compiler/intrinsics/IntrinsicDisabledTest.java with/without the fix on
> all platforms, the test behaves as expected.
> Thank you and best regards,
> Zoltan

More information about the hotspot-compiler-dev mailing list