[PATCH] D80007: [AMDGPU] Fix assertion failure in SIInsertHardClauses

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 15 08:39:36 PDT 2020


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


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertHardClauses.cpp:115-116
+    // end.
+    unsigned Size =
+        std::distance(CI.First->getIterator(), CI.Last->getIterator()) + 1;
+    if (Size < 2)
----------------
arsenm wrote:
> Seems like we should be able to avoid rescanning through all the instructions though
True. I may try that as a follow-up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80007





More information about the llvm-commits mailing list