diff options
author | Amara Emerson <aemerson@apple.com> | 2019-01-19 00:36:11 +0000 |
---|---|---|
committer | Amara Emerson <aemerson@apple.com> | 2019-01-19 00:36:11 +0000 |
commit | d5015edb37173444caccb03680656722fafd3643 (patch) | |
tree | 1b96966f101c59a31b9e1e883ecabc4f58311f0e /llvm/lib | |
parent | 38f9900aa5241dd8fcb70dce3543a294d0d3e870 (diff) | |
download | bcm5719-llvm-d5015edb37173444caccb03680656722fafd3643.tar.gz bcm5719-llvm-d5015edb37173444caccb03680656722fafd3643.zip |
Revert r351584: "GlobalISel: Verify g_zextload and g_sextload"
This new assertion triggered on the AArch64 GlobalISel bots. Reverting while it's being investigated.
llvm-svn: 351617
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/CodeGen/MachineVerifier.cpp | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/llvm/lib/CodeGen/MachineVerifier.cpp b/llvm/lib/CodeGen/MachineVerifier.cpp index a19049c7cd3..c11b6d4bb22 100644 --- a/llvm/lib/CodeGen/MachineVerifier.cpp +++ b/llvm/lib/CodeGen/MachineVerifier.cpp @@ -986,24 +986,11 @@ void MachineVerifier::visitMachineInstrBefore(const MachineInstr *MI) { break; case TargetOpcode::G_LOAD: case TargetOpcode::G_STORE: - case TargetOpcode::G_ZEXTLOAD: - case TargetOpcode::G_SEXTLOAD: // Generic loads and stores must have a single MachineMemOperand // describing that access. - if (!MI->hasOneMemOperand()) { + if (!MI->hasOneMemOperand()) report("Generic instruction accessing memory must have one mem operand", MI); - } else { - if (MI->getOpcode() == TargetOpcode::G_ZEXTLOAD || - MI->getOpcode() == TargetOpcode::G_SEXTLOAD) { - const MachineMemOperand &MMO = **MI->memoperands_begin(); - LLT DstTy = MRI->getType(MI->getOperand(0).getReg()); - if (MMO.getSize() * 8 >= DstTy.getSizeInBits()) { - report("Generic extload must have a narrower memory type", MI); - } - } - } - break; case TargetOpcode::G_PHI: { LLT DstTy = MRI->getType(MI->getOperand(0).getReg()); |