[PATCH] D28459: Make processing @llvm.assume more efficient - Add affected values to the assumption cache

[Hal Finkel via Phabricator via llvm-commits]

hfinkel marked 3 inline comments as done.
hfinkel added inline comments.


================
Comment at: include/llvm/Analysis/AssumptionCache.h:57
+
+    AffectedValueCallbackVH(Value *V, AssumptionCache *AC = nullptr)
+        : CallbackVH(V), AC(AC) {}
----------------
Prazek wrote:
> explicit?
No, DenseMap won't work if this is explicit (because we're using Value*'s KeyInfo).


https://reviews.llvm.org/D28459