Skip to content

Commit 8a6af9f

Browse files
committed
compiler: Refactor mpi distro check
1 parent efe26d4 commit 8a6af9f

File tree

1 file changed

+3
-10
lines changed

1 file changed

+3
-10
lines changed

devito/arch/compiler.py

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -683,9 +683,9 @@ class IntelCompiler(Compiler):
683683
def __init__(self, *args, **kwargs):
684684
super().__init__(*args, **kwargs)
685685

686-
self.cflags.append("-xHost")
687686
platform = kwargs.pop('platform', configuration['platform'])
688687
language = kwargs.pop('language', configuration['language'])
688+
self.cflags.append("-xHost")
689689

690690
if configuration['safe-math']:
691691
self.cflags.append("-fp-model=strict")
@@ -703,8 +703,8 @@ def __init__(self, *args, **kwargs):
703703
if kwargs.get('mpi'):
704704
mpi_distro = sniff_mpi_distro('mpiexec')
705705
if mpi_distro != 'IntelMPI':
706-
warning("The MPI compiler `%s` doesn't use the Intel "
707-
"C/C++ compiler underneath" % self.MPICC)
706+
warning("Expected Intel MPI distribution with `%s`, but found `%s`"
707+
% (self.__class__.__name__, mpi_distro))
708708

709709
def __lookup_cmds__(self):
710710
self.CC = 'icc'
@@ -768,13 +768,6 @@ def __init__(self, *args, **kwargs):
768768
self.cflags.append('-gline-tables-only')
769769
self.cflags.append('-fdebug-info-for-profiling')
770770

771-
# Make sure the MPI compiler uses `icx` underneath -- whatever the MPI distro is
772-
if kwargs.get('mpi'):
773-
mpi_distro = sniff_mpi_distro('mpiexec')
774-
if mpi_distro != 'IntelMPI':
775-
warning("The MPI compiler `%s` doesn't use the Intel(R) oneAPI "
776-
"`%s` compiler underneath" % self.MPICC, self.CXX)
777-
778771
def __lookup_cmds__(self):
779772
# OneAPI HPC ToolKit comes with icpx, which is clang++,
780773
# and icx, which is clang

0 commit comments

Comments
 (0)