[PATCH] D121268: [AMDGPU] Control flow pseudos are not part of block prologue

Ruiling, Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 8 19:37:24 PST 2022


ruiling added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:7604
+    return false;
+  switch (MI.getOpcode()) {
+  case AMDGPU::COPY:
----------------
I think SI_IF_BREAK can be removed from the list as it does not modifies EXEC. SI_END_CF should be removed from the list, as SI_END_CF means the join point, we should treat it as block prologue. Other instructions should never be prologue instructions, so it is correct to list them here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121268



More information about the llvm-commits mailing list