<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 9, 2015 at 11:42 AM, Shankar Kalpathi Easwaran <span dir="ltr"><<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">================<br>
Comment at: lib/ReaderWriter/ELF/DefaultLayout.h:350<br>
@@ +349,3 @@<br>
+/// override some of the functionalities.<br>
+template <class ELFT> class ScriptLayout : public DefaultLayout<ELFT> {<br>
+public:<br>
----------------<br>
</span><span class="">ruiu wrote:<br>
> shankarke wrote:<br>
> > rafaelauler wrote:<br>
> > > ruiu wrote:<br>
> > > > Merge this with DefaultLayout and remove this class.<br>
> > > No problem, but first can we get a consensus on whether do we need ScriptLayout or not? I'm asking this because I only moved the code out of DefaultLayout due to Shankar's solicitation.<br>
> > DefaultLayout was designed as a class to handle generic ELF Layout when not having linker scripts. Some of the linker script usecases are not even relevant for the DefaultLayout. Conditionally executing things in the DefaultLayout if we have a linker script IMO is a bad design.<br>
> I'm sorry but I completely disagree. I need to keep an eye on this kind of overkill abstraction as the history of this project shows. I spent so much time to reduce complexity of the linker, such as removal of InputGraph. Don't overdesign class hierarchy. That's going to become technical debt.<br>
</span>I dont think making changes in the DefaultLayout to accomodate LinkerScripts is a good idea.<br>
<br>
If others / Rafael think its a great idea, please go for it.</blockquote><div><br></div><div>Maybe you are not going to be a person to pay technical debt, but others will (or be suffered).</div></div></div></div>