Use any_of instead of a loop

john-brawn-arm · john-brawn-arm · commit 7017311dfed6 · 2025-03-28T11:47:55.000Z
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -4514,11 +4514,9 @@ static bool willGenerateVectors(VPlan &Plan, ElementCount VF,
 }
 
 static bool hasReplicatorRegion(VPlan &Plan) {
-  for (auto *VPRB : VPBlockUtils::blocksOnly<VPRegionBlock>(
-           vp_depth_first_deep(Plan.getEntry())))
-    if (VPRB->isReplicator())
-      return true;
-  return false;
+  return any_of(VPBlockUtils::blocksOnly<VPRegionBlock>(
+                    vp_depth_first_deep(Plan.getEntry())),
+                [](auto *VPRB) { return VPRB->isReplicator(); });
 }
 
 #ifndef NDEBUG

Original file line number	Diff line number	Diff line change
`@@ -4514,11 +4514,9 @@ static bool willGenerateVectors(VPlan &Plan, ElementCount VF,`
`4514`	`4514`	`}`
`4515`	`4515`
`4516`	`4516`	`static bool hasReplicatorRegion(VPlan &Plan) {`
`4517`		`- for (auto *VPRB : VPBlockUtils::blocksOnly<VPRegionBlock>(`
`4518`		`- vp_depth_first_deep(Plan.getEntry())))`
`4519`		`- if (VPRB->isReplicator())`
`4520`		`- return true;`
`4521`		`- return false;`
	`4517`	`+ return any_of(VPBlockUtils::blocksOnly<VPRegionBlock>(`
	`4518`	`+ vp_depth_first_deep(Plan.getEntry())),`
	`4519`	`+ [](auto *VPRB) { return VPRB->isReplicator(); });`
`4522`	`4520`	`}`
`4523`	`4521`
`4524`	`4522`	`#ifndef NDEBUG`