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

Test compilation with Intel 21 #1202

Closed
lifflander opened this issue Jan 6, 2021 · 6 comments · Fixed by #1203
Closed

Test compilation with Intel 21 #1202

lifflander opened this issue Jan 6, 2021 · 6 comments · Fixed by #1203
Assignees

Comments

@lifflander
Copy link
Collaborator

Describe the bug
I have an image set up. Here's the error I get near the end:

[352/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_construct_no_idx.extended.cc.o
FAILED: tests/CMakeFiles/collection_extended.dir/unit/collection/test_construct_no_idx.extended.cc.o
/usr/bin/ccache /opt/intel/oneapi/mpi/2021.1.1/bin/mpicxx -DFMT_HEADER_ONLY=1 -DFMT_USE_USER_DEFINED_LITERALS=0 -DHAS_DETECTION_COMPONENT=1 -I/vt/tests/unit -I/vt/lib/fmt -I/vt/lib/CLI -I/vt/lib/libfort/lib -Irelease -I/vt/src -Ilib/checkpoint/src -I/vt/lib/checkpoint/src -I/vt/lib/detector/src -isystem /vt/tests/extern/googletest/googletest/include -isystem /vt/tests/extern/googletest/googletest -Wall -pedantic -Wshadow -Wno-unknown-pragmas -Wsign-compare -O3 -DNDEBUG -fPIC -qopenmp -std=c++14 -MD -MT tests/CMakeFiles/collection_extended.dir/unit/collection/test_construct_no_idx.extended.cc.o -MF tests/CMakeFiles/collection_extended.dir/unit/collection/test_construct_no_idx.extended.cc.o.d -o tests/CMakeFiles/collection_extended.dir/unit/collection/test_construct_no_idx.extended.cc.o -c /vt/tests/unit/collection/test_construct_no_idx.extended.cc
icpc: error #10106: Fatal error in /opt/intel/oneapi/compiler/2021.1.1/linux/bin/intel64/../../bin/intel64/mcpcom, terminated by kill signal
compilation aborted for /vt/tests/unit/collection/test_construct_no_idx.extended.cc (code 1)
[353/511] Building CXX object tests/CMakeFiles/runtime_basic.dir/unit/runtime/test_memory_footprint.cc.o
[354/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_insert.extended.cc.o
[355/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_lb_lite.extended.cc.o
/vt/tests/unit/collection/test_lb_lite.extended.cc(109): warning #3868: declaration hides member "vt::tests::unit::lb_lite::LBTest::val2" (declared at line 95) in same scope
    double val2 = 0.4f;
           ^

[356/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_lb.extended.cc.o
[357/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_model_per_collection.extended.cc.o
[358/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_collection_group.extended.cc.o
[359/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_checkpoint.extended.cc.o
[360/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_index_types.extended.cc.o
[361/511] Building CXX object tests/CMakeFiles/collection_extended.dir/unit/collection/test_broadcast.extended.cc.o
ninja: build stopped: subcommand failed.

real    18m21.599s
user    117m48.416s
sys     8m43.522s
The command '/bin/sh -c /vt/ci/build_cpp.sh /vt /build' returned a non-zero code: 1
@lifflander lifflander self-assigned this Jan 6, 2021
@lifflander
Copy link
Collaborator Author

@PhilMiller

@PhilMiller
Copy link
Member

This is likely just the container not running with enough memory to get through the compilation. Run with less parallelism, and give the container more breathing room.

lifflander added a commit that referenced this issue Jan 6, 2021

Verified

This commit was signed with the committer’s verified signature.
nmm0 Nicolas Morales
@lifflander
Copy link
Collaborator Author

I also get some warnings, which probably don't matter too much:

icc: command line warning #10148: option '-Wdouble-promotion' not supported
icc: command line warning #10148: option '-Wcast-align' not supported
icc: command line warning #10148: option '-W=strict-overflow=5' not supported
icc: command line warning #10148: option '-Wpedantic' not supported
remark #11074: Inlining inhibited by limit max-size
remark #11074: Inlining inhibited by limit max-total-size
remark #11076: To get full report use -qopt-report=4 -qopt-report-phase ipo

@lifflander
Copy link
Collaborator Author

Building sequentially it works. Needs the option --shm-size=4gb for mpiexec to work properly in the container.

One test is failing:

The following tests FAILED:
        150 - vt:TestCollectionInvoke.test_collection_invoke_1_proc_8 (Failed)

/opt/intel/oneapi/mpi/2021.1.1/bin/mpiexec "-n" "8" "./collection_basic" "--gtest_filter=TestCollectionInvoke.test_collection_invoke_1"

lifflander added a commit that referenced this issue Jan 7, 2021

Verified

This commit was signed with the committer’s verified signature.
nmm0 Nicolas Morales
lifflander added a commit that referenced this issue Jan 7, 2021

Verified

This commit was signed with the committer’s verified signature.
nmm0 Nicolas Morales
cz4rs added a commit that referenced this issue Jan 8, 2021

Verified

This commit was signed with the committer’s verified signature.
nmm0 Nicolas Morales
@PhilMiller PhilMiller changed the title Fix compile bug with Intel 21 Test compilation with Intel 21 Jan 12, 2021
@PhilMiller
Copy link
Member

Looks like Kokkos has builds targeting "github-Linux / CI (fedora:intel-oneapi, icpx," - it may be possible to use their configuration, or even see if the images they use are published.

lifflander added a commit that referenced this issue Jun 28, 2021

Verified

This commit was signed with the committer’s verified signature.
nmm0 Nicolas Morales
@cz4rs cz4rs self-assigned this Jun 29, 2021
cz4rs added a commit that referenced this issue Jul 5, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
cz4rs added a commit that referenced this issue Jul 6, 2021
cz4rs added a commit that referenced this issue Jul 6, 2021
cz4rs added a commit that referenced this issue Jul 6, 2021
@cz4rs
Copy link
Contributor

cz4rs commented Jul 6, 2021

cz4rs added a commit that referenced this issue Nov 3, 2021
cz4rs added a commit that referenced this issue Nov 3, 2021
- use icx / icpx for compilation
- revert to using environmental variables directly (skip setvars.sh)
- build mpich from sources
cz4rs added a commit that referenced this issue Nov 3, 2021
cz4rs added a commit that referenced this issue Nov 4, 2021
cz4rs added a commit that referenced this issue Nov 4, 2021
- use icx / icpx for compilation
- revert to using environmental variables directly (skip setvars.sh)
- build mpich from sources
cz4rs added a commit that referenced this issue Nov 4, 2021
cz4rs added a commit that referenced this issue Nov 4, 2021
cz4rs added a commit that referenced this issue Nov 4, 2021
cz4rs added a commit that referenced this issue Nov 5, 2021
cz4rs added a commit that referenced this issue Nov 5, 2021
- use icx / icpx for compilation
- revert to using environmental variables directly (skip setvars.sh)
- build mpich from sources
cz4rs added a commit that referenced this issue Nov 5, 2021
cz4rs added a commit that referenced this issue Nov 5, 2021
cz4rs added a commit that referenced this issue Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants