[llvm] [WIP][Assume] Generic assume-based uniformity optimization (PR #160670)
Sameer Sahasrabuddhe via llvm-commits
llvm-commits at lists.llvm.org
Sat Oct 4 21:10:14 PDT 2025
================
@@ -3540,6 +3540,79 @@ Instruction *InstCombinerImpl::visitCallInst(CallInst &CI) {
}
}
+ // Basic assume equality optimization: assume(x == c) -> replace dominated uses of x with c
+ if (auto *ICmp = dyn_cast<ICmpInst>(IIOperand)) {
+ if (ICmp->getPredicate() == ICmpInst::ICMP_EQ) {
+ Value *LHS = ICmp->getOperand(0);
+ Value *RHS = ICmp->getOperand(1);
+ Value *Variable = nullptr;
+ Constant *ConstantVal = nullptr;
+
+ if (auto *C = dyn_cast<Constant>(RHS)) {
+ Variable = LHS;
+ ConstantVal = C;
+ } else if (auto *C = dyn_cast<Constant>(LHS)) {
+ Variable = RHS;
+ ConstantVal = C;
+ }
+
+ if (Variable && ConstantVal && Variable->hasUseList()) {
+ SmallVector<Use *, 8> DominatedUses;
+ for (Use &U : Variable->uses()) {
+ if (auto *UseInst = dyn_cast<Instruction>(U.getUser())) {
+ if (UseInst != II && UseInst != ICmp &&
+ isValidAssumeForContext(II, UseInst, &DT)) {
+ DominatedUses.push_back(&U);
+ }
+ }
+ }
+
+ for (Use *U : DominatedUses) {
+ U->set(ConstantVal);
+ Worklist.pushValue(U->getUser());
----------------
ssahasra wrote:
I don't know how the whole traversal works, but do we need to push `Variable` as well as its uses? Will that result in duplicate work?
https://github.com/llvm/llvm-project/pull/160670
More information about the llvm-commits
mailing list