[PATCH] D46752: [Analysis] Validate the return type of s(n)printf libcalls

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 11 12:19:12 PDT 2018


efriedma added a comment.

I'm not convinced this really solves the problem correctly... I mean, obviously we shouldn't crash, but this doesn't deal with the underlying issue: once we transform the internal function, it is no longer the C library function, and that difference may not be reflected in the signature.  See http://lists.llvm.org/pipermail/llvm-dev/2017-October/118578.html .


Repository:
  rL LLVM

https://reviews.llvm.org/D46752





More information about the llvm-commits mailing list