Skip to content

src/cunumeric/matrix: stop including coll.h in solve_template.inl (#620)#637

Merged
magnatelee merged 1 commit intonv-legate:branch-22.12from
magnatelee:no-solve-coll-backport
Sep 30, 2022
Merged

src/cunumeric/matrix: stop including coll.h in solve_template.inl (#620)#637
magnatelee merged 1 commit intonv-legate:branch-22.12from
magnatelee:no-solve-coll-backport

Conversation

@magnatelee
Copy link
Contributor

@magnatelee magnatelee commented Sep 30, 2022

This file was included unnecessarily, and led to build issues on distributed machines. In particular, including coll.h pulls in mpi.h, which is an unresolved header to NVCC.

Signed-off-by: Rohan Yadav rohany@alumni.cmu.edu

…-legate#620)

This file was included unnecessarily, and led to build issues on
distributed machines. In particular, including coll.h pulls in mpi.h,
which is an unresolved header to NVCC.

Signed-off-by: Rohan Yadav <rohany@alumni.cmu.edu>

Signed-off-by: Rohan Yadav <rohany@alumni.cmu.edu>
@magnatelee magnatelee added the category:task PR is a simple task and will not be included in release notes label Sep 30, 2022
@magnatelee magnatelee merged commit 298f094 into nv-legate:branch-22.12 Sep 30, 2022
@magnatelee magnatelee deleted the no-solve-coll-backport branch September 30, 2022 18:54
manopapad pushed a commit that referenced this pull request Mar 5, 2025
This adds in the missing CUDA toolkit libraries and the rpath entries for them relative to our wheel.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:task PR is a simple task and will not be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants