RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics

[Nils Eliasson]

In vmSymbols.cpp together with the other flag checks.

Regards,
Nils

On 2016-04-20 02:44, Deshpande, Vivek R wrote:
>
> HI Nils
>
> Yes you are right the function accesses the command line flag 
> DisableIntrinsic and changes are static.
>
> Could you point me the right location for the function ?
>
> Also I have updated the webrev with rest of the comments here:
>
> http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.01/
>
> Regards,
>
> Vivek
>
> *From:*hotspot-compiler-dev 
> [mailto:hotspot-compiler-dev-bounces at openjdk.java.net] *On Behalf Of 
> *Nils Eliasson
> *Sent:* Tuesday, April 19, 2016 5:55 AM
> *To:* hotspot-compiler-dev at openjdk.java.net
> *Subject:* Re: RFR (S): 8154473: Update for CompilerDirectives to 
> control stub generation and intrinsics
>
> Hi Vivek,
>
> The changes in is_intrinsic_disabled in compilerDirectives.* are 
> static and only access the command line flag DisableIntrinsics. As 
> long as stubs are only generated during startup and don't have a 
> method context - that is ok - but it doesn't belong in the 
> compilerDirectives-files if it doens't use directives.
>
> Regards,
> Nils
>
> On 2016-04-18 19:38, Deshpande, Vivek R wrote:
>
>     Hi all
>
>     I would like to contribute a patch which helps to control the
>     intrinsics in interpreter, c1 and c2 by disabling the stub generation.
>
>     This uses -XX:DisableIntrinsic option to achieve the same.
>
>     Could you please review and sponsor this patch.
>
>     Bug-id:
>
>     https://bugs.openjdk.java.net/browse/JDK-8154473
>     webrev:
>
>     http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.00/
>     <http://cr.openjdk.java.net/%7Evdeshpande/CompilerDirectives/8154473/webrev.00/>
>
>     Thanks and regards,
>
>     Vivek
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160420/de0cf911/attachment.html>