6953539: after 6892658 c1 reports that it doesn't inline StringBuffer.append

Tue, 18 May 2010 13:45:03 -0700

author
never
date
Tue, 18 May 2010 13:45:03 -0700
changeset 1895
99791ad65936
parent 1894
c52275c698d1
child 1896
b5fdf39b9749

6953539: after 6892658 c1 reports that it doesn't inline StringBuffer.append
Reviewed-by: kvn, twisti

src/share/vm/c1/c1_GraphBuilder.cpp file | annotate | diff | comparison | revisions
     1.1 --- a/src/share/vm/c1/c1_GraphBuilder.cpp	Tue May 18 09:54:05 2010 -0700
     1.2 +++ b/src/share/vm/c1/c1_GraphBuilder.cpp	Tue May 18 13:45:03 2010 -0700
     1.3 @@ -2978,7 +2978,11 @@
     1.4  
     1.5  bool GraphBuilder::try_inline_intrinsics(ciMethod* callee) {
     1.6    if (!InlineNatives           ) INLINE_BAILOUT("intrinsic method inlining disabled");
     1.7 -  if (callee->is_synchronized()) INLINE_BAILOUT("intrinsic method is synchronized");
     1.8 +  if (callee->is_synchronized()) {
     1.9 +    // We don't currently support any synchronized intrinsics
    1.10 +    return false;
    1.11 +  }
    1.12 +
    1.13    // callee seems like a good candidate
    1.14    // determine id
    1.15    bool preserves_state = false;

mercurial