[sledge] Make workaround for broken llvm-config easier

Summary:
Change the Makefile comment about how to enable shared library linking
despite llvm-config being broken into a rule.

Reviewed By: ngorogiannis

Differential Revision: D28907814

fbshipit-source-id: ac63d0104
master
Josh Berdine 3 years ago committed by Facebook GitHub Bot
parent 0d430efb42
commit 713d090743

@ -27,13 +27,14 @@ LLVM_OCAML_SRC = $(shell git ls-files -- vendor/llvm-dune)
# file to use as a sentinel indicating llvm ocaml bindings are up-to-date # file to use as a sentinel indicating llvm ocaml bindings are up-to-date
LLVM_OCAML_SENTINEL=vendor/llvm-dune/src/llvm/common/dune LLVM_OCAML_SENTINEL=vendor/llvm-dune/src/llvm/common/dune
# Note that this does not correctly detect llvm shared libraries on # Note that llvm-config is broken and does not correctly detect llvm
# macos since llvm-config is broken. A filthy workaround seems to be # shared libraries. A filthy workaround seems to be to create the
# to create the libLLVM-11.dylib file llvm-config looks for as a # libLLVM-11.dylib file llvm-config looks for as a symbolic link to
# symbolic link to libLLVM.dylib. That is, to manually in the shell # libLLVM.dylib.
# execute: link_llvm_dylib:
# (LLVMLIB=$($(opam config var conf-llvm:config) --libdir); @set -e ;\
# ln -s $LLVMLIB/libLLVM.dylib $LLVMLIB/libLLVM-11.dylib) LLVMLIB=$$($$(opam config var conf-llvm:config) --libdir) ;\
ln -s $$LLVMLIB/libLLVM.dylib $$LLVMLIB/libLLVM-11.dylib
$(LLVM_OCAML_SENTINEL): $(LLVM_OCAML_SRC) $(LLVM_OCAML_SENTINEL): $(LLVM_OCAML_SRC)
cd vendor/llvm-dune; ./setup.sh $$(opam config var conf-llvm:config) &>/dev/null cd vendor/llvm-dune; ./setup.sh $$(opam config var conf-llvm:config) &>/dev/null

Loading…
Cancel
Save