[llvm] [AMDGPU] Use llvm::count_if (NFC) (PR #137492)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Apr 26 22:20:34 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/137492.diff
1 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp (+22-27)
``````````diff
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp b/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
index 4ff49c1e46dfe..a2b7cd088093a 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
@@ -1046,17 +1046,16 @@ class MFMAExpInterleaveOpt final : public IGLPStrategy {
if (!SyncPipe.size())
return false;
- auto SuccSize = std::count_if(
- SU->Succs.begin(), SU->Succs.end(),
- [](const SDep &Succ) { return Succ.getKind() == SDep::Data; });
+ auto SuccSize = llvm::count_if(SU->Succs, [](const SDep &Succ) {
+ return Succ.getKind() == SDep::Data;
+ });
if (SuccSize >= Size)
return false;
if (HasIntermediary) {
for (auto Succ : SU->Succs) {
- auto SuccSize = std::count_if(
- Succ.getSUnit()->Succs.begin(), Succ.getSUnit()->Succs.end(),
- [](const SDep &SuccSucc) {
+ auto SuccSize =
+ llvm::count_if(Succ.getSUnit()->Succs, [](const SDep &SuccSucc) {
return SuccSucc.getKind() == SDep::Data;
});
if (SuccSize >= Size)
@@ -1087,17 +1086,16 @@ class MFMAExpInterleaveOpt final : public IGLPStrategy {
if (!SyncPipe.size())
return false;
- auto SuccSize = std::count_if(
- SU->Succs.begin(), SU->Succs.end(),
- [](const SDep &Succ) { return Succ.getKind() == SDep::Data; });
+ auto SuccSize = llvm::count_if(SU->Succs, [](const SDep &Succ) {
+ return Succ.getKind() == SDep::Data;
+ });
if (SuccSize >= Size)
return true;
if (HasIntermediary) {
for (auto Succ : SU->Succs) {
- auto SuccSize = std::count_if(
- Succ.getSUnit()->Succs.begin(), Succ.getSUnit()->Succs.end(),
- [](const SDep &SuccSucc) {
+ auto SuccSize =
+ llvm::count_if(Succ.getSUnit()->Succs, [](const SDep &SuccSucc) {
return SuccSucc.getKind() == SDep::Data;
});
if (SuccSize >= Size)
@@ -1474,18 +1472,17 @@ bool MFMAExpInterleaveOpt::analyzeDAG(const SIInstrInfo *TII) {
MFMAChainLength = MFMAPipeCount / MFMAChains;
// The number of bit pack operations that depend on a single V_EXP
- unsigned PackSuccCount = std::count_if(
- PackSUs.begin(), PackSUs.end(), [this, &TempExp](SUnit *VPack) {
+ unsigned PackSuccCount =
+ llvm::count_if(PackSUs, [this, &TempExp](SUnit *VPack) {
return DAG->IsReachable(VPack, *TempExp);
});
// The number of bit pack operations an MFMA depends on
unsigned PackPredCount =
- std::count_if((*TempMFMA)->Preds.begin(), (*TempMFMA)->Preds.end(),
- [&isBitPack](SDep &Pred) {
- auto Opc = Pred.getSUnit()->getInstr()->getOpcode();
- return isBitPack(Opc);
- });
+ llvm::count_if((*TempMFMA)->Preds, [&isBitPack](SDep &Pred) {
+ auto Opc = Pred.getSUnit()->getInstr()->getOpcode();
+ return isBitPack(Opc);
+ });
auto *PackPred = llvm::find_if((*TempMFMA)->Preds, [&isBitPack](SDep &Pred) {
auto Opc = Pred.getSUnit()->getInstr()->getOpcode();
@@ -1499,20 +1496,18 @@ bool MFMAExpInterleaveOpt::analyzeDAG(const SIInstrInfo *TII) {
ExpRequirement = 0;
// How many MFMAs depend on a single bit pack operation
MFMAEnablement =
- std::count_if(PackPred->getSUnit()->Succs.begin(),
- PackPred->getSUnit()->Succs.end(), [&TII](SDep &Succ) {
- return TII->isMFMAorWMMA(*Succ.getSUnit()->getInstr());
- });
+ llvm::count_if(PackPred->getSUnit()->Succs, [&TII](SDep &Succ) {
+ return TII->isMFMAorWMMA(*Succ.getSUnit()->getInstr());
+ });
// The number of MFMAs that depend on a single V_EXP
MFMAEnablement *= PackSuccCount;
// The number of V_EXPs required to resolve all dependencies for an MFMA
ExpRequirement =
- std::count_if(ExpPipeCands.begin(), ExpPipeCands.end(),
- [this, &PackPred](SUnit *ExpBase) {
- return DAG->IsReachable(PackPred->getSUnit(), ExpBase);
- });
+ llvm::count_if(ExpPipeCands, [this, &PackPred](SUnit *ExpBase) {
+ return DAG->IsReachable(PackPred->getSUnit(), ExpBase);
+ });
ExpRequirement *= PackPredCount;
return true;
``````````
</details>
https://github.com/llvm/llvm-project/pull/137492
More information about the llvm-commits
mailing list