[Octave-bug-tracker] [bug #44346] configure summary shows OpenMP enabled even if not detected

[Rik]

Follow-up Comment #6, bug #44346 (project octave):

I don't think it's a problem to append to the user's CFLAGS/CXXFLAGS.  Because
the operation is append, the user gets to decide what the base flags are at
all times either through an environment variable or passing it into the make
invocation.

The pthreads case was special as noted in the comments in configure.ac.


AX_PTHREAD
## Include pthread libs and flags early in case other tests need them.
## They seem to be required for the OpenGL tests on Debian systems.
LIBS="$PTHREAD_LIBS $LIBS"
CFLAGS="$CFLAGS $PTHREAD_CFLAGS"
CXXFLAGS="$CXXFLAGS $PTHREAD_CFLAGS"


So configure itself and the conftest.c files that it produces when testing for
various compiler features needed the flag so sticking it in AM_CFLAGS would
not have been enough.

I don't think OpenMP has those kinds of affects so you could put it in
AM_CFLAGS or keep the current situation.

As for Fortran, most of those libraries were written decades ago and I don't
see people going back to re-engineer support for OpenMP.  So I think it would
be pretty useless to enable openmp for those files.



    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?44346>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/