Change Details

If a MachineInstr is incomplete (e.g. an operand should be Reg but is currently FrameIndex), isCompressibleInst() will terminate at MachineOperandType::getReg(). With this patch, isCompressibleInst() will now return false if an operand's type doesn't match, allowing TargetInstrInfo::getInstSizeInBytes() to return a value and EstimateFunctionSizeInBytes() to work correctly.

If a MachineInstr's operand should be Reg in compiler's output but is currently FrameIndex, `isCompressibleInst()` will terminate at `MachineOperandType::getReg()`. This patch adds `.isReg()` checks to make `isCompressibleInst()` return false for these MachineInstr, allowing `getInstSizeInBytes()` to return a value and `EstimateFunctionSizeInBytes()` to work as intended. See https://reviews.llvm.org/D129999#3694222 for details.

If a MachineInstr is incomplete (e.g. an's operand should be Reg in compiler's output but is currently FrameIndex), `isCompressibleInst()` will terminate at `MachineOperandType::getReg()`. With this patchThis patch adds `.isReg()` checks to make `isCompressibleInst()` return false for these MachineInstr, isCompressibleInst() will now return false if an operand's type doesn't match, allowing TargetInstrInfo::`getInstSizeInBytes()` to return a value and `EstimateFunctionSizeInBytes()` to work correctly. as intended. See https://reviews.llvm.org/D129999#3694222 for details.