[llvm] [AMDGPU][NewPM] Port TailDuplicate pass to NPM (PR #113293)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 22 03:21:08 PDT 2024
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 75ec65e384d73d97bab2f1a4e273df4b622af4c4 d5d50d63e2c43166807a6c747a52789f81e866a9 --extensions h,cpp -- llvm/include/llvm/CodeGen/TailDuplication.h llvm/include/llvm/CodeGen/Passes.h llvm/include/llvm/InitializePasses.h llvm/include/llvm/Passes/CodeGenPassBuilder.h llvm/lib/CodeGen/CodeGen.cpp llvm/lib/CodeGen/TailDuplication.cpp llvm/lib/CodeGen/TargetPassConfig.cpp llvm/lib/Passes/PassBuilder.cpp llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/include/llvm/CodeGen/TailDuplication.h b/llvm/include/llvm/CodeGen/TailDuplication.h
index 983c9be406..687a592ccf 100644
--- a/llvm/include/llvm/CodeGen/TailDuplication.h
+++ b/llvm/include/llvm/CodeGen/TailDuplication.h
@@ -18,25 +18,30 @@ namespace llvm {
template <typename DerivedT, bool PreRegAlloc>
class TailDuplicatePassBase : public PassInfoMixin<DerivedT> {
private:
- std::unique_ptr<MBFIWrapper> MBFIW;
+ std::unique_ptr<MBFIWrapper> MBFIW;
+
public:
PreservedAnalyses run(MachineFunction &MF,
MachineFunctionAnalysisManager &MFAM);
};
-class EarlyTailDuplicatePass : public TailDuplicatePassBase<EarlyTailDuplicatePass, true> {
+class EarlyTailDuplicatePass
+ : public TailDuplicatePassBase<EarlyTailDuplicatePass, true> {
public:
MachineFunctionProperties getClearedProperties() const {
- return MachineFunctionProperties().set(MachineFunctionProperties::Property::NoPHIs);
+ return MachineFunctionProperties().set(
+ MachineFunctionProperties::Property::NoPHIs);
}
};
-class TailDuplicatePass : public TailDuplicatePassBase<TailDuplicatePass, false> {};
+class TailDuplicatePass
+ : public TailDuplicatePassBase<TailDuplicatePass, false> {};
} // namespace llvm
-extern template class llvm::TailDuplicatePassBase<llvm::EarlyTailDuplicatePass, true>;
-extern template class llvm::TailDuplicatePassBase<llvm::TailDuplicatePass, false>;
+extern template class llvm::TailDuplicatePassBase<llvm::EarlyTailDuplicatePass,
+ true>;
+extern template class llvm::TailDuplicatePassBase<llvm::TailDuplicatePass,
+ false>;
#endif // LLVM_CODEGEN_TAILDUPLICATIONPASS_H
-
diff --git a/llvm/lib/CodeGen/TailDuplication.cpp b/llvm/lib/CodeGen/TailDuplication.cpp
index 56a4dff278..cf17dd3ca8 100644
--- a/llvm/lib/CodeGen/TailDuplication.cpp
+++ b/llvm/lib/CodeGen/TailDuplication.cpp
@@ -79,7 +79,8 @@ char EarlyTailDuplicateLegacy::ID;
char &llvm::TailDuplicateLegacyID = TailDuplicateLegacy::ID;
char &llvm::EarlyTailDuplicateLegacyID = EarlyTailDuplicateLegacy::ID;
-INITIALIZE_PASS(TailDuplicateLegacy, DEBUG_TYPE, "Tail Duplication", false, false)
+INITIALIZE_PASS(TailDuplicateLegacy, DEBUG_TYPE, "Tail Duplication", false,
+ false)
INITIALIZE_PASS(EarlyTailDuplicateLegacy, "early-tailduplication",
"Early Tail Duplication", false, false)
@@ -105,16 +106,17 @@ bool TailDuplicateBase::runOnMachineFunction(MachineFunction &MF) {
}
template <typename DerivedT, bool PreRegAlloc>
-PreservedAnalyses TailDuplicatePassBase<DerivedT, PreRegAlloc>::run(MachineFunction &MF,
- MachineFunctionAnalysisManager &MFAM) {
- MFPropsModifier<DerivedT> _(static_cast<DerivedT&>(*this), MF);
+PreservedAnalyses TailDuplicatePassBase<DerivedT, PreRegAlloc>::run(
+ MachineFunction &MF, MachineFunctionAnalysisManager &MFAM) {
+ MFPropsModifier<DerivedT> _(static_cast<DerivedT &>(*this), MF);
auto *MBPI = &MFAM.getResult<MachineBranchProbabilityAnalysis>(MF);
auto *PSI = MFAM.getResult<ModuleAnalysisManagerMachineFunctionProxy>(MF)
- .getCachedResult<ProfileSummaryAnalysis>(*MF.getFunction().getParent());
- auto *MBFI = (PSI && PSI->hasProfileSummary() ?
- &MFAM.getResult<MachineBlockFrequencyAnalysis>(MF):
- nullptr);
+ .getCachedResult<ProfileSummaryAnalysis>(
+ *MF.getFunction().getParent());
+ auto *MBFI = (PSI && PSI->hasProfileSummary()
+ ? &MFAM.getResult<MachineBlockFrequencyAnalysis>(MF)
+ : nullptr);
if (MBFI)
MBFIW = std::make_unique<MBFIWrapper>(*MBFI);
``````````
</details>
https://github.com/llvm/llvm-project/pull/113293
More information about the llvm-commits
mailing list