[llvm] [Analysis][ValueTracking] Unify most of the tracking between AssumptionCache and DomConditionCache (PR #83161)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 29 05:18:00 PST 2024


================
@@ -813,6 +813,8 @@ void llvm::computeKnownBitsFromContext(const Value *V, KnownBits &Known,
     if (!isValidAssumeForContext(I, Q.CxtI, Q.DT))
       continue;
 
+    // Change to `computeKnownBitsFromCond` if we start supporting assume of
+    // logic op in ConditionCacheUtil.
----------------
dtcxzyw wrote:

Conditions for `@llvm.assume` should be atomic expressions. TBH I don't think we can benefit from handling logical ops here.


https://github.com/llvm/llvm-project/pull/83161


More information about the llvm-commits mailing list