[PATCH] D136811: WIP: RFC: NFC: C++ Buffer Hardening user documentation.

2022-11-03 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added inline comments. Comment at: clang/docs/SafeBuffers.rst:31 +convert large amounts of old code to conform to the warning; + - Attribute ``[[unsafe_buffer_usage]]`` lets you annotate custom functions as +unsafe, while providing a safe alternative that c

[PATCH] D136811: WIP: RFC: NFC: C++ Buffer Hardening user documentation.

2022-11-02 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added inline comments. Comment at: clang/docs/SafeBuffers.rst:31 +convert large amounts of old code to conform to the warning; + - Attribute ``[[unsafe_buffer_usage]]`` lets you annotate custom functions as +unsafe, while providing a safe alternative that can o

[PATCH] D136811: WIP: RFC: NFC: C++ Buffer Hardening user documentation.

2022-11-02 Thread Jan Korous via Phabricator via cfe-commits
jkorous added a comment. Thank you for the feedback Aaron! We really appreciate the effort you put into this! Comment at: clang/docs/SafeBuffers.rst:69-70 +containers such as ``std::span``, you can achieve bounds safety by +*simply writing good modern C++ code*. This is what t

[PATCH] D136811: WIP: RFC: NFC: C++ Buffer Hardening user documentation.

2022-11-02 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Thank you for getting a start on this documentation, this adds a lot of clarity to what was being proposed! I have some questions, and some minor things to fix. Comment at: clang/docs/SafeBuffers.rst:19 +guidelines, and it provides runtime mitiga

[PATCH] D136811: WIP: RFC: NFC: C++ Buffer Hardening user documentation.

2022-10-26 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ created this revision. NoQ added reviewers: aaron.ballman, gribozavr2, xazax.hun, ldionne, jkorous, t-rasmud, ziqingluo-90, malavikasamak. Herald added subscribers: steakhal, martong, rnkovacs. Herald added a project: All. NoQ requested review of this revision. This document accompanies the L