[PATCH] D55682: [ELF] Support defining __start/__stop symbols as hidden

[George Rimar via Phabricator via llvm-commits]

grimar added a comment.

`-z hide-start-stop` maybe? (to shorten the name a bit)



================
Comment at: lld/ELF/Writer.cpp:1849
     return;
-  addOptionalRegular(Saver.save("__start_" + S), Sec, 0, STV_PROTECTED);
-  addOptionalRegular(Saver.save("__stop_" + S), Sec, -1, STV_PROTECTED);
+  uint8_t B = Config->ZHiddenStartStopSymbols ? STV_HIDDEN : STV_PROTECTED;
+  addOptionalRegular(Saver.save("__start_" + S), Sec, 0, B);
----------------
What is `B` stands for? I would expect to see `V` (Visibility).


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D55682