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

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

Can you provide an explanation at a high level for why you're going with this approach rather than having the AC for ephemeral value finding and operand bundles for affected values? Essentially, why not just use both previous approaches rather than adding the affected values to the AC? (I'm not questioning what you've done here, just trying to understand the thought process behind it.)


https://reviews.llvm.org/D28459