[PATCH] D64709: [AMDGPU] fixed scheduler crash in gfx908

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec marked an inline comment as done.
rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUSubtarget.cpp:769
+    const MachineInstr *MI = SU->getInstr();
+    return MI && TII->isSALU(*MI) && !MI->isTerminator();
   }
----------------
kerbowa wrote:
> rampitec wrote:
> > kerbowa wrote:
> > > There is SUnit::isInstr() also
> > It's actually very strange to see anything but MI in the PostRA scheduler.
> Yeah I just meant as a null check.
Same thing, but a little bit more source ;)


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

https://reviews.llvm.org/D64709