[PATCH] D23751: [ELF] Linkerscript: define symbols outside SECTIONS
Petr Hosek via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 25 11:52:57 PDT 2016
phosek added a comment.
Do you have any opinion on what to do about this? I'd say that since we already support having multiple `SECTIONS` commands and it works reasonable (as in the same as in BFD ld), there's no reason to break that functionality in which case the current patch is probably a better albeit a slightly uglier solution? I still feel feel that renaming `HasContents` to `HasSections` would improve the readability.
https://reviews.llvm.org/D23751
More information about the llvm-commits
mailing list