<div>Yep, I am the only author at the moment and the file is pretty new is why I propose it. <br></div><div><br></div><div>PL<br></div><div><br></div><div class="protonmail_signature_block"><div class="protonmail_signature_block-user protonmail_signature_block-empty"><br></div><div class="protonmail_signature_block-proton">Sent with <a href="https://protonmail.com" target="_blank">ProtonMail</a> Secure Email.<br></div></div><div><br></div><div>‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐<br></div><div> On April 9, 2018 7:12 PM, David Blaikie via llvm-commits <llvm-commits@lists.llvm.org> wrote:<br></div><div> <br></div><blockquote class="protonmail_quote" type="cite"><div dir="ltr">It's usually best to avoid formatting a whole file if you can help it  - as it can complicate the revision history a bit. But if a file is way out of format & you're about to do a bunch of work on it (such that your new, well formatted code would be out of place in the otherwise misformatted file), or you're the only author & the file is relatively new, it can be worthwhile/the right way to go.<br></div><div><br></div><div class="gmail_quote"><div dir="ltr">On Mon, Apr 9, 2018 at 7:10 PM Puyan Lotfi <<a href="mailto:puyan@puyan.org">puyan@puyan.org</a>> wrote:<br></div><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex" class="gmail_quote"><div>Ah, sorry this was a quick change to quite some unfortunate Release mode warnings. I'll probably do a whole commit just to run clang-format on the whole file. Thanks for the heads up.<br></div><div><br></div><div>PL<br></div><div><br></div><div class="m_7386008715309145953protonmail_signature_block"><div class="m_7386008715309145953protonmail_signature_block-user m_7386008715309145953protonmail_signature_block-empty"><br></div><div class="m_7386008715309145953protonmail_signature_block-proton">Sent with <a target="_blank" href="https://protonmail.com">ProtonMail</a> Secure Email.<br></div></div><div><br></div><div>‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐<br></div><div>On April 9, 2018 11:20 AM, David Blaikie <<a target="_blank" href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br></div><div><br></div><blockquote type="cite" class="m_7386008715309145953protonmail_quote"><div dir="ltr">Probably best to run code changes through clang-format, which I think would put these two statements on separate lines.<br></div><div><br></div><div class="gmail_quote"><div dir="ltr">On Wed, Apr 4, 2018 at 11:59 PM Puyan Lotfi via llvm-commits <<a target="_blank" href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Author: zer0<br></div><div>Date: Wed Apr  4 23:56:44 2018<br></div><div>New Revision: 329258<br></div><div><br></div><div>URL: <a target="_blank" href="http://llvm.org/viewvc/llvm-project?rev=329258&view=rev" rel="noreferrer">http://llvm.org/viewvc/llvm-project?rev=329258&view=rev</a><br></div><div>Log:<br></div><div>[MIR-Canon] Fixing warnings in Non-assert builds.<br></div><div><br></div><div>Modified:<br></div><div>    llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp<br></div><div><br></div><div>Modified: llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp<br></div><div>URL: <a target="_blank" href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp?rev=329258&r1=329257&r2=329258&view=diff" rel="noreferrer">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp?rev=329258&r1=329257&r2=329258&view=diff</a><br></div><div>==============================================================================<br></div><div>--- llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp (original)<br></div><div>+++ llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp Wed Apr  4 23:56:44 2018<br></div><div>@@ -484,7 +484,7 @@ GetVRegRenameMap(const std::vector<Typed<br></div><div>       // (especially when comparing SelectionDAG to GlobalISel generated MIR)<br></div><div>       // that in the other file we are just getting an incoming vreg that comes<br></div><div>       // from a copy from a frame index. So it's safe to skip by one.<br></div><div>-      unsigned LastRenameReg = NVC.incrementVirtualVReg();<br></div><div>+      unsigned LastRenameReg = NVC.incrementVirtualVReg(); (void)LastRenameReg;<br></div><div>       DEBUG(dbgs() << "Skipping rename for FI " << LastRenameReg << "\n";);<br></div><div>       continue;<br></div><div>     } else if (vreg.isCandidate()) {<br></div><div>@@ -500,7 +500,7 @@ GetVRegRenameMap(const std::vector<Typed<br></div><div>       FirstCandidate = false;<br></div><div>       continue;<br></div><div>     } else if (!TargetRegisterInfo::isVirtualRegister(vreg.getReg())) {<br></div><div>-      unsigned LastRenameReg = NVC.incrementVirtualVReg();<br></div><div>+      unsigned LastRenameReg = NVC.incrementVirtualVReg(); (void)LastRenameReg;<br></div><div>       DEBUG({<br></div><div>         dbgs() << "Skipping rename for Phys Reg " << LastRenameReg << "\n";<br></div><div>       });<br></div><div><br></div><div><br></div><div>_______________________________________________<br></div><div>llvm-commits mailing list<br></div><div><a target="_blank" href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br></div><div><a target="_blank" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br></div></blockquote></div></blockquote><div><br></div></blockquote></div></blockquote><div><br></div>