Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work

[John Rose]

On Feb 20, 2013, at 10:20 AM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:

> I noticed that JVM_CONSTANT_Object is no longer used so reused that constant to be PseudoString.   I had an edit to go in the other direction (if a pseudo-string is patched into a constant pool, make it a JVM_CONSTANT_Object), but this didn't fully capture that the constant pool slot also points to a Symbol.

I think the change to is_loadable_constant in constantTag is wrong.  Pseudo-strings are loadable constants.  That's what they are there for (ldc #my_magic_oop).

— John


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20130220/f86542d5/attachment.html