RFR(S): 8244407: JVM crashes after transformation in C2 IdealLoopTree::split_fall_in

[Yangfei (Felix)]

Thank you Vladimir and Tobias for reviewing this.
I have modified accordingly using {} for the if() body.
Also removed a pair of redundant curly braces in the newly added test case.
Pushed to the submit repo and test results received looks good:

Job: mach5-one-fyang-JDK-8244407-20200511-2157-10959986
BuildId: 2020-05-11-2156071.felix.yang.source
No failed tests
Tasks Summary
•	EXECUTED_WITH_FAILURE: 0
•	NOTHING_TO_RUN: 0
•	KILLED: 0
•	HARNESS_ERROR: 0
•	FAILED: 0
•	PASSED: 101
•	UNABLE_TO_RUN: 0
•	NA: 0

Pushed as: http://hg.openjdk.java.net/jdk/jdk/rev/c9f5a16d6980 

Best regards,
Felix

> Hi, Felix

> Yes, the fix looks good.

> Please, use {} for if() body.

> Thanks,
> Vladimir

> On 5/11/20 1:58 AM, Tobias Hartmann wrote:
>> Hi Felix,
>> 
>> your fix looks reasonable to me but Vladimir K. (who reviewed your original fix), should also have a
>> look.
>> 
>> Best regards,
>> Tobias