[PATCH] D27259: Make processing @llvm.assume more efficient - operand bundles

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

In https://reviews.llvm.org/D27259#620931, @Prazek wrote:

> Does it solve my problem with assume vtable loads being processed slowly? If so, do I understand it correctly that it just keep track of what variables are effected by assume?


Fixing that, and related symptoms, is certainly my motivation.



================
Comment at: lib/Analysis/LazyValueInfo.cpp:966
+  for (auto *U : Val->users()) {
+    const IntrinsicInst *I = dyn_cast<IntrinsicInst>(U);
+    if (!I)
----------------
Prazek wrote:
> auto
> rename it to II?
Sure.


================
Comment at: lib/Analysis/ValueTracking.cpp:531
+  for (auto *U : V->users()) {
+    const IntrinsicInst *I = dyn_cast<IntrinsicInst>(U);
+    if (!I)
----------------
Prazek wrote:
> DITTO
Sure.


https://reviews.llvm.org/D27259