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?<br><div class="gmail_quote">On Sun, Feb 1, 2015 at 8:36 PM Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Although I don't see major problems with your patch, I like Rui's proposed approach better.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7323" target="_blank">http://reviews.llvm.org/D7323</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>