[PATCH] D53342: [SimplifyLibCalls] Mark known arguments with nonnull
Dávid Bolvanský via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 22 05:02:50 PDT 2019
xbolva00 added a comment.
In D53342#1639205 <https://reviews.llvm.org/D53342#1639205>, @xbolva00 wrote:
> And nobody wrote a paper to fix “the bogus”.
>
> @jdoerfert what do you think?
>
> As I see, annotating callsites is atm pointless since FunctionAttrs/Attributor cannot propagate it - they do only if “function signature (decl)” contains nonnull. I would continue to work on this current “safe” approach if there is a small promise that Attributor will handle this kind of propagation too :)
@uenoku informed me that Attributor will handle it with a WIP patch which is not yet commited. Good.
So this patch has a sense in terms of global scope. PTAL as I finished here all annotation work.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53342/new/
https://reviews.llvm.org/D53342
More information about the llvm-commits
mailing list