[PATCH] D116273: [AMDGPU] Iterate LoweredEndCf in the reverse order

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 31 03:43:34 PST 2021


foad added a comment.

What was the effect of inserting multiple branch instructions? Did it fail MIR verification?



================
Comment at: llvm/lib/Target/AMDGPU/SILowerControlFlow.cpp:585
 
-  for (MachineInstr *MI : LoweredEndCf) {
+  for (MachineInstr *MI : llvm::reverse(LoweredEndCf)) {
     MachineBasicBlock &MBB = *MI->getParent();
----------------
There's a "using namespace llvm" at the top of this file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116273



More information about the llvm-commits mailing list