Roland Westrelin
Roland Westrelin
> You could also add the regression tests from the duplicated issue [JDK-8298851](https://bugs.openjdk.org/browse/JDK-8298851). I added one of them because it doesn't seem to need `StressGCM`. Does it really make sense...
> I guess the issue is that ConvL2I and ConvI2L are also type nodes, which can restrict their type, just like CastII nodes. And that restricting of the type is...
Before split if: ``` long i = 100; for (; i > 0;) { // i here is 1..100 int j = (int)i; // ConvL2I type is 1..100, same as...
> @rwestrel which "split_if" optimization was applied in your example? Split the ConvI2L through the phi? If so, the problem seems to be that the ConvI2L floats by the exit-check,...
FTR, I double checked that fuzzer test failures from JDK-8298851 are indeed the same issue and are fixed with this.
> Otherwise, looks good! Thanks for reviewing this.
Thanks for the reviews @chhagedorn @vnkozlov