<div dir="ltr">I think I don't like neither, as neither don't demonstrate what we can get from these changes.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 16, 2017 at 1:05 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><span class="">
<p>><span style="color:rgb(33,33,33);font-size:12pt">Please do not use inheritance. You don't need that at all here.</span></p>
</span><div style="color:rgb(33,33,33)">
<div>
<div dir="ltr">
<div class="gmail_extra"><br>
So do you prefer approach used in first version of patch ? (<a href="https://reviews.llvm.org/D30939?id=91714" target="_blank">https://reviews.llvm.org/<wbr>D30939?id=91714</a>),<br>
​it did not use inheritance but changed much more places in code.<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><span class="HOEnZb"><font color="#888888">
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">George.<br>
</div>
</font></span></div>
</div>
</div>
</div>

</blockquote></div><br></div>