[PATCH] D117296: Document several clang-supported builtins

2022-01-14 Thread serge via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6cacd420a1d7: Document several clang-supported builtins (authored by serge-sans-paille). Changed prior to commit:

[PATCH] D117296: Document several clang-supported builtins

2022-01-14 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from a minor nit with duplicate punctuation. Comment at: clang/docs/LanguageExtensions.rst:2653 +data is going to be reused a lot once in cache.

[PATCH] D117296: Document several clang-supported builtins

2022-01-14 Thread serge via Phabricator via cfe-commits
serge-sans-paille updated this revision to Diff 400080. serge-sans-paille marked 8 inline comments as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117296/new/ https://reviews.llvm.org/D117296 Files: clang/docs/LanguageExtensions.rst Index: clang/docs/LanguageExtensions.rst

[PATCH] D117296: Document several clang-supported builtins

2022-01-14 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Thanks for this new documentation! I'm by no means an expert in these builtins, so I had some questions on places we might want to expand things a bit more. Comment at: clang/docs/LanguageExtensions.rst:2183 +n *

[PATCH] D117296: Document several clang-supported builtins

2022-01-14 Thread serge via Phabricator via cfe-commits
serge-sans-paille created this revision. serge-sans-paille added reviewers: aaron.ballman, rsmith. serge-sans-paille requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Namely __builtin_alloca __builtin_alloca_with_align