[PATCH] D108479: [Clang] Add __builtin_addressof_nocfi

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 2 20:20:21 PST 2021


rjmccall accepted this revision.
rjmccall added a comment.

This looks great, thanks.  Please feel free to commit with the requested minor change to the docs.



================
Comment at: clang/docs/LanguageExtensions.rst:2555
+This builtin is not supported on all targets.  The returned pointer may differ
+from the normally taken function address and is not safe to call.
+
----------------
Please take this last sentence and make it the first sentence of the next paragraph; I think it leads into that example very nicely.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108479



More information about the cfe-commits mailing list