[llvm] [DropUnnecessaryAssumes] Make the ephemeral value check more precise (PR #160700)
Andreas Jonson via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 25 14:53:00 PDT 2025
================
@@ -17,13 +18,44 @@ using namespace llvm;
using namespace llvm::PatternMatch;
static bool affectedValuesAreEphemeral(ArrayRef<Value *> Affected) {
- // If all the affected uses have only one use (part of the assume), then
- // the assume does not provide useful information. Note that additional
- // users may appear as a result of inlining and CSE, so we should only
- // make this assumption late in the optimization pipeline.
- // TODO: Handle dead cyclic usages.
- // TODO: Handle multiple dead assumes on the same value.
- return all_of(Affected, match_fn(m_OneUse(m_Value())));
+ // Check whether all the uses are ephemeral, i.e. recursively only used
+ // by assumes. In that case, the assume does not provide useful information.
+ // Note that additional users may appear as a result of inlining and CSE,
+ // so we should only make this assumption late in the optimization pipeline.
+ SmallSetVector<User *, 16> Worklist;
----------------
andjo403 wrote:
```suggestion
SmallSetVector<User *, 32> Worklist;
```
Change the size to match max number of elements maybe?
https://github.com/llvm/llvm-project/pull/160700
More information about the llvm-commits
mailing list