[PATCH] Expose MCInst in C Disassembler API

Kevin Enderby enderby at apple.com
Fri Jun 20 13:45:17 PDT 2014


Hi Matthew,

I think the C API is to be stable.  So changing LLVMDisasmInstruction in include/llvm-c/Disassembler.h by adding the LLVMMCInstRef* Inst break things.  Don’t think you can do that.

Kev
 
On Jun 20, 2014, at 12:08 PM, Matthew Maurer <maurer+llvmphabricator at matthewmaurer.org> wrote:

> Partially expose the MCInst struct from the C Disassembler API to allow mechanized examination of opcodes and arguments.
> 
> http://reviews.llvm.org/D4236
> 
> Files:
>  include/llvm-c/Disassembler.h
>  include/llvm-c/MCInst.h
>  lib/MC/MCDisassembler/Disassembler.cpp
>  lib/MC/MCInst.cpp
>  tools/llvm-c-test/disassemble.c
> <D4236.10701.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list