[PATCH] D93974: [ValueTracking] Safe assumption context for args

Gil Rapaport via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 13 14:30:32 PST 2021


gilr added a comment.

In D93974#2494331 <https://reviews.llvm.org/D93974#2494331>, @jdoerfert wrote:

> Oh.. this is bad. That happens if you mix two logically differnet things into a single instruction pointer.
> I put it on the list of things that need to fixed wrt. assumes ... :(

Excellent. Is the list published anywhere?

> Thanks for the patience. I'm fine with the nullptr solution for now. I need to think more about this "workaround" before I'd suggest an alternative.

Sure, thanks for taking the time!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93974



More information about the llvm-commits mailing list