[PATCH] D57190: [MC] Teach the MachO object writer about N_FUNC_COLD

[Nico Weber via Phabricator via llvm-commits]

thakis added inline comments.
Herald added a subscriber: MaskRay.
Herald added a reviewer: jhenderson.
Herald added a project: LLVM.


================
Comment at: llvm/trunk/include/llvm/BinaryFormat/MachO.h:337
   N_ALT_ENTRY = 0x0200u,
+  N_COLD_FUNC = 0x0400u,
   // For undefined symbols coming from libraries, see GET_LIBRARY_ORDINAL()
----------------
This (and the previous bits too) overlap with the ordinal written by http://llvm-cs.pcc.me.uk/include/llvm/BinaryFormat/MachO.h/rSET_LIBRARY_ORDINAL 

I suppose the thinking is that that's safe because the ordinal is only set on undefs while these 3 bits here can't be set on undefs? If so, maybe this warrants a comment?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57190