[PATCH] D66967: [Attributor] ValueSimplify Abstract Attribute

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:2514
+        // Current value is regarded as Simplified Value.
+        indicateOptimisticFixpoint();
+        return false;
----------------
I am unsure. There are more values in the worklist of the genericValueTraversal which we need to visit and which could change things, right?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66967/new/

https://reviews.llvm.org/D66967