Skip to content

Commit

Permalink
Restore LLVMLinkModules C API until it is properly deprecated.
Browse files Browse the repository at this point in the history
Add the enum "LLVMLinkerMode" back for backwards-compatibility and add the
linker mode parameter back to the "LLVMLinkModules" function. The paramter is
ignored and has no effect.

Patch provided by: Filip Pizlo
Reviewed by: Rafael and Sean

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@230988 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
ributzka committed Mar 2, 2015
1 parent e8676a2 commit 0a5c301
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 4 deletions.
2 changes: 1 addition & 1 deletion bindings/go/llvm/linker.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import "errors"

func LinkModules(Dest, Src Module) error {
var cmsg *C.char
failed := C.LLVMLinkModules(Dest.C, Src.C, 0, &cmsg)
failed := C.LLVMLinkModules(Dest.C, Src.C, C.LLVMLinkerDestroySource, &cmsg)
if failed != 0 {
err := errors.New(C.GoString(cmsg))
C.LLVMDisposeMessage(cmsg)
Expand Down
14 changes: 12 additions & 2 deletions include/llvm-c/Linker.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,23 @@
extern "C" {
#endif

/* This enum is provided for backwards-compatibility only. It has no effect. */
typedef enum {
LLVMLinkerDestroySource = 0, /* This is the default behavior. */
LLVMLinkerPreserveSource_Removed = 1 /* This option has been deprecated and
should not be used. */
} LLVMLinkerMode;

/* Links the source module into the destination module, taking ownership
* of the source module away from the caller. Optionally returns a
* human-readable description of any errors that occurred in linking.
* OutMessage must be disposed with LLVMDisposeMessage. The return value
* is true if an error occurred, false otherwise. */
* is true if an error occurred, false otherwise.
*
* Note that the linker mode parameter \p Unused is no longer used, and has
* no effect. */
LLVMBool LLVMLinkModules(LLVMModuleRef Dest, LLVMModuleRef Src,
unsigned Unused, char **OutMessage);
LLVMLinkerMode Unused, char **OutMessage);

#ifdef __cplusplus
}
Expand Down
2 changes: 1 addition & 1 deletion lib/Linker/LinkModules.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1777,7 +1777,7 @@ bool Linker::LinkModules(Module *Dest, Module *Src) {
//===----------------------------------------------------------------------===//

LLVMBool LLVMLinkModules(LLVMModuleRef Dest, LLVMModuleRef Src,
unsigned Unused, char **OutMessages) {
LLVMLinkerMode Unused, char **OutMessages) {
Module *D = unwrap(Dest);
std::string Message;
raw_string_ostream Stream(Message);
Expand Down

0 comments on commit 0a5c301

Please sign in to comment.