[PATCH] D64189: Allow llc to run passes under the new pass manager one at a time.

Charles Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 10:10:49 PDT 2019


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


================
Comment at: llvm/tools/llc/llc.cpp:62
+#include <llvm/CodeGen/MachineCopyPropagation.h>
+#include <llvm/CodeGen/MachineDominators.h>
 using namespace llvm;
----------------
probinson wrote:
> Is there some reason these includes aren't in order with the others above?
Yes, these are only needed for the included new pass manager registry. This diff is only meant to hack up llc to actually use the new pass manager, so I thought it would be better to keep these includes visually separate for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64189





More information about the llvm-commits mailing list