truncateToMinimalBitWidths() doesn't handle all Instructions and the worst case is compiler crash via llvm_unreachable(). Fix is to add a case to handle PHINode and changed the worst case to NO-OP (from compiler crash).
Seems reasonable to me.
Ideally we should be handling the phi node case (which is why the unreachable was there), but crashing doesn't seem like the right thing to do.
|3279 ↗||(On Diff #155996)|
Why do we need to have this if statement if we skip the transformation for unhanded instructions?