(original) (raw)

Looks good.

igor

On Jan 6, 2014, at 2:13 PM, Christian Thalinger <christian.thalinger@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: