[PATCH] D11103: Remove access to the DataLayout in the TargetMachine

David Blaikie dblaikie at gmail.com
Thu Jul 23 17:07:23 PDT 2015


dblaikie added a subscriber: dblaikie.
dblaikie added a comment.

The C API back compat looks plausible to me - but it's not an area I dabble in, so I've no idea how legitimate it is as an idea. Totally seems like a fair way to do back-compat.


================
Comment at: lib/Target/TargetMachineC.cpp:41
@@ +40,3 @@
+struct LLVMOpaqueTargetMachine {
+  TargetMachine *Machine;
+  DataLayout DL;
----------------
This could be a unique_ptr<TargetMachine>, perhaps?


http://reviews.llvm.org/D11103







More information about the llvm-commits mailing list