diff options
| author | Jon Roelofs <jonathan_roelofs@apple.com> | 2021-07-21 08:23:17 -0700 |
|---|---|---|
| committer | Jon Roelofs <jonathan_roelofs@apple.com> | 2021-07-21 08:47:17 -0700 |
| commit | 4de74a7c4da3516d0d9119452cc073b061716b5d (patch) | |
| tree | 81e6835876994a2b75abbca77f443ea5d5da43f2 /llvm/lib/CodeGen/MachineVerifier.cpp | |
| parent | 5fba6058965c6b2cd2fe17335bb1fb78d99815bd (diff) | |
| download | llvm-4de74a7c4da3516d0d9119452cc073b061716b5d.zip llvm-4de74a7c4da3516d0d9119452cc073b061716b5d.tar.gz llvm-4de74a7c4da3516d0d9119452cc073b061716b5d.tar.bz2 | |
[MachineVerifier] Make INSERT_SUBREG diagnostic respect operand 2 subregs
This came out of post-commit review: https://reviews.llvm.org/D105953#inline-1012919
Thanks uabelho!
Diffstat (limited to 'llvm/lib/CodeGen/MachineVerifier.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/MachineVerifier.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/llvm/lib/CodeGen/MachineVerifier.cpp b/llvm/lib/CodeGen/MachineVerifier.cpp index dde0a24..7e3198a 100644 --- a/llvm/lib/CodeGen/MachineVerifier.cpp +++ b/llvm/lib/CodeGen/MachineVerifier.cpp @@ -1783,8 +1783,11 @@ void MachineVerifier::visitMachineInstrBefore(const MachineInstr *MI) { // TODO: verify we have properly encoded deopt arguments } break; case TargetOpcode::INSERT_SUBREG: { - unsigned InsertedSize = - TRI->getRegSizeInBits(MI->getOperand(2).getReg(), *MRI); + unsigned InsertedSize; + if (unsigned SubIdx = MI->getOperand(2).getSubReg()) + InsertedSize = TRI->getSubRegIdxSize(SubIdx); + else + InsertedSize = TRI->getRegSizeInBits(MI->getOperand(2).getReg(), *MRI); unsigned SubRegSize = TRI->getSubRegIdxSize(MI->getOperand(3).getImm()); if (SubRegSize < InsertedSize) { report("INSERT_SUBREG expected inserted value to have equal or lesser " |