[PATCH] D46648: [SimplifyLibcalls] Optimize string concats using s(n)printf

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 9 15:22:14 PDT 2018


xbolva00 added a comment.

I am just wondering...

snprintf(buf , n, "some data %s", buf) is UB..
but..
snprintf(buf, n , "%s", buf) should be totally fine. Also for %s%s. If fmt starts with "%s", there should not be a problem.

Eli?


Repository:
  rL LLVM

https://reviews.llvm.org/D46648





More information about the llvm-commits mailing list