[PATCH] D12263: Fix LLVM C API for DataLayout

[Mehdi AMINI via llvm-commits]

joker.eph added a comment.

There is no API changes in this patch: `LLVMGetTargetMachineData ()` was already deprecated, and I don't introduce `LLVMGetDataLayout()` it was already there.

This patch just fixes a bug that I introduced last month, in the way I tried to support this deprecated API with respect to the new C++ API. Because the current C++ API does not provide anymore a way to implement `LLVMGetTargetMachineData`, I had to hack around to keep it alive. Unfortunately it is broken with JIT clients.

I know you are interested in the C API, but the discussion about how to move forward on the C API policy is separated from this patch, ping me Monday and I'll try to make progress on this.


http://reviews.llvm.org/D12263