| Prepared by: | Thomas Rodriguez (never) on Tue Mar 18 13:51:10 PDT 2008 |
|---|---|
| Workspace: | /net/smite.sfbay/export/ws/hs12 |
| Compare against: | /net/jano2.sfbay/export2/hotspot/hg/hotspot-comp.clean |
| Compare against version: | 49 |
| Summary of changes: | 11 lines changed: 10 ins; 0 del; 1 mod; 3506 unchg |
| Patch of changes: | 6676841.patch |
| Author comments: |
6676841: ClearArrayNode::Identity is incorrect for 64-bit Summary: ClearArrayNode::Identity should use TypeX instead of TypeInt Reviewed-by: ClearArray::identity assumes that the count of words to clear is an int when in fact it is an intptr_t so it's a long in 64-bit. This inhibits elimination of useless ClearArrays. The fix is to use TypeX instead of TypeInt. Additionally in many cases it's possible to know at creation time there is nothing to do so we shouldn't bother to create them in those cases. |
| Bug id: | 6676841: ClearArrayNode::Identity is incorrect for 64-bit |
| Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/memnode.cpp
11 lines changed: 10 ins; 0 del; 1 mod; 3506 unchg
This code review page was prepared using /net/smite.sfbay/never/bin/hgwebrev (vers 23.12-hg-never).