[PATCH] D154205: [MachineLICM] Handle subloops

JinGu Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 12:25:17 PDT 2023


jaykang10 added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineLICM.cpp:134
 
-    // Exit blocks for CurLoop.
-    SmallVector<MachineBasicBlock *, 8> ExitBlocks;
+    enum { NotHoisted = 0, Hoisted = 1, CSEd = 2 };
 
----------------
nikic wrote:
> Any reason not to name this enum and then use it instead of `unsigned`?
Ah sorry.
Let me update the code with named enum tomorrow.


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

https://reviews.llvm.org/D154205



More information about the llvm-commits mailing list