<div dir="ltr">LGTM, thanks.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 6, 2014 at 12:49 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping.<div><div class="h5"><br>
<br>
On 26/02/2014 13:23, Tobias Grosser wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
Hi,<br>
<br>
I would like to commit this patch (mostly mechanical)<br>
<br>
[PATCH] tblgen: Modularize the diagnostic emitter<br>
<br>
Replace a large monolithic function with per-table functions which all<br>
nicely fit on my screen. I also added documentation to each function<br>
that describes what kind of tables are generated and which information<br>
is contained. Finally, I run clang-format over the moved code.<br>
<br>
I spent a significant amount of time to understand this code when<br>
reasoning about possible extensions to the diagnostic interface to<br>
support 'remark' diagnostics. This change will definitely help such an<br>
implementation, but already by itself it will save other people a lot of<br>
time when trying to understand this functionality.<br>
<br>
Even though the patch touches the full function, it is mostly<br>
mechanical. No functional change intended. The generated tblgen files<br>
are identical.<br>
---<br>
  utils/TableGen/<u></u>ClangDiagnosticsEmitter.cpp | 257<br>
+++++++++++++++++++++--------<br>
  1 file changed, 185 insertions(+), 72 deletions(-)<br>
<br>
As it touches a couple of lines, I would appreciate a quick review.<br>
<br>
Cheers,<br>
Tobias<br>
<br>
<br>
<br></div></div><div class="">
______________________________<u></u>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
<br>
</div></blockquote>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>