Request for reviews (XL): 6894206: JVM needs a way to traverse method handle structures

Christian Thalinger Christian.Thalinger at Sun.COM
Fri Dec 4 07:14:15 PST 2009


On Fri, 2009-12-04 at 00:49 -0800, John Rose wrote:
> P.S.  I don't think there's a portability issue with the long switch.
>  C++ allows any "integral type" (including int64_t) as a switch
> selector.
> 
> 
> I tried a little fault-injection by having the ID3 fields overlap
> completely.  The result was a nice compile-time error about duplicate
> case statements.
> 
> 
> The code would be better with more asserts of the form Vladimir asked
> for.  One extra assert that would be nice would be to have find_id,
> when it returns a hit, immediately check the resulting i-id against
> the i-id query functions, such as in this sketch:

I integrated both changes but I omitted a check for flags as it seems
not to be a simple assert.

http://cr.openjdk.java.net/~twisti/6894206/webrev.04/

-- Christian



More information about the hotspot-compiler-dev mailing list