Skip to content

Commit

Permalink
Adjust build systems to new file location and linking.
Browse files Browse the repository at this point in the history
rootcling now links against libRIO, libCore, libCling.
Any symbol from llvm/clang must now be resolved from within libCling.
libCling thus exports only ROOT_rootcling_Driver and CreateInterpreter().
This will allow to fix the symbol visibility inconsistency.
And it reduces link time.
  • Loading branch information
Axel-Naumann authored and sftnight committed Dec 20, 2016
1 parent 88c48f6 commit 2a7ff15
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions Module.mk
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,11 @@ INCLUDEFILES += $(CLINGDEP)
# 2) rely on TCling to addIncludePath instead of using CLING_..._INCL below
# -fvisibility=hidden renders libCore unusable.
# Filter out warning flags.
CLINGLLVMCXXFLAGS = $(filter-out -fvisibility-inlines-hidden,$(filter-out -fvisibility=hidden,\
CLINGLLVMCXXFLAGSRAW = $(shell $(LLVMCONFIG) --cxxflags)
CLINGLLVMCXXFLAGS = $(filter-out -pedantic,$(filter-out -fvisibility-inlines-hidden,$(filter-out -fvisibility=hidden,\
$(filter-out -W%,\
$(patsubst -O%,,$(shell $(LLVMCONFIG) --cxxflags)))))
$(patsubst -O%,,$(CLINGLLVMCXXFLAGSRAW)))))) \
$(filter -Wno-%,$(CLINGLLVMCXXFLAGSRAW))
# -ffunction-sections breaks the debugger on some platforms ... and does not help libCling at all.

# FIXME: This is temporary until I update my compiler on mac and add -fmodules-local-submodule-visibility.
Expand Down

0 comments on commit 2a7ff15

Please sign in to comment.