Skip to content

Merging with current QUDA develop#29

Open
pittlerf wants to merge 29 commits into
masterfrom
merge_quda_develop
Open

Merging with current QUDA develop#29
pittlerf wants to merge 29 commits into
masterfrom
merge_quda_develop

Conversation

@pittlerf

@pittlerf pittlerf commented Oct 5, 2022

Copy link
Copy Markdown
Contributor

Now I see errors only coming from not declaring comm_rank() comm_abort() and function like these, so I hope I finish this today

@pittlerf

Copy link
Copy Markdown
Contributor Author

Now I see errors only coming from not declaring comm_rank() comm_abort() and function like these, so I hope I finish this today

This is compiling now with the with quda develop e2415ef161f06f439d5aa46bd1385ad14ea65105 (28.09.2022)

@sbacchio sbacchio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can be merged after merging Jacopo's

Comment thread CMakeLists.txt
set(CMAKE_CXX_FLAGS_DEVICEDEBUG "${OpenMP_CXX_FLAGS} -std=c++17 -Wall -Wno-unknown-pragmas ${CLANG_FORCE_COLOR}" CACHE STRING
"Flags used by the C++ compiler during device-debug builds.")
set(CMAKE_CXX_FLAGS_DEBUG "${OpenMP_CXX_FLAGS} -Wall -Wno-unknown-pragmas -g -fno-inline -DHOST_DEBUG ${CLANG_FORCE_COLOR}" CACHE STRING
set(CMAKE_CXX_FLAGS_DEBUG "${OpenMP_CXX_FLAGS} -std=c++17 -Wall -Wno-unknown-pragmas -g -fno-inline -DHOST_DEBUG ${CLANG_FORCE_COLOR}" CACHE STRING

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

isn't enough the change in CUDA_NVCC_FLAGS?

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

Successfully merging this pull request may close these issues.

2 participants