<p dir="ltr"><br>
On Dec 1, 2014 5:10 PM, "Nick Lewycky" <<a href="mailto:nicholas@mxc.ca">nicholas@mxc.ca</a>> wrote:<br>
><br>
> Author: nicholas<br>
> Date: Mon Dec  1 19:09:56 2014<br>
> New Revision: 223101<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=223101&view=rev">http://llvm.org/viewvc/llvm-project?rev=223101&view=rev</a><br>
> Log:<br>
> Fix variable used only in assertion.<br>
><br>
> Modified:<br>
>     llvm/trunk/lib/Target/X86/X86MCInstLower.cpp<br>
><br>
> Modified: llvm/trunk/lib/Target/X86/X86MCInstLower.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86MCInstLower.cpp?rev=223101&r1=223100&r2=223101&view=diff">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86MCInstLower.cpp?rev=223101&r1=223100&r2=223101&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/Target/X86/X86MCInstLower.cpp (original)<br>
> +++ llvm/trunk/lib/Target/X86/X86MCInstLower.cpp Mon Dec  1 19:09:56 2014<br>
> @@ -816,8 +816,9 @@ static void LowerSTATEPOINT(MCStreamer &<br>
>    assert(Is64Bit && "Statepoint currently only supports X86-64");<br>
><br>
>    // We need to record the frame size for stack walking<br>
> -  const MachineFunction* MF = MI.getParent()->getParent();<br>
> +  const MachineFunction *MF = MI.getParent()->getParent();<br>
>    assert(MF && "can't find machine function?");<br>
> +  (void)MF;</p>
<p dir="ltr">Could we just roll the expression into the assertion? Seems simple enough.</p>
<p dir="ltr">><br>
>    //<br>
>    // Emit call instruction<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</p>