[PATCH] D76134: [Hexagon] Check isInstr() before getInstr() with SUnit.

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 14 06:59:03 PDT 2020


This revision was automatically updated to reflect the committed changes.
Closed by commit rG5f3f45dc5338: [Hexagon] Check isInstr() before getInstr() with SUnit (authored by Xinglong, committed by kparzysz).

Changed prior to commit:
  https://reviews.llvm.org/D76134?vs=250197&id=263994#toc

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76134

Files:
  llvm/lib/Target/Hexagon/HexagonSubtarget.cpp


Index: llvm/lib/Target/Hexagon/HexagonSubtarget.cpp
===================================================================
--- llvm/lib/Target/Hexagon/HexagonSubtarget.cpp
+++ llvm/lib/Target/Hexagon/HexagonSubtarget.cpp
@@ -318,11 +318,11 @@
 void HexagonSubtarget::adjustSchedDependency(SUnit *Src, int SrcOpIdx,
                                              SUnit *Dst, int DstOpIdx,
                                              SDep &Dep) const {
-  MachineInstr *SrcInst = Src->getInstr();
-  MachineInstr *DstInst = Dst->getInstr();
   if (!Src->isInstr() || !Dst->isInstr())
     return;
 
+  MachineInstr *SrcInst = Src->getInstr();
+  MachineInstr *DstInst = Dst->getInstr();
   const HexagonInstrInfo *QII = getInstrInfo();
 
   // Instructions with .new operands have zero latency.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76134.263994.patch
Type: text/x-patch
Size: 791 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200514/a491b675/attachment.bin>


More information about the llvm-commits mailing list