review (S) for 6920293: OptimizeStringConcat causing core dumps

[Vladimir Kozlov]

The changes seem correct.

Vladimir

Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/6920293
> 
> 6920293: OptimizeStringConcat causing core dumps
> Reviewed-by:
> 
> The code for the idiom a == null ? "null" : a) in OptimizeStringConcat
> was putting an explicit NOTNULL on the result Phi which allowed the
> optimizer to move some loads above the null check which caused a
> crash.  The fix is to use the same idiom the parser uses which is to
> cast the resulting value to be notnull and leave the phi type alone.
> Tested with failing test from report.  I also made some changes to
> error reporting to make the crash output more useful.  Currently if we
> don't find an implicit exception handler we die inside the VM and
> don't print out much useful information.  Instead we should simply
> return null and let the VMError machinery produce a normal crash dump.
> I also fixed the formatting for the register in 64 bit mode on solaris
> to match the linux formatting.
> 
> src/share/vm/opto/stringopts.cpp
> src/share/vm/runtime/sharedRuntime.cpp
> src/share/vm/code/nmethod.cpp
> src/os_cpu/solaris_x86/vm/os_solaris_x86.cpp