[clang] [CIR] Add framework for CIR to LLVM IR lowering (PR #124650)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 27 15:05:19 PST 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 3861b9db882d5637725ceeccb801c2bb837e8fc5 41546e2d096310d8a1474539b5036152c7df3d11 --extensions h,cpp,c -- clang/include/clang/CIR/LowerToLLVM.h clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp clang/test/CIR/Lowering/hello.c clang/include/clang/CIR/CIRGenerator.h clang/include/clang/CIR/FrontendAction/CIRGenAction.h clang/lib/CIR/FrontendAction/CIRGenAction.cpp clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/include/clang/CIR/LowerToLLVM.h b/clang/include/clang/CIR/LowerToLLVM.h
index 82a28270cc..ee9f70495e 100644
--- a/clang/include/clang/CIR/LowerToLLVM.h
+++ b/clang/include/clang/CIR/LowerToLLVM.h
@@ -28,8 +28,8 @@ class ModuleOp;
namespace cir {
namespace direct {
-std::unique_ptr<llvm::Module> lowerDirectlyFromCIRToLLVMIR(
- mlir::ModuleOp M, llvm::LLVMContext &Ctx);
+std::unique_ptr<llvm::Module>
+lowerDirectlyFromCIRToLLVMIR(mlir::ModuleOp M, llvm::LLVMContext &Ctx);
} // namespace direct
} // namespace cir
diff --git a/clang/lib/CIR/FrontendAction/CIRGenAction.cpp b/clang/lib/CIR/FrontendAction/CIRGenAction.cpp
index 583fd8126e..9101f3cc38 100644
--- a/clang/lib/CIR/FrontendAction/CIRGenAction.cpp
+++ b/clang/lib/CIR/FrontendAction/CIRGenAction.cpp
@@ -7,13 +7,13 @@
//===----------------------------------------------------------------------===//
#include "clang/CIR/FrontendAction/CIRGenAction.h"
+#include "mlir/IR/MLIRContext.h"
+#include "mlir/IR/OwningOpRef.h"
#include "clang/CIR/CIRGenerator.h"
#include "clang/CIR/LowerToLLVM.h"
#include "clang/CodeGen/BackendUtil.h"
#include "clang/Frontend/CompilerInstance.h"
#include "llvm/IR/Module.h"
-#include "mlir/IR/MLIRContext.h"
-#include "mlir/IR/OwningOpRef.h"
using namespace cir;
using namespace clang;
@@ -51,8 +51,7 @@ class CIRGenConsumer : public clang::ASTConsumer {
std::unique_ptr<CIRGenerator> Gen;
public:
- CIRGenConsumer(CIRGenAction::OutputType Action,
- CompilerInstance &CI,
+ CIRGenConsumer(CIRGenAction::OutputType Action, CompilerInstance &CI,
std::unique_ptr<raw_pwrite_stream> OS)
: Action(Action), CI(CI), OutputStream(std::move(OS)),
FS(&CI.getVirtualFileSystem()),
@@ -88,10 +87,9 @@ public:
std::move(MLIRCtx), LLVMCtx);
BackendAction BEAction = getBackendActionFromOutputType(Action);
- emitBackendOutput(CI, CI.getCodeGenOpts(),
- C.getTargetInfo().getDataLayoutString(),
- LLVMModule.get(), BEAction, FS,
- std::move(OutputStream));
+ emitBackendOutput(
+ CI, CI.getCodeGenOpts(), C.getTargetInfo().getDataLayoutString(),
+ LLVMModule.get(), BEAction, FS, std::move(OutputStream));
break;
}
}
@@ -125,8 +123,8 @@ CIRGenAction::CreateASTConsumer(CompilerInstance &CI, StringRef InFile) {
if (!Out)
Out = getOutputStream(CI, InFile, Action);
- auto Result = std::make_unique<cir::CIRGenConsumer>(
- Action, CI, std::move(Out));
+ auto Result =
+ std::make_unique<cir::CIRGenConsumer>(Action, CI, std::move(Out));
return Result;
}
diff --git a/clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp b/clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp
index 4bf0f3ab78..b69e063170 100644
--- a/clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp
+++ b/clang/lib/CIR/Lowering/DirectToLLVM/LowerToLLVM.cpp
@@ -18,7 +18,6 @@
#include "llvm/IR/Module.h"
#include "llvm/Support/TimeProfiler.h"
-
using namespace cir;
using namespace llvm;
@@ -30,7 +29,8 @@ lowerDirectlyFromCIRToLLVMIR(mlir::ModuleOp theModule, LLVMContext &llvmCtx) {
llvm::TimeTraceScope scope("lower from CIR to LLVM directly");
auto ModuleName = theModule.getName();
- auto llvmModule = std::make_unique<llvm::Module>(ModuleName ? *ModuleName : "CIRToLLVMModule", llvmCtx);
+ auto llvmModule = std::make_unique<llvm::Module>(
+ ModuleName ? *ModuleName : "CIRToLLVMModule", llvmCtx);
if (!llvmModule)
report_fatal_error("Lowering from LLVMIR dialect to llvm IR failed!");
``````````
</details>
https://github.com/llvm/llvm-project/pull/124650
More information about the cfe-commits
mailing list