[PATCH] [lld] Make ELFLinkingContext own LinkerScript buffers

Rafael Auler rafaelauler at gmail.com
Sun Feb 1 14:50:50 PST 2015


The reason I created a new class that encapsulates the parser is because I
plan to later add code to this class (or to another new class called
ScriptSema that is owned by ScriptInstance) that cooks the AST with
semantic info to answer high level questions about the linker script, such
as "in which output section to put this input section, according to this
linker script?". Anyway, I will need to find a location to put new data
structures to store this data, and I thought of this as a first step. What
do you think?
On Sun, Feb 1, 2015 at 8:36 PM Davide Italiano <davide at freebsd.org> wrote:

> Although I don't see major problems with your patch, I like Rui's proposed
> approach better.
>
>
> http://reviews.llvm.org/D7323
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150201/1a0797cb/attachment.html>


More information about the llvm-commits mailing list