[9] RFR (S): 8029305: add type tag to AbstractCompiler

[Christian Thalinger]

Thanks, Igor.

On Jan 6, 2014, at 2:23 PM, Igor Veresov <igor.veresov at oracle.com> wrote:

> Looks good.
> 
> igor
> 
> On Jan 6, 2014, at 2:13 PM, Christian Thalinger <christian.thalinger at oracle.com> wrote:
> 
>> https://bugs.openjdk.java.net/browse/JDK-8029305
>> http://cr.openjdk.java.net/~twisti/8029305
>> 
>> 8029305: add type tag to AbstractCompiler
>> Reviewed-by:
>> 
>> Add a closed set of concrete compiler classes. Using an tag like this avoids a confusing use of macros around the definition of the is_<compiler type> methods.
>> 
>> As a followup to this change I’m going to replace nmethod::_compiler with a flags field that stores AbstractCompiler::Type values instead:
>> 
>> [#JDK-8031211] replace nmethod::_compiler with a flags field that stores AbstractCompiler::Type values instead
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140106/449da548/attachment.html