[llvm-commits] [llvm] r104029 - /llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp

[Jeffrey Yasskin]

I'm guessing this is the change that broke
http://google1.osuosl.org:8011/builders/llvm-i686-linux-vg_leak/builds/755/steps/test-llvm/logs/mcinst-lowering.ll
(since it admits leaking memory). Could you fix it?

On Tue, May 18, 2010 at 10:22 AM, Daniel Dunbar <daniel at zuster.org> wrote:
> Author: ddunbar
> Date: Tue May 18 12:22:19 2010
> New Revision: 104029
>
> URL: http://llvm.org/viewvc/llvm-project?rev=104029&view=rev
> Log:
> llc (et al): Add support for --show-encoding and --show-inst.
>
> Modified:
>    llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp
>
> Modified: llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp?rev=104029&r1=104028&r2=104029&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp (original)
> +++ llvm/trunk/lib/CodeGen/LLVMTargetMachine.cpp Tue May 18 12:22:19 2010
> @@ -65,6 +65,10 @@
>     cl::desc("Print LLVM IR input to isel pass"));
>  static cl::opt<bool> PrintGCInfo("print-gc", cl::Hidden,
>     cl::desc("Dump garbage collector data"));
> +static cl::opt<bool> ShowMCEncoding("show-mc-encoding", cl::Hidden,
> +    cl::desc("Show encoding in .s output"));
> +static cl::opt<bool> ShowMCInst("show-mc-inst", cl::Hidden,
> +    cl::desc("Show instruction structure in .s output"));
>  static cl::opt<bool> VerifyMachineCode("verify-machineinstrs", cl::Hidden,
>     cl::desc("Verify generated machine code"),
>     cl::init(getenv("LLVM_VERIFY_MACHINEINSTRS")!=NULL));
> @@ -131,10 +135,18 @@
>   case CGFT_AssemblyFile: {
>     MCInstPrinter *InstPrinter =
>       getTarget().createMCInstPrinter(MAI.getAssemblerDialect(), MAI);
> +
> +    // Create a code emitter if asked to show the encoding.
> +    //
> +    // FIXME: These are currently leaked.
> +    MCCodeEmitter *MCE = 0;
> +    if (ShowMCEncoding)
> +      MCE = getTarget().createCodeEmitter(*this, *Context);
> +
>     AsmStreamer.reset(createAsmStreamer(*Context, Out,
>                                         getTargetData()->isLittleEndian(),
>                                         getVerboseAsm(), InstPrinter,
> -                                        /*codeemitter*/0));
> +                                        MCE, ShowMCInst));
>     break;
>   }
>   case CGFT_ObjectFile: {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>