Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work (original) (raw)
John Rose john.r.rose at oracle.com
Wed Feb 20 12:13:46 PST 2013
- Previous message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Next message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Feb 20, 2013, at 10:20 AM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:
I noticed that JVMCONSTANTObject 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 JVMCONSTANTObject), 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/serviceability-dev/attachments/20130220/f86542d5/attachment.html
- Previous message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Next message: Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]