
<html><head></head><body>The wins of being able to remove needles templates far exceeds the cost of and extra class IMHO.<br>

<br>

The write method is the only thing that needs ELFT, so the class is just syntax for holding a function pointer.<br>

<br>

I am definitely up for suggestions on how to remove these templates some other way, but keeping all this just because of write doesn't look like a good idea.<br>

<br>

Cheers,<br>

Rafael<br><br><div class="gmail_quote">On May 12, 2017 2:07:46 PM EDT, Rui Ueyama via Phabricator <reviews@reviews.llvm.org> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">ruiu added a comment.<br /><br />I'm not in favor of this change. I don't want to introduce a new class in general unless it is worthwhile. It seems this patch introduced a new class hierarchy too casually.<br /><br /><br /><a href="https://reviews.llvm.org/D33124">https://reviews.llvm.org/D33124</a><br /><br /><br /><br /></pre></blockquote></div><br>

-- <br>

Sent from my Android device with K-9 Mail. Please excuse my brevity.</body></html>