[llvm-commits] [llvm] r74038 - /llvm/trunk/include/llvm/MC/MCStreamer.h

[Bill Wendling]

On Jun 23, 2009, at 5:51 PM, Chris Lattner <sabre at nondot.org> wrote:

> Author: lattner
> Date: Tue Jun 23 19:51:35 2009
> New Revision: 74038
>
> URL: http://llvm.org/viewvc/llvm-project?rev=74038&view=rev
> Log:
> stub out a trivial constructor method.
>
> Modified:
>    llvm/trunk/include/llvm/MC/MCStreamer.h
>
> Modified: llvm/trunk/include/llvm/MC/MCStreamer.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCStreamer.h?rev=74038&r1=74037&r2=74038&view=diff
>
> ===
> ===
> ===
> =====================================================================
> --- llvm/trunk/include/llvm/MC/MCStreamer.h (original)
> +++ llvm/trunk/include/llvm/MC/MCStreamer.h Tue Jun 23 19:51:35 2009
> @@ -116,7 +116,7 @@
>   /// createAsmStreamer - Create a machine code streamer which will  
> print out
>   /// assembly for the native target, suitable for compiling with a  
> native
>   /// assembler.
> -  MCStreamer *createAsmStreamer(MCContext &Ctx, raw_ostream &OS);
> +  inline MCStreamer *createAsmStreamer(MCContext &Ctx, raw_ostream  
> &OS) { return 0; }
>
It's a good idea to omit parameter names if they aren't used. This  
prevents spurious warning in Release mode. :-)

-bw


>   // FIXME: These two may end up getting rolled into a single
>   // createObjectStreamer interface, which implements the assembler  
> backend, and
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits