awarzynski added inline comments.
================ Comment at: flang/lib/FrontendTool/ExecuteCompilerInvocation.cpp:15 +#include "mlir/IR/MLIRContext.h" +#include "mlir/Pass/PassManager.h" #include "flang/Frontend/CompilerInstance.h" ---------------- rovka wrote: > Nit: Should these come after the llvm/ headers? (So it's alphabetical except > for flang, which may go first) Thanks, I will fix this before merging! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123297/new/ https://reviews.llvm.org/D123297 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
