Skip to content

Commit

Permalink
libLTO: Add a utility method to initialize the disassemblers.
Browse files Browse the repository at this point in the history
Necessary to give disassembler users (like darwin's otool) a possibility to
dlopen libLTO and still initialize the required LLVM bits. This used to go
through libMCDisassembler but that's a gross layering violation, the MC layer
can't pull in functions from the targets. Adding a function to libLTO is a bit
of a hack but not worse than exposing other disassembler bits from libLTO.

Fixes PR14362.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@168545 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
d0k committed Nov 24, 2012
1 parent d3022b8 commit 8a2ce5d
Show file tree
Hide file tree
Showing 5 changed files with 35 additions and 13 deletions.
7 changes: 7 additions & 0 deletions include/llvm-c/lto.h
Original file line number Diff line number Diff line change
Expand Up @@ -291,6 +291,13 @@ lto_codegen_compile_to_file(lto_code_gen_t cg, const char** name);
extern void
lto_codegen_debug_options(lto_code_gen_t cg, const char *);

/**
* Initializes LLVM disassemblers.
* FIXME: This doesn't really belong here.
*/
extern void
lto_initialize_disassembler(void);

#ifdef __cplusplus
}
#endif
Expand Down
13 changes: 0 additions & 13 deletions lib/MC/MCDisassembler/Disassembler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
#include "llvm/MC/MCSubtargetInfo.h"
#include "llvm/Support/MemoryObject.h"
#include "llvm/Support/TargetRegistry.h"
#include "llvm/Support/TargetSelect.h"
#include "llvm/Support/ErrorHandling.h"

namespace llvm {
Expand All @@ -38,18 +37,6 @@ using namespace llvm;
LLVMDisasmContextRef LLVMCreateDisasm(const char *TripleName, void *DisInfo,
int TagType, LLVMOpInfoCallback GetOpInfo,
LLVMSymbolLookupCallback SymbolLookUp) {
// Initialize targets and assembly printers/parsers.
// FIXME: Clients are responsible for initializing the targets. And this
// would be done by calling routines in "llvm-c/Target.h" which are static
// line functions. But the current use of LLVMCreateDisasm() is to dynamically
// load libLTO with dlopen() and then lookup the symbols using dlsym().
// And since these initialize routines are static that does not work which
// is why the call to them in this 'C' library API was added back.
llvm::InitializeAllTargetInfos();
llvm::InitializeAllTargetMCs();
llvm::InitializeAllAsmParsers();
llvm::InitializeAllDisassemblers();

// Get the target.
std::string Error;
const Target *TheTarget = TargetRegistry::lookupTarget(TripleName, Error);
Expand Down
1 change: 1 addition & 0 deletions tools/lto/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ add_definitions( -DLLVM_VERSION_INFO=\"${PACKAGE_VERSION}\" )

set(SOURCES
LTOCodeGenerator.cpp
LTODisassembler.cpp
lto.cpp
LTOModule.cpp
)
Expand Down
26 changes: 26 additions & 0 deletions tools/lto/LTODisassembler.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
//===-- LTODisassembler.cpp - LTO Disassembler interface ------------------===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
//
// This function provides utility methods used by clients of libLTO that want
// to use the disassembler.
//
//===----------------------------------------------------------------------===//

#include "llvm-c/lto.h"
#include "llvm/Support/TargetSelect.h"

using namespace llvm;

void lto_initialize_disassembler() {
// Initialize targets and assembly printers/parsers.
llvm::InitializeAllTargetInfos();
llvm::InitializeAllTargetMCs();
llvm::InitializeAllAsmParsers();
llvm::InitializeAllDisassemblers();
}
1 change: 1 addition & 0 deletions tools/lto/lto.exports
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
lto_get_error_message
lto_get_version
lto_initialize_disassembler
lto_module_create
lto_module_create_from_fd
lto_module_create_from_fd_at_offset
Expand Down

0 comments on commit 8a2ce5d

Please sign in to comment.