review request (S/M): 6951319 enable Sun Studio 12 update 1

David Holmes David.Holmes at oracle.com
Tue May 11 23:55:23 PDT 2010


John,

First a dumb question :) Are the .il files only used with SunStudio?

For the dtrace tail-call problem won't this:

+ // Work around dtrace tail call bug 6672627 until it is fixed in 
solaris 10.
+ #define HS_DTRACE_WORKAROUND_TAIL_CALL_BUG() \
+   do { volatile size_t dtrace_workaround_tail_call_bug = 1; } while (0)

be elided by any decent compiler - even though volatile it is a local 
and so the assignment is vacuous ?

David

John Coomes said the following on 05/12/10 08:39:
> I'd appreciate reviews for the changes to enable builds using Sun
> Studio 12 update 1 on solaris.  The webrev includes some comments
> describing the various changes.
> 
> 	http://cr.openjdk.java.net/~jcoomes/6951319-ss12u1/
> 
> -John
> 


More information about the hotspot-dev mailing list