[llvm-commits] [llvm] r83123 - in /llvm/trunk: include/llvm/CodeGen/MachineModuleInfo.h lib/CodeGen/MachineModuleInfo.cpp

[Bill Wendling]

On Sep 29, 2009, at 5:14 PM, Devang Patel wrote:

> Author: dpatel
> Date: Tue Sep 29 19:14:40 2009
> New Revision: 83123
>
> URL: http://llvm.org/viewvc/llvm-project?rev=83123&view=rev
> Log:
> Simplify.
>
> Modified:
>    llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h
>    llvm/trunk/lib/CodeGen/MachineModuleInfo.cpp
>
> Modified: llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h?rev=83123&r1=83122&r2=83123&view=diff
>
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h (original)
> +++ llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h Tue Sep 29  
> 19:14:40 2009
> @@ -153,7 +153,7 @@
>
>   /// BeginFunction - Begin gathering function meta information.
>   ///
> -  void BeginFunction(MachineFunction *MF);
> +  void BeginFunction(MachineFunction *MF) {}
>
Could you remove the "MF" parameter name here? Some people might  
compile this with `-pedantic' or `-Wunused-variable' or some other  
silly warning and then it will break things and I'll be sad. :-)

-bw