[PATCH] D86396: [Attributor] Change AANoUndef not to deduce noundef for dead returned positions

Shinji Okumura via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 10:13:31 PDT 2020


okura added a comment.

When I fix the identification of AANoUndef in `identifyDefaultAbstractAttributes`, I noticed that noundef should not be deduced for dead argument and dead call site argument position too. (I found wrong transformation in e.g. `callbacks.ll`)
Not to deduce for dead positions, we should not indicate optimistic fixpoint in `initialize` because once fixpoint is indicated we cannot change the state. However, it weakens the deduction significantly.
So I want to suggest that we check not only noundef but also liveness in AAUndefinedBehavior.


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

https://reviews.llvm.org/D86396



More information about the llvm-commits mailing list