[PATCH] D29104: Add !associated metadata.

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 17 15:11:36 PST 2017


mehdi_amini added a comment.

The IR part (LangRef) seems OK to me. 
The ELF part is out of my scope, if rafael or pcc already checked it, that's fine.



================
Comment at: docs/LangRef.rst:5113
+this feature is spotty. For best compatibility, globals carrying this
+metadata must also
+
----------------
colon?
Also, since it seems to be advisory and not mandatory I wouldn't use "must" but "may".


Repository:
  rL LLVM

https://reviews.llvm.org/D29104





More information about the llvm-commits mailing list