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

add support for 'mt_blas_lapack' toolchain option (WIP) #2356

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

boegel
Copy link
Member

@boegel boegel commented Nov 29, 2017

see discussion in #2350

@boegel boegel added this to the 3.5.0 milestone Nov 29, 2017
@boegel boegel changed the title add support for 'mt_blas_lapack' toolchain option add support for 'mt_blas_lapack' toolchain option (WIP) Nov 29, 2017
@boegel boegel modified the milestones: 3.5.0, next release Dec 6, 2017
@boegel boegel modified the milestones: 3.5.1, 3.6.0 Jan 12, 2018
@boegel boegel modified the milestones: 3.5.2, 3.6.0 Feb 22, 2018
@boegel boegel modified the milestones: 3.6.0, 3.x Mar 16, 2018
libscalack_ictce3 += " -lmkl_intel_lp64 -lmkl_sequential -lmkl_core"
libscalapack_ictce3_tpl = "-lmkl_scalapack_lp64 %s -lmkl_blacs_intelmpi_lp64 -lmkl_intel_lp64 %s -lmkl_core"
libscalapack_ictce3 = libscalapack_ictce3_tpl % ('-lmkl_solver_lp64_sequential', '-lmkl_sequential')
libscalapack_mt_ictce3 = libscalapack_ictce3_tpl % ('-lmkl_solver_lp64', '-lmkl_intel_thread')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it doesn't matter but the iomp library is missing

libscalack_ictce4 = "-lmkl_scalapack_lp64 -lmkl_blacs_intelmpi_lp64 -lmkl_intel_lp64 -lmkl_sequential -lmkl_core"
libscalapack_ictce4_tpl = "-lmkl_scalapack_lp64 -lmkl_blacs_intelmpi_lp64 -lmkl_intel_lp64 %s -lmkl_core"
libscalapack_ictce4 = libscalapack_ictce4_tpl % '-lmkl_sequential'
libscalapack_mt_ictce4 = libscalapack_ictce4_tpl % '-lmkl_intel_thread'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here too

@akesandgren
Copy link
Contributor

This looks ok in its current form. Should we merge it or try to make it take a list form first?

@akesandgren
Copy link
Contributor

@boegel Care to fix the conflict?

@boegel boegel modified the milestones: 3.x, 4.x Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants