[PATCH] D127391: [AMDGPU] Update SIInsertHardClauses for GFX11

[Ivan Kosarev via Phabricator via llvm-commits]

kosarev added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertHardClauses.cpp:182
+    // documentation (at least for GFX11) says that 63 is the maximum allowed.
+    assert(CI.Length <= 63 && "Hard clause is too long!");
 
----------------
We seem to use the value twice. What if to give it a name and move the comment to where we define it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127391