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 origin/main HEAD --extensions cpp,h --
clang/lib/CodeGen/Targets/AMDGPU.cpp llvm/lib/Target/AMDGPU/AMDGPU.h
llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp
llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/CodeGen/Targets/AMDGPU.cpp
b/clang/lib/CodeGen/Targets/AMDGPU.cpp
index 588a7d4ab..6bdf1bc89 100644
--- a/clang/lib/CodeGen/Targets/AMDGPU.cpp
+++ b/clang/lib/CodeGen/Targets/AMDGPU.cpp
@@ -784,7 +784,7 @@ void AMDGPUTargetCodeGenInfo::emitTargetGlobals(
if (!Sin || !Cos || M.getFunction(Name))
continue;
llvm::Type *FPTy = IsF32 ? llvm::Type::getFloatTy(M.getContext())
- : llvm::Type::getDoubleTy(M.getContext());
+ : llvm::Type::getDoubleTy(M.getContext());
llvm::Type *PtrTy = llvm::PointerType::get(M.getContext(), 5);
ToAdd.push_back(llvm::Function::Create(
llvm::FunctionType::get(FPTy, {FPTy, PtrTy}, false),
diff --git a/llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp
b/llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp
index 7e490bda2..b39e2db5f 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPULibCalls.cpp
@@ -1731,8 +1731,8 @@ PreservedAnalyses AMDGPUUseNativeCallsPass::run(Function
&F,
return Changed ? PreservedAnalyses::none() : PreservedAnalyses::all();
}
-PreservedAnalyses AMDGPUUnusedLibFuncCleanupPass::run(Module &M,
- ModuleAnalysisManager
&AM) {
+PreservedAnalyses
+AMDGPUUnusedLibFuncCleanupPass::run(Module &M, ModuleAnalysisManager &AM) {
// Remove device-library functions from @llvm.compiler.used and erase them
// if they have no callers. These may have been eagerly pulled in before
// device-library linking to enable later optimisation passes (e.g. sin/cos
@@ -1743,11 +1743,11 @@ PreservedAnalyses
AMDGPUUnusedLibFuncCleanupPass::run(Module &M,
Function *F = M.getFunction(Name);
if (!F)
continue;
- if (any_of(F->uses(), [](const Use &U) { return
isa<CallBase>(U.getUser()); }))
+ if (any_of(F->uses(),
+ [](const Use &U) { return isa<CallBase>(U.getUser()); }))
continue;
- removeFromUsedLists(M, [F](Constant *C) {
- return C->stripPointerCasts() == F;
- });
+ removeFromUsedLists(
+ M, [F](Constant *C) { return C->stripPointerCasts() == F; });
F->eraseFromParent();
Changed = true;
}
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
b/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
index ae8a5ed2a..536d5a5bb 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
@@ -979,13 +979,13 @@ void
AMDGPUTargetMachine::registerPassBuilderCallbacks(PassBuilder &PB) {
FPM.addPass(AMDGPUUniformIntrinsicCombinePass());
});
- PB.registerOptimizerLastEPCallback(
- [](ModulePassManager &MPM, OptimizationLevel Level,
- ThinOrFullLTOPhase Phase) {
- if (Level == OptimizationLevel::O0)
- return;
- MPM.addPass(AMDGPUUnusedLibFuncCleanupPass());
- });
+ PB.registerOptimizerLastEPCallback([](ModulePassManager &MPM,
+ OptimizationLevel Level,
+ ThinOrFullLTOPhase Phase) {
+ if (Level == OptimizationLevel::O0)
+ return;
+ MPM.addPass(AMDGPUUnusedLibFuncCleanupPass());
+ });
PB.registerCGSCCOptimizerLateEPCallback(
[this](CGSCCPassManager &PM, OptimizationLevel Level) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/181774
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
