review request (S) 6866585 debug code in ciObjectFactory too slow

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Fri Aug 7 13:08:23 PDT 2009



On Aug 7, 2009, at 1:04 PM, John Coomes wrote:

> John Rose (John.Rose at Sun.COM) wrote:
>> On Aug 6, 2009, at 2:15 PM, John Coomes wrote:
>>> In the short-term, I just want to be able to check in my test case.
>>> How about if I restore the debugging code, but put it under  
>>> control of
>>> the CIObjectFactoryVerify option?
>>> I'll also file a bug to capture the comments.
>> Good.  I very much like Tom/Keith/Ramki's suggestions of simplifying
>> the code and reducing perm. distinctions.
> I interpret this as you're ok with putting the debugging code under
> control of the option for now, as long as I capture the discussion in
> a bug so it gets fixed properly.  Tom, you didn't like the idea much.
> Since John is willing to fix it properly--are you ok with it now?
>> This internal CI simplification is a natural add-on to my
>> nonperm-6863023 work.  Shall I roll it in, or make a separate bug?
> I'd think of your reviewers--what would be easier for them?  IMHO,
> smaller is usually better.
> -John

More information about the hotspot-compiler-dev mailing list