Skip to content

Commit 2d33e12

Browse files
authoredSep 30, 2021
Make sure that Netlib LAPACK respects FFLAGS
OpenBLAS allows users to specify `FFLAGS` and then uses `override` to append additional options. However, without such an override in lapack's make.inc, lapack would use the external FFLAGS, rather than the ones being computed by OpenBLAS. For example the `DEBUG=1` flag would not apply to LAPACK code. This is all a bit messy but forced by the integration with netlib lapack. Note that `CFLAGS` already has this override for the same reason. It is possible that other variables here should have a similar override, but I think for most of the other ones, OpenBLAS's build system does not append to the flags passed in by the user.
1 parent 2405550 commit 2d33e12

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

‎Makefile

+1-1
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ prof_lapack : lapack_prebuild
269269
lapack_prebuild :
270270
ifeq ($(NOFORTRAN), $(filter 0,$(NOFORTRAN)))
271271
-@echo "FC = $(FC)" > $(NETLIB_LAPACK_DIR)/make.inc
272-
-@echo "FFLAGS = $(LAPACK_FFLAGS)" >> $(NETLIB_LAPACK_DIR)/make.inc
272+
-@echo "override FFLAGS = $(LAPACK_FFLAGS)" >> $(NETLIB_LAPACK_DIR)/make.inc
273273
-@echo "FFLAGS_DRV = $(LAPACK_FFLAGS)" >> $(NETLIB_LAPACK_DIR)/make.inc
274274
-@echo "POPTS = $(LAPACK_FPFLAGS)" >> $(NETLIB_LAPACK_DIR)/make.inc
275275
-@echo "FFLAGS_NOOPT = -O0 $(LAPACK_NOOPT)" >> $(NETLIB_LAPACK_DIR)/make.inc

0 commit comments

Comments
 (0)
Please sign in to comment.