[PATCH] D23315: [ELF] - Let linkerscript to use own outputsections factory.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 10 18:26:53 PDT 2016


On Wed, Aug 10, 2016 at 12:29 AM, George Rimar <grimar at accesssoftek.com>
wrote:

> grimar added a comment.
>
> In https://reviews.llvm.org/D23315#510206, @ruiu wrote:
>
> > Is there any problem if we group sections just by name?
>
>
> Do you mean group into one output section by name ? That is exactly what
> this patch do.
> For linkerscript's OutputSectionFactory "key" is simple StringRef which is
> output section name.
> And I combine the flags when add more sections.
>

I meant to say that why don't you do this even in Writer.cpp.


>
>
> https://reviews.llvm.org/D23315
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160810/7d0713cf/attachment.html>


More information about the llvm-commits mailing list