[PATCH] D65295: [opt] Ensure the IR-Linker is available to plugins.

Fri Jul 26 06:57:36 PDT 2019

Meinersbur added a comment.

In D65295#1601930 <https://reviews.llvm.org/D65295#1601930>, @mehdi_amini wrote:

> What about "force-load" the archive libraries on link? See what we do in MLIR here: https://github.com/tensorflow/mlir/blob/master/CMakeLists.txt#L22

Thank you for the suggestion. During a first test with `--whole-archive`, I got multiple definitions errors. I will try a bit more.

As another possibility in cmake we could link the object libraries instead of the archive,

In D65295#1602219 <https://reviews.llvm.org/D65295#1602219>, @serge-sans-paille wrote:

> @Meinersbur what about linking with `LLVM-IR linker` in your shared module?

This would cause global variables to be present multiple times within the same address space. If it is an `cl::opt`, it will immediately error-out.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65295/new/

https://reviews.llvm.org/D65295