I'll keep it as is, but having a hypothetical case is very different from "we're in the middle of doing this, that's the end goal, I'd like so save a bunch of work, even if right now the savings aren't that big".<div>After all, it wouldn't make much sense to stop after converting a few of these functions.<span></span><br><div><br></div><div>Thank you,</div><div><br></div><div> Filipe</div><div><br><br>On Thursday, 8 September 2016, Vitaly Buka <<a href="mailto:vitalybuka@google.com">vitalybuka@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Then let's keep it on hold. It does not improve code as is. We should avoid adding real complexity to simplify hypothetical use-cases.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 8, 2016 at 11:34 AM Filipe Cabecinhas <<a href="javascript:_e(%7B%7D,'cvml','filcab%2Bllvm.phabricator@gmail.com');" target="_blank">filcab+llvm.phabricator@<wbr>gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It will save much more after the other errors are reified. Putting it in now avoids writing all that mechanical code which we know we will delete.<div><br></div><div>All the Report* functions (one per error) will have a structure to describe that error. Now there are 4 of those. Later they will be around 12 (don't have the code handy right now).<span></span></div><div><br></div><div>Thank you, </div><div><br></div><div></div><div> Filipe</div><div><br><br>On Thursday, 8 September 2016, Vitaly Buka <<a href="javascript:_e(%7B%7D,'cvml','vitalybuka@google.com');" target="_blank">vitalybuka@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vitalybuka added a comment.<br>
<br>
It saves 9 line of code, but makes it less readable.<br>
I strongly prefer existing code.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23875" target="_blank">https://reviews.llvm.org/<wbr>D23875</a><br>
<br>
<br>
<br>
</blockquote></div></blockquote></div>
</blockquote></div></div>