[PATCH] D27276: [ELF] Allow defined symbols to be assigned from linker script

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 7 00:46:16 PST 2017


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: ELF/LinkerScript.cpp:82
+  replaceBody<DefinedSynthetic>(Sym, Cmd->Name, 0, Sec);
 
+  return Sym->body();
----------------
nit: remove this blank line so that it matches with the style in the above function.


https://reviews.llvm.org/D27276





More information about the llvm-commits mailing list