In escape.cpp, the function reduce_phi has the following fragment:
```
...
} else if (use->is_SafePoint()) {
// processed later
} else {
assert(use->is_SafePoint(), "Unexpected user of reducible Phi %d -> %d:%s:%d", ophi->_idx, use->_idx, use->Name(), use->outcnt());
}
```
This is unnecessary as the assert will always trigger. We can either remove the else if branch or change the assert for a ShouldNotReachHere
```
...
} else if (use->is_SafePoint()) {
// processed later
} else {
assert(use->is_SafePoint(), "Unexpected user of reducible Phi %d -> %d:%s:%d", ophi->_idx, use->_idx, use->Name(), use->outcnt());
}
```
This is unnecessary as the assert will always trigger. We can either remove the else if branch or change the assert for a ShouldNotReachHere
- caused by
-
JDK-8316991 Reduce nullable allocation merges
-
- Resolved
-