[llvm] [InstCombine] Use llvm::set_is_subset (NFC) (PR #102778)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Aug 10 17:21:00 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/102778.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp (+2-4)
``````````diff
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp b/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
index 1661fa564c65c7..0b51845ab5e257 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
@@ -12,6 +12,7 @@
#include "InstCombineInternal.h"
#include "llvm/ADT/MapVector.h"
+#include "llvm/ADT/SetOperations.h"
#include "llvm/ADT/SmallString.h"
#include "llvm/ADT/Statistic.h"
#include "llvm/Analysis/AliasAnalysis.h"
@@ -287,10 +288,7 @@ bool PointerReplacer::collectUsers() {
// Ensure that all outstanding (indirect) users of I
// are inserted into the Worklist. Return false
// otherwise.
- for (auto *Inst : ValuesToRevisit)
- if (!Worklist.contains(Inst))
- return false;
- return true;
+ return llvm::set_is_subset(ValuesToRevisit, Worklist);
}
bool PointerReplacer::collectUsersRecursive(Instruction &I) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/102778
More information about the llvm-commits
mailing list