[PATCH] D36579: [ELF] - Do not fail when set versions for linkerscript's symbol aliases

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: ELF/Driver.cpp:978
+template <class ELFT>
+static void addScriptSymbols() {
+  // Some symbols (such as __ehdr_start) are defined lazily only when there
----------------
Inline this function.


================
Comment at: ELF/Driver.cpp:984
+
+  // Add symbols defined in linkerscript.
+  for (BaseCommand *Base : Script->Opt.Commands)
----------------
Add symbols defined in linker scripts early because otherwise version
scripts referring script-synthesized symbols wouldn't work due to undefined
symbol error. (Version scripts are processed earlier than linker scripts.)


https://reviews.llvm.org/D36579