[PATCH] D100875: [SimplifyLibCalls] Don't change alignment when creating memset

Stefan Stipanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 20 14:33:12 PDT 2021


sstefan1 added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp:578
   if (SrcLen == 0) {
     // strncpy(x, "", y) -> memset(align 1 x, '\0', y)
+    Align MemSetAlign =
----------------
lebedev.ri wrote:
> The comment is now wrong
Yeah, forgot to update. I guess alignment shouldn't be mentioned anymore.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100875



More information about the llvm-commits mailing list