aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib
diff options
context:
space:
mode:
authorMatt Arsenault <Matthew.Arsenault@amd.com>2019-01-18 20:17:37 +0000
committerMatt Arsenault <Matthew.Arsenault@amd.com>2019-01-18 20:17:37 +0000
commitf67ae61131270cd87f7e5e35f1dc8c74871272b8 (patch)
tree2977fbc85f67cc4e68bda65c0bd0eadd2c591903 /llvm/lib
parent08d3d32ead15373123de7c2c0a06e9f9e3f96e35 (diff)
downloadllvm-f67ae61131270cd87f7e5e35f1dc8c74871272b8.zip
llvm-f67ae61131270cd87f7e5e35f1dc8c74871272b8.tar.gz
llvm-f67ae61131270cd87f7e5e35f1dc8c74871272b8.tar.bz2
GlobalISel: Verify g_zextload and g_sextload
llvm-svn: 351584
Diffstat (limited to 'llvm/lib')
-rw-r--r--llvm/lib/CodeGen/MachineVerifier.cpp15
1 files changed, 14 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/MachineVerifier.cpp b/llvm/lib/CodeGen/MachineVerifier.cpp
index 534d369..aae0958 100644
--- a/llvm/lib/CodeGen/MachineVerifier.cpp
+++ b/llvm/lib/CodeGen/MachineVerifier.cpp
@@ -986,11 +986,24 @@ 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());