[PATCH] D53342: [SimplifyLibCalls] Mark known arguments with nonnull

Fri Sep 6 11:03:24 PDT 2019

xbolva00 added a comment.
Herald added a subscriber: ychen.

In D53342#1654084 <https://reviews.llvm.org/D53342#1654084>, @xbolva00 wrote:

> We should ditch “removeNonNull” handling since Clang never annotates callsites (Clang only annotates function signatures - currently nonnull is dropped, great!)

What do you think, @jdoerfert ? removeNonNull sometimes caused instcombine’s infinite loop + i think we dont need removeNonNull at all.

I would like to move forward here..

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

https://reviews.llvm.org/D53342