[PATCH] D57918: Add an attribute that causes clang to emit fortified calls to C stdlib functions

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 8 11:56:44 PST 2019


rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: clang/include/clang/Basic/AttrDocs.td:986
+  }
+  }];
+}
----------------
erik.pilkington wrote:
> rjmccall wrote:
> > Probably worth clarifying somewhere in here that only a specific set of stdlib functions are supported.  I don't know that it's important to spell that set out.
> I just copied a list in, there really aren't that many, so may as well just be explicit.
WFM


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

https://reviews.llvm.org/D57918





More information about the cfe-commits mailing list