| Prepared by: | never on Wed Aug 12 15:20:51 PDT 2009 |
|---|---|
| Workspace: | /net/smite.sfbay/export/ws/postaloc |
| Compare against: | /net/smite.sfbay/export/ws/baseline |
| Summary of changes: | 31 lines changed: 21 ins; 4 del; 6 mod; 1100 unchg |
| Patch of changes: | 6862863.patch |
| Author comments: |
The fix for 5032515 included copy elimination on constants but the new code was missing a check for dead values in the regnd array. This allowed it to resurrect a dead value leading to later failure during copy elimination. The fix is to always check for this case instead of having each use do so. Tested with failing replay compile and full CTW. I can't see any way to build a test case for this. |
| Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/chaitin.hpp
10 lines changed: 10 ins; 0 del; 0 mod; 511 unchg
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/postaloc.cpp
21 lines changed: 11 ins; 4 del; 6 mod; 589 unchg
This code review page was prepared using /net/smite.sfbay/never/bin/webrev (vers 23.18-hg-never).