[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