Prepared by: | never on Thu Dec 15 12:04:04 PST 2011 |
---|---|
Workspace: | /net/smite.us.oracle.com/export/ws/typek |
Compare against: | /net/smite.us.oracle.com/export/ws/baseline |
Summary of changes: | 7 lines changed: 2 ins; 0 del; 5 mod; 3239 unchg |
Patch of changes: | 7121073.patch |
Author comments: |
7121073: secondary_super_cache memory slice has incorrect bounds in flatten_alias_type Reviewed-by: To ensure that memory operations for type checks are properly ordered, several fields are smeared together into the same alias slice. The existing logic is sloppy and includes a slightly larger chunk of memory than intended. Currently this is fine but changes to the shape of Klass can invalidate it. The fix is to properly test the boundaries. Additionally I changed the logic for setting the PTR to use NotNull instead of Constant to be consisten with all the following logic which uses NotNull. Tested with full CTW. |
Bug id: | Bug Database |
Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/compile.cpp
7 lines changed: 2 ins; 0 del; 5 mod; 3239 unchg
This code review page was prepared using /never/bin/webrev (vers 23.18-hg-never-dev).