[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 06:42:11 PDT 2019
xbolva00 added a comment.
>> But we should have this after we filter out the annotations from memcpy/memset (etc.) in Clang (when the size is dynamic or zero)
I dont think we want add/remove attributes in Clang based on size argument. It seems just bad to look at function names in Clang and go thru Clang AST to inspect size argument.
Either drop all and possibly add it back here in instcombine or leave it and drop it in eg. BuildLibCalls?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D53342/new/
https://reviews.llvm.org/D53342
More information about the llvm-commits
mailing list