Request for review: 8000797: NPG: is_pseudo_string_at() doesn't work (original) (raw)
serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Thu Feb 21 22:52:37 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 ]
Nice catch. Looks good.
Thanks, Serguei
On 2/21/13 9:09 PM, Coleen Phillimore wrote:
I'm sorry about this but I need one more review. I omitted one instance of JVMCONSTANTObject in templateTablesparc.cpp and I forgot that you can't make any edit to the VM without making a duplicate edit to the serviceability agent. open webrev at http://cr.openjdk.java.net/~coleenp/80007973/ Reran runThese jcks and sajdi and mlvm tests on solaris/sparc. Thanks, Coleen On 2/21/2013 6:22 PM, John Rose wrote: On Feb 20, 2013, at 8:11 PM, Coleen Phillimore <coleen.phillimore at oracle.com <mailto:coleen.phillimore at oracle.com>> wrote:
Thanks to John's feedback, I've modified this change again to revert back to my original change to zero out the Symbol* to indicate pseudo string (could be thought of as "patched string"). Also cleaned up unused constant pool Object constant and unused flags. Reran all the tests.
http://cr.openjdk.java.net/~coleenp/80007972/ <http://cr.openjdk.java.net/%7Ecoleenp/80007972/> Good; lots of nice deletions. You can use me as a reviewer. — John
-------------- next part -------------- An HTML attachment was scrubbed... URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20130221/c9ce57e5/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 ]