Skip to content

Commit 195fcf4

Browse files
authored
Added GNU compile flags to enable -fPIC for llvm (#4553)
Currently, running ./configure with --enable-fpic=yes and --comp=llvm does not add the `-fPIC` flag. This PR adds the -fPIC flag to c++ and fortran compile flags when using GNUMake and LLVM.
1 parent dc8243b commit 195fcf4

File tree

2 files changed

+17
-4
lines changed

2 files changed

+17
-4
lines changed

Tools/GNUMake/comps/gnu.mak

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,8 @@ endif
133133

134134

135135
ifeq ($(USE_COMPILE_PIC),TRUE)
136-
CXXFLAGS = -fPIC
137-
CFLAGS = -fPIC
136+
CXXFLAGS += -fPIC
137+
CFLAGS += -fPIC
138138
endif
139139

140140
ifeq ($(ERROR_DEPRECATED),TRUE)
@@ -195,8 +195,8 @@ endif
195195

196196
ifeq ($(USE_COMPILE_PIC),TRUE)
197197

198-
FFLAGS = -fPIC
199-
F90FLAGS = -fPIC
198+
FFLAGS += -fPIC
199+
F90FLAGS += -fPIC
200200

201201
endif
202202

Tools/GNUMake/comps/llvm.mak

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,11 @@ ifeq ($(WARN_ERROR),TRUE)
5959
CFLAGS += -Werror
6060
endif
6161

62+
ifeq ($(USE_COMPILE_PIC),TRUE)
63+
CXXFLAGS += -fPIC
64+
CFLAGS += -fPIC
65+
endif
66+
6267
# disable some warnings
6368
CXXFLAGS += -Wno-c++17-extensions
6469

@@ -73,6 +78,14 @@ endif
7378
CXXFLAGS += -std=$(CXXSTD)
7479
CFLAGS += -std=c11
7580

81+
82+
ifeq ($(USE_COMPILE_PIC),TRUE)
83+
84+
FFLAGS += -fPIC
85+
F90FLAGS += -fPIC
86+
87+
endif
88+
7689
FFLAGS += -ffixed-line-length-none -fno-range-check -fno-second-underscore
7790
F90FLAGS += -ffree-line-length-none -fno-range-check -fno-second-underscore -fimplicit-none
7891

0 commit comments

Comments
 (0)