Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove need for #include <omp.h> #14

Open
cpinter opened this issue Sep 15, 2017 · 1 comment
Open

Remove need for #include <omp.h> #14

cpinter opened this issue Sep 15, 2017 · 1 comment
Assignees
Milestone

Comments

@cpinter
Copy link
Member

cpinter commented Sep 15, 2017

It should not be necessary to include <omp.h> in the ContourComparison module.
This ticket will track the effort to remove this.

Migrated from https://app.assembla.com/spaces/slicerrt/tickets/227-remove-need-for--include--lt-omp-h-gt-/details

@cpinter
Copy link
Member Author

cpinter commented Sep 15, 2017

2013-02-19 20:50
Greg Sharp
I did some testing on this, using MSVC 2010 compiler.

(1) SlicerRT builds plastimatch as shared lib

(2) plmutil.lib correctly includes dependency on VCOMP.LIB, as evidenced by:
dumpbin /all plmutil.lib
Which shows
/DEFAULTLIB:"VCOMP"

(3) A test executable which does not include <omp.h> and does not use "/openmp" option was created and linked against plmutil.h.
cl /MD main.cxx plmutil.lib plmbase.lib plmsys.lib
This executable was found to correctly depend on VCOMP100.DLL

@cpinter cpinter added this to the Future milestone Sep 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants