[9] RFR(XS): 8086057: Crash with "modified node is not on IGVN._worklist" when running with -XX:-SplitIfBlocks

Tobias Hartmann tobias.hartmann at oracle.com
Fri Apr 22 05:19:06 UTC 2016

Thanks, Vladimir!

Best regards,

On 22.04.2016 00:39, Vladimir Kozlov wrote:
> Good.
> Thanks,
> Vladimir
> On 4/21/16 3:11 AM, Tobias Hartmann wrote:
>> Hi,
>> please review the following patch:
>> https://bugs.openjdk.java.net/browse/JDK-8086057
>> http://cr.openjdk.java.net/~thartmann/8086057/webrev.00/
>> The fastdebug VM crashes with "modified node is not on IGVN._worklist" because SuperWord::align_initial_loop_index() resets the input of the pre-loop Opaque1 node 'pre_opaq' without putting it on the IGVN worklist afterwards. This only shows up with -XX:-SplitIfBlocks because otherwise surrounding nodes are changed causing the OpagueNode to be put on the worklist. The method should use PhaseIterGVN::replace_input_of() which uses rehash_node_delayed() to ensure the modified node is put on the worklist.
>> Tested with regression test, Nashorn+Octane with -XX:-SplitIfBlocks and RBT (running).
>> Thanks,
>> Tobias

More information about the hotspot-compiler-dev mailing list