[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Request for reviews (L): 6695810: null oop passed to encode_heap_oop_not_null
From:       Vladimir.Kozlov () Sun ! COM (Vladimir Kozlov)
Date:       2008-05-15 3:53:07
Message-ID: 482BB3A3.4080102 () sun ! com
[Download RAW message or body]

Note: diffs contains changes (in node.cpp and c2_globals.hpp)
       for 6701887 which I will push first.

http://webrev.invokedynamic.info/kvn/6695810/index.html

Fixed 6695810: null oop passed to encode_heap_oop_not_null

These changes include fixes for problems found in Escape Analysis
and Compressed Oops implementations.

Also they include additional optimizations for Compressed Oops:
  - use the 32-bits gap after klass in a object for
    a narrow oop field,
  - use the 32-bits gap after klass in a object for
    boxing objects value (except Long and Double),
  - use heapOopSize for instanceKlass::_nonstatic_field_size value
    instead of wordSize,
  - add LoadNKlass and CMoveN nodes and use CmpN and ConN nodes
    and add correspondent platform specific assembler instructions
    to generate narrow oops (32-bits) compares for oop type and
    oop NULL checks.

Reviewed by:
Fix verified (y/n): y, failed tests and generated code

Other testing:
JPRT, CTW, nsk tests, refworkload




[prev in list] [next in list] [prev in thread] [next in thread]