<div dir="ltr">I committed the code reorganization in r239788.</div><div class="gmail_extra"><br><div class="gmail_quote">2015-06-15 16:18 GMT-07:00 Alex L <span dir="ltr"><<a href="mailto:arphaman@gmail.com" target="_blank">arphaman@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">2015-06-15 12:14 GMT-07:00 Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><br>
> On 2015-Jun-15, at 10:50, Alex Lorenz <<a href="mailto:arphaman@gmail.com" target="_blank">arphaman@gmail.com</a>> wrote:<br>
><br>
> Hi dexonsmith, bob.wilson, bogner,<br>
><br>
> This patch is based on the patch that connects MIRParser to the machine function analysis pass (<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D9928&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=0g_0CROoMQHy-LyGpd8g_thnOoQMvGBefMCmPZTZm1U&s=B2trOxLYe9eOxAZhOvmvBuVW5iuRe14CQNVjkuv_lxs&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D9928</a>).<br>
><br>
> This patch prints and parses the simple, scalar attributes of the machine function. It also moves the mir printing code out of the mir printing pass and into a separate file to allow it to be moved into a separate machineIR library in the future.<br>
<br>
</span>Both changes LGTM, but they shouldn't be bundled up like this.<br>
<br>
Please commit the code reorganization first, and then add the new<br>
features in a second (separate) commit.<br></blockquote><div><br></div></span><div>Thanks, I will do that.</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><br>
><br>
> REPOSITORY<br>
>  rL LLVM<br>
><br>
> <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10449&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=0g_0CROoMQHy-LyGpd8g_thnOoQMvGBefMCmPZTZm1U&s=AFQCmaEUV9Br5ntyEcZhy6CjSwdaMoDmFZKgqbbzA3E&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10449</a><br>
><br>
> Files:<br>
>  include/llvm/CodeGen/MIRYamlMapping.h<br>
>  lib/CodeGen/CMakeLists.txt<br>
>  lib/CodeGen/MIRParser/MIRParser.cpp<br>
>  lib/CodeGen/MIRPrinter.cpp<br>
>  lib/CodeGen/MIRPrinter.h<br>
>  lib/CodeGen/MIRPrintingPass.cpp<br>
>  test/CodeGen/MIR/machine-function.mir<br>
><br>
> EMAIL PREFERENCES<br>
>  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=0g_0CROoMQHy-LyGpd8g_thnOoQMvGBefMCmPZTZm1U&s=xgMAYKBN3QEdwkvx3enccf9KHzh4sLTWPvMGPk6mMIE&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</span>> <D10449.27687.patch><br>
<br>
</blockquote></span></div><br></div></div>
</blockquote></div><br></div>