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.

Sign up for GitHub

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

Jump to bottom

Update NVHPC version from 23.7 to 23.9. #3736

Merged
merged 2 commits into from
Nov 2, 2023
Merged

Update NVHPC version from 23.7 to 23.9. #3736

merged 2 commits into from
Nov 2, 2023

Conversation

hyoklee
Copy link
Member

@hyoklee hyoklee commented Oct 20, 2023

New version of NVHPC is available for CI.

@hyoklee
Copy link
Member Author

hyoklee commented Oct 20, 2023 •
edited
Loading

Interesting error w/ cmake:

/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/compilers/bin/nvc -DH5_BUILT_AS_DYNAMIC_LIB -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -D_LARGEFILE64_SOURCE -D_LARGEFILE_SOURCE -D_POSIX_C_SOURCE=200809L -I/home/runner/work/hdf5/build/_deps/hdf5_zlib-build -I/home/runner/work/hdf5/build/_deps/hdf5_zlib-src -I/home/runner/work/hdf5/hdf5/src -I/home/runner/work/hdf5/hdf5/src/H5FDsubfiling -I/home/runner/work/hdf5/build/src -I/home/runner/work/hdf5/build/test -I/home/runner/work/hdf5/hdf5/test -isystem /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/comm_libs/12.2/openmpi4/openmpi-4.1.5/include -c99  -Minform=warn -s -fast -O3 -DNDEBUG -c99 -fPIE -MD -MT test/CMakeFiles/testhdf5.dir/tselect.c.o -MF test/CMakeFiles/testhdf5.dir/tselect.c.o.d -o test/CMakeFiles/testhdf5.dir/tselect.c.o -c /home/runner/work/hdf5/hdf5/test/tselect.c
/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/compilers/share/llvm/bin/opt: /tmp/nvcNq-c1J8vodv9.ll:12484:22: error: use of undefined value '%L.B10147'
        br i1  %5871, label %L.B10147, label %L.B10227, !llvm.loop !780, !dbg !1035
                            ^

@jhendersonHDF
Copy link
Collaborator

Interesting error:

/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/compilers/bin/nvc -DH5_BUILT_AS_DYNAMIC_LIB -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -D_LARGEFILE64_SOURCE -D_LARGEFILE_SOURCE -D_POSIX_C_SOURCE=200809L -I/home/runner/work/hdf5/build/_deps/hdf5_zlib-build -I/home/runner/work/hdf5/build/_deps/hdf5_zlib-src -I/home/runner/work/hdf5/hdf5/src -I/home/runner/work/hdf5/hdf5/src/H5FDsubfiling -I/home/runner/work/hdf5/build/src -I/home/runner/work/hdf5/build/test -I/home/runner/work/hdf5/hdf5/test -isystem /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/comm_libs/12.2/openmpi4/openmpi-4.1.5/include -c99  -Minform=warn -s -fast -O3 -DNDEBUG -c99 -fPIE -MD -MT test/CMakeFiles/testhdf5.dir/tselect.c.o -MF test/CMakeFiles/testhdf5.dir/tselect.c.o.d -o test/CMakeFiles/testhdf5.dir/tselect.c.o -c /home/runner/work/hdf5/hdf5/test/tselect.c
/opt/nvidia/hpc_sdk/Linux_x86_64/23.9/compilers/share/llvm/bin/opt: /tmp/nvcNq-c1J8vodv9.ll:12484:22: error: use of undefined value '%L.B10147'
        br i1  %5871, label %L.B10147, label %L.B10227, !llvm.loop !780, !dbg !1035
                            ^

I seemed to need to add the -Mnovect flag to fix this since the compiler seems to have issues

@mattjala mattjala added Merge - To 1.14 Priority - 3. Low 🔽 Code cleanup, small feature change requests, etc. Component - Testing Code in test or testpar directories, GitHub workflows Type - Improvement Improvements that don't add a new feature or functionality labels Oct 24, 2023
@lrknox lrknox merged commit 94a9926 into HDFGroup:develop Nov 2, 2023
40 checks passed
@hyoklee hyoklee deleted the hyoklee-hdf5-105 branch November 3, 2023 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bmribler bmribler bmribler approved these changes

@jhendersonHDF jhendersonHDF jhendersonHDF approved these changes

@lrknox lrknox lrknox approved these changes

@derobins derobins derobins approved these changes

@byrnHDF byrnHDF Awaiting requested review from byrnHDF byrnHDF is a code owner

@fortnern fortnern Awaiting requested review from fortnern fortnern is a code owner

@qkoziol qkoziol Awaiting requested review from qkoziol qkoziol is a code owner

@vchoi-hdfgroup vchoi-hdfgroup Awaiting requested review from vchoi-hdfgroup vchoi-hdfgroup is a code owner

@glennsong09 glennsong09 Awaiting requested review from glennsong09 glennsong09 is a code owner

@mattjala mattjala Awaiting requested review from mattjala mattjala is a code owner

@brtnfld brtnfld Awaiting requested review from brtnfld brtnfld is a code owner

Assignees
No one assigned
Labels
Component - Testing Code in test or testpar directories, GitHub workflows Priority - 3. Low 🔽 Code cleanup, small feature change requests, etc. Type - Improvement Improvements that don't add a new feature or functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@hyoklee @jhendersonHDF @derobins @lrknox @bmribler @mattjala