RFR(S): 8214862: assert(proj != __null) at compile.cpp:3251

Roland Westrelin rwestrel at redhat.com
Fri Dec 7 09:41:38 UTC 2018


Thanks for looking at this, Vladimir.

> Why relaxing assert is not enough? Why you want to delay edge removal?

Relaxing the assert causes the compilation to fail with malformed
control flow (not retryable). My change would cause the compilation to
fail with infinite loop (not retryable). So the end result is the same,
indeed.

Without my change, the IR graph will have IfNodes with a single
projection (for instance). So an "illegal" graph. Wouldn't we want to
keep the graph legal?

Roland.


More information about the hotspot-compiler-dev mailing list