<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="mailto:filcab%2Bllvm.phabricator@gmail.com">filcab+llvm.phabricator@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 class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">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 class="gmail_msg"></span></div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Thank you, </div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"></div><div class="gmail_msg"> Filipe</div><div class="gmail_msg"><br class="gmail_msg"><br class="gmail_msg">On Thursday, 8 September 2016, Vitaly Buka <<a href="mailto:vitalybuka@google.com" class="gmail_msg" target="_blank">vitalybuka@google.com</a>> wrote:<br class="gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vitalybuka added a comment.<br class="gmail_msg">
<br class="gmail_msg">
It saves 9 line of code, but makes it less readable.<br class="gmail_msg">
I strongly prefer existing code.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D23875" class="gmail_msg" target="_blank">https://reviews.llvm.org/D23875</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></blockquote></div>