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

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 05:54:21 PST 2016


Prazek added a comment.

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?



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


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


https://reviews.llvm.org/D27259





More information about the llvm-commits mailing list