Re: Push invalidation + bimorphic fix

2011-08-09 Thread Christian Thalinger
On Aug 9, 2011, at 7:58 AM, Charles Oliver Nutter wrote: I have set up a local build against hsx hotspot compiler dev branch, thanks to Christian. I'll try to write something up with the process tomorrow. Anyway, I wanted to test out Tom's patch for GWT bimorphic inlining and Christian's

Re: Request for review (L): 7071653: JSR 292: call site change notification should be pushed not pulled

2011-08-09 Thread Christian Thalinger
On Aug 8, 2011, at 8:49 PM, Tom Rodriguez wrote: dependencies.cpp: in check_call_site_target_value, the changes == NULL case should be checking that the call site hasn't changed. It should probably look more like this: klassOop Dependencies::check_call_site_target_value(klassOop ctxk,

Re: Push invalidation + bimorphic fix

2011-08-09 Thread Charles Oliver Nutter
On Tue, Aug 9, 2011 at 2:31 PM, Charles Oliver Nutter head...@headius.com wrote: On Tue, Aug 9, 2011 at 4:30 AM, Christian Thalinger christian.thalin...@oracle.com wrote: Hmm.  I remember you were saying this is some old Ultra 20.  The only box I could find which is similar is a 2.0GHz