<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="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 11:20 AM, David Blaikie <dblaikie@gmail.com> wrote:<br></div><div> <br></div><blockquote class="protonmail_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">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 target="_blank" rel="noreferrer" href="http://llvm.org/viewvc/llvm-project?rev=329258&view=rev">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" rel="noreferrer" href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MIRCanonicalizerPass.cpp?rev=329258&r1=329257&r2=329258&view=diff">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" rel="noreferrer" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br></div></blockquote></div></blockquote><div><br></div>