<div dir="ltr">This patch can't wait in your local checkout until the rest (or some draft of it) is ready to start review.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 8, 2016 at 11:53 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">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 class="gmail_msg">After all, it wouldn't make much sense to stop after converting a few of these functions.</div><div class="gmail_msg"><span class="gmail_msg"></span><br class="gmail_msg"><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"> 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"><div dir="ltr" class="gmail_msg">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 class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Thu, Sep 8, 2016 at 11:34 AM Filipe Cabecinhas <<a class="gmail_msg">filcab+llvm.phabricator@gmail.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" 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 class="gmail_msg">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>
</blockquote></div></div></blockquote></div>