[PATCH] D123348: [AMDGPU] Fix inline asm causing assert during PreRARematerialize stage in scheduler pass

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 18:24:12 PDT 2022


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNSchedStrategy.cpp:744
     MachineInstr *Def = MRI.getOneDef(Reg)->getParent();
-    if (!Def || Def->getOperand(0).getSubReg() != 0 ||
+    MachineOperand &Op = Def->getOperand(0);
+    if (!Def || !Op.isReg() || Op.getSubReg() != 0 ||
----------------
Null dereference before null check


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123348/new/

https://reviews.llvm.org/D123348



More information about the llvm-commits mailing list