Prepared by: | never on Tue May 18 12:29:24 PDT 2010 |
---|---|
Workspace: | /net/smite.sfbay/export/ws/baseline |
Compare against: | ssh://hg.openjdk.java.net/jdk7/hotspot-comp-gate/hotspot |
Summary of changes: | 5 lines changed: 4 ins; 0 del; 1 mod; 3789 unchg |
Patch of changes: | 6953539.patch |
Author comments: |
6953539: after 6892658 c1 reports that it doesn't inline StringBuffer.append Reviewed-by: 6892658 added intrinsic ids for several StringBuffer/StringBuilder methods which triggered some long dormant logic in try_inline_intrinsic which acts as if it bails out of inlining if an intrinsic is synchronized. It should simply return false to indicate that it didn't do anything and not set the inline bailout message. |
Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/c1/c1_GraphBuilder.cpp
5 lines changed: 4 ins; 0 del; 1 mod; 3789 unchg
This code review page was prepared using /never/bin/webrev (vers 23.18-hg-never).