Project

General

Profile

Feature #430

Use -fopenmp flag when compiling CoCoALib with libnormaliz

Added by Christof Soeger about 10 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Portability
Start date:
29 Jan 2014
Due date:
% Done:

100%

Estimated time:
1.51 h
Spent time:

Description

Right now libnormaliz can be included to CoCoALib only without using OpenMP. We did it this way to have an easier start.

But I think it would be no problem to add the -fopenmp flag. It is recognized by many compilers (gcc, icc, ...) and should not give a disadvantage if it is not used.

To support the case where the flag is not supported there are two possibilities:
  • Detect whether the compiler supports it by compiling a test programm.
  • Detect whether libnormaliz was compiled with OpenMP. A possible test might be something like nm -g | grep " omp_".

Related issues

Related to CoCoA - Support #425: Osnabrueck 2014-01Closed2014-01-27

History

#1 Updated by John Abbott about 10 years ago

  • Status changed from New to In Progress
  • Assignee set to John Abbott
  • % Done changed from 0 to 30

JAA has written a new script, and added a suitable call to configure.
Christof confirms that it works for him, except for the GUI :-(

#2 Updated by John Abbott about 10 years ago

  • Status changed from In Progress to Feedback
  • % Done changed from 30 to 90

Resolved the outstanding problem -- the flag had to passed correctly to qmake.
JAA has fixed the make-c5makefile.sh script; Christof reports that it works OK now.

#3 Updated by Christof Soeger about 10 years ago

without --with-libnormaliz it still tries to check and get

nm: '': No such file

#4 Updated by Christof Soeger about 10 years ago

For me it now works fine. With and without libnormaliz.

#5 Updated by Anna Maria Bigatti almost 10 years ago

  • Target version set to CoCoALib-0.99533 Easter14

#6 Updated by John Abbott almost 10 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 90 to 100

No problems reported in over 2 months. Closing!

#7 Updated by Anna Maria Bigatti almost 10 years ago

  • Estimated time set to 1.51 h

Also available in: Atom PDF