<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.</div><br><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 class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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 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 11:20 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br></div><div> <br></div><blockquote class="m_7386008715309145953protonmail_quote" type="cite"><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 href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex" class="gmail_quote"><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 rel="noreferrer" href="http://llvm.org/viewvc/llvm-project?rev=329258&view=rev" target="_blank">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 rel="noreferrer" href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp?rev=329258&r1=329257&r2=329258&view=diff" target="_blank">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 href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br></div><div> <a rel="noreferrer" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br></div></blockquote></div></blockquote><div><br></div></blockquote></div>