review (S) for 6972540: sun/nio/ch/SocketChannelImpl compilation crashed when executing CompileTheWorld
tom.rodriguez at oracle.com
Wed Sep 22 14:49:42 PDT 2010
On Sep 22, 2010, at 2:46 PM, Vladimir Kozlov wrote:
> Changes looks good.
> Did you test 64-bit C1 which could be affected by these changes?
No i didn't but it's really about abstract typing in the LIR and doesn't effect code generation. I can run CTW if you want.
> Tom Rodriguez wrote:
>> 6972540: sun/nio/ch/SocketChannelImpl compilation crashed when executing CompileTheWorld
>> The fix 6932496 exposed T_ADDRESS as a constant so that deopt with
>> jsr's could work properly. Since it didn't fully expose T_ADDRESS as
>> a support lir type it create type mismatches that showed up in some
>> complex cases. The fix is support T_ADDRESS in the LIR directly.
>> Tested with full CTW.
More information about the hotspot-compiler-dev