[PATCH] D63604: [Attributor] Deduce "nonnull" attribute

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 10 10:39:22 PDT 2019


jdoerfert added a comment.

In D63604#1578854 <https://reviews.llvm.org/D63604#1578854>, @xbolva00 wrote:

> Maybe you remember https://reviews.llvm.org/D53342 ? Should we do it “InstCombine”-way or some alternative one ?


I did not, thanks for reminding me. What do you mean by the "InstCombine"-way and what alternatives do you have in mind?
Sorry that I'm a bit lost at the moment, I looked at D53342 <https://reviews.llvm.org/D53342> and I seemed there are still problems with that patch even though the idea is very useful and we should add it.


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

https://reviews.llvm.org/D63604





More information about the llvm-commits mailing list