[PATCH] D38239: [ELF] - Define linkerscript symbols early.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 15 13:47:41 PDT 2017


ruiu added inline comments.


================
Comment at: ELF/LinkerScript.cpp:168-172
+    // Symbol already defined is a simple case, we only want to set the flag.
+    if (SymbolBody *B = Symtab->find(Cmd->Name)) {
+      B->symbol()->CanInline = false;
+      continue;
+    }
----------------
I don't think you need this.


https://reviews.llvm.org/D38239





More information about the llvm-commits mailing list