Code Review for 6953539

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).