RFR(S): 8166836: Elimination of clone's ArrayCopyNode may make compilation fail silently
vladimir.kozlov at oracle.com
Wed Sep 28 18:38:39 UTC 2016
I thought we do that. There are several places in loopnode.cpp where we
hit assert if graph is bad.
On 9/28/16 3:04 AM, Vladimir Ivanov wrote:
> Looks good.
>> It's quite unfortunate that this wasn't found by testing because
>> compilations where the graph is non schedulable simply fail. This could
>> have gone unnoticed much longer. In debug builds shouldn't we abort the
>> VM in C2Compiler::compile_method() if the compilation fails because of a
>> non schedulable graph?
> Sounds reasonable. I expect there are other cases when compilers bail
> out unexpectely. It would be good to have an assert checking it doesn't
> Best regards,
> Vladimir Ivanov
More information about the hotspot-compiler-dev