Code Review for 6879921

Prepared by:cfang on Tue Sep 15 10:01:56 PDT 2009
Workspace:/home/cfang/hg/6879921
Compare against: http://hg.openjdk.java.net/jdk7/hotspot-comp/hotspot
Summary of changes: 6 lines changed: 6 ins; 0 del; 0 mod; 2224 unchg
Patch of changes: 6879921.patch
Author comments:
Problem:
We do not intend to apply superword to non-primitive types. Current SuperWord
implementation filters out non-primitives when it creates the initial set of
packs through "find_adjacent_refs". However, when it extends the packlists
using the def-use or use-def chains, non-primitives may still be put into
consideration, and this leads to the assertion failure when we compute the
data size of a T_ADDRESS type (bug 6879921).

Solution:
Filter out non-primitives when we extend the initial packlists (i.e. don't consider
packing non-primitive defs and uses).

Fixes 6879921: CTW failure jdk6_18/hotspot/src/share/vm/utilities/globalDefinitions.cpp:268

Tests:
JPRT and CTW
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/superword.cpp

6 lines changed: 6 ins; 0 del; 0 mod; 2224 unchg

This code review page was prepared using /home/cfang/bin/tom_webrev (vers 23.18-hg-never).