Revert "Fix LLVM C API for DataLayout"

This reverts commit r246044.
Build broken, still. It builds for me...

From: Mehdi Amini <mehdi.amini@apple.com>
llvm-svn: 246049
This commit is contained in:
Mehdi Amini 2015-08-26 18:37:59 +00:00
parent 19c5488015
commit 8b3dda3f71
2 changed files with 22 additions and 21 deletions

View File

@ -59,13 +59,6 @@ class PassManagerBase;
}
using legacy::PassManagerBase;
extern "C" {
// This function from the C API is deprecated. We still supports it using a
// private method on the TargetMachine for now. But it needs to be friended and
// so we forward declare it here.
LLVMTargetDataRef LLVMGetTargetMachineData(LLVMTargetMachineRef T);
}
//===----------------------------------------------------------------------===//
///
/// Primary interface to the complete machine description for the target
@ -110,12 +103,6 @@ protected: // Can only create subclasses.
unsigned RequireStructuredCFG : 1;
/// This API is here to support the C API, deprecated in 3.7 release.
/// This should never be used outside of legacy existing client.
const DataLayout &getDataLayout() const { return DL; }
friend struct LLVMOpaqueTargetData *
LLVMGetTargetMachineData(LLVMTargetMachineRef T);
public:
mutable TargetOptions Options;

View File

@ -32,14 +32,25 @@
using namespace llvm;
// The TargetMachine uses to offer access to a DataLayout member. This is reflected
// in the C API. For backward compatibility reason, this structure allows to keep
// a DataLayout member accessible to C client that have a handle to a
// LLVMTargetMachineRef.
struct LLVMOpaqueTargetMachine {
std::unique_ptr<TargetMachine> Machine;
DataLayout DL;
};
static TargetMachine *unwrap(LLVMTargetMachineRef P) {
return reinterpret_cast<TargetMachine *>(P);
return P->Machine.get();
}
static Target *unwrap(LLVMTargetRef P) {
return reinterpret_cast<Target*>(P);
}
static LLVMTargetMachineRef wrap(const TargetMachine *P) {
return reinterpret_cast<LLVMTargetMachineRef>(const_cast<TargetMachine *>(P));
return new LLVMOpaqueTargetMachine{ std::unique_ptr<TargetMachine>(const_cast<TargetMachine*>(P)), P->createDataLayout() };
}
static LLVMTargetRef wrap(const Target * P) {
return reinterpret_cast<LLVMTargetRef>(const_cast<Target*>(P));
@ -68,16 +79,16 @@ LLVMTargetRef LLVMGetTargetFromName(const char *Name) {
LLVMBool LLVMGetTargetFromTriple(const char* TripleStr, LLVMTargetRef *T,
char **ErrorMessage) {
std::string Error;
*T = wrap(TargetRegistry::lookupTarget(TripleStr, Error));
if (!*T) {
if (ErrorMessage)
*ErrorMessage = strdup(Error.c_str());
return 1;
}
return 0;
}
@ -144,7 +155,10 @@ LLVMTargetMachineRef LLVMCreateTargetMachine(LLVMTargetRef T,
CM, OL));
}
void LLVMDisposeTargetMachine(LLVMTargetMachineRef T) { delete unwrap(T); }
void LLVMDisposeTargetMachine(LLVMTargetMachineRef T) {
delete T;
}
LLVMTargetRef LLVMGetTargetMachineTarget(LLVMTargetMachineRef T) {
const Target* target = &(unwrap(T)->getTarget());
@ -166,9 +180,9 @@ char* LLVMGetTargetMachineFeatureString(LLVMTargetMachineRef T) {
return strdup(StringRep.c_str());
}
/** Deprecated: use LLVMGetDataLayout(LLVMModuleRef M) instead. */
/// @deprecated: see "struct LLVMOpaqueTargetMachine" description above
LLVMTargetDataRef LLVMGetTargetMachineData(LLVMTargetMachineRef T) {
return wrap(&unwrap(T)->getDataLayout());
return wrap(&T->DL);
}
void LLVMSetTargetMachineAsmVerbosity(LLVMTargetMachineRef T,