[PATCH] D129151: [Metadata] Add 'exclude' metadata to add the exclude flags on globals
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 5 23:52:33 PDT 2022
jhenderson resigned from this revision.
jhenderson added a comment.
Spotted a couple of minor issues in the docs, but otherwise, I don't have the knowledge to review this.
================
Comment at: llvm/docs/LangRef.rst:6400
+'``exclude``' Metadata
+^^^^^^^^^^^^^^^^^^^^^^^
+
----------------
Nit: the underline is too long here, and will probably cause a doc build warning.
================
Comment at: llvm/docs/LangRef.rst:6405
+option is only valid for global variables with an explicit section targeting ELF
+or COFF.This is done using the ``SHF_EXCLUDE`` flag on ELF targets and the
+``IMAGE_SCN_LNK_REMOVE`` and ``IMAGE_SCN_MEM_DISCARDABLE`` flags for COFF
----------------
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129151/new/
https://reviews.llvm.org/D129151
More information about the llvm-commits
mailing list