RFR 8035735: Metaspace::contains become extremely slow in some cases (original) (raw)
Coleen Phillimore coleen.phillimore at oracle.com
Fri Feb 28 14:47:36 PST 2014
- Previous message: RFR 8035735: Metaspace::contains become extremely slow in some cases
- Next message: Re.: CHA ignores default methods during analysis leading to incorrect code generation
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 2/28/2014 5:03 PM, Daniel D. Daugherty wrote:
On 2/28/14 1:08 PM, Coleen Phillimore wrote:
Summary: Call ismetadata instead which does less work for the call in debugInfo.hpp which is called for all compiled code stack frames.
open webrev at http://cr.openjdk.java.net/~coleenp/8035735/ src/share/vm/code/debugInfo.hpp line 276: // ismetadata is a faster check In the context of the current code review, the comment is fine, but when you read it down the road (by itself), you'll ask yourself: Faster than what? :-) Perhaps: // ismetadata() is a faster check than ismetaspaceobject()
Thanks - you are right. I updated the comment.
Thank you for the code review. Coleen
src/share/vm/oops/metadata.hpp No comments. Thumbs up, modulo Jon M's question... Dan
bug link https://bugs.openjdk.java.net/browse/JDK-8035735 Tested with failing test case, nsk.quick.testlist, jcks and jtreg tests. Coleen
- Previous message: RFR 8035735: Metaspace::contains become extremely slow in some cases
- Next message: Re.: CHA ignores default methods during analysis leading to incorrect code generation
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]