Skip to content

Conversation

ezyang
Copy link
Contributor

@ezyang ezyang commented Nov 13, 2018

Stack:
    :black_circle:  #13900 Add c10 cuda library.  💛
    :white_circle:  #13910 Move AT_CUDA_CHECK to c10  💛
    :white_circle:  #13912 Split out CUDAMultiStreamGuard from CUDAGuard  💛
    :white_circle:  #13920 Delete dependencies from CUDAStream; remove synchronize_with  💛
    :white_circle:  #14109 Move CUDAStreamInternals inside detail namespace.  💛
    :white_circle:  #14110 Move stream functions from CUDAContext to CUDAStream  💛
    :white_circle:  #14191 Add hash functions for Stream, CUDAStream; fix Device hash function  💛

Add c10 cuda library.

Right now, this is not used by anything, and only tests if the CUDA
headers are available (and not, e.g., that linking works.)

Extra changes:

  • cmake/public/cuda.cmake now is correctly include guarded, so you
    can include it multiple times without trouble.

Signed-off-by: Edward Z. Yang ezyang@fb.com

Differential Revision: D13025313

Differential Revision: D13025313
Differential Version: 63437155
Differential Revision: D13025313
Differential Version: 63455020
Differential Revision: D13025313
Differential Version: 63458489
Differential Revision: D13025313
Differential Version: 63651373
Differential Revision: D13025313
Differential Version: 63809737
Copy link
Contributor

@smessmer smessmer left a comment

Choose a reason for hiding this comment

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

I don't see any changes in TARGETS files? Shouldn't you change them as well?

impl/*.cpp
impl/*.cu
)
file(GLOB_RECURSE C10_CUDA_ALL_TEST_FILES test/*.cpp)

This comment was marked as off-topic.

This comment was marked as off-topic.

set(test_name "c10_cuda_${test_file_name}")
add_executable(${test_name} "${test_src}")
target_link_libraries(${test_name} c10_cuda gtest_main)
add_test(NAME ${test_name} COMMAND $<TARGET_FILE:${test_name}>)

This comment was marked as off-topic.

@ezyang
Copy link
Contributor Author

ezyang commented Nov 16, 2018

TARGETS are in fbcode

@ezyang
Copy link
Contributor Author

ezyang commented Nov 19, 2018

Differential Revision: D13025313
Differential Version: 63973996
Differential Revision: D13025313
Differential Version: 63974173
Differential Revision: D13025313
Differential Version: 63974553
zdevito pushed a commit to zdevito/ATen that referenced this pull request Nov 19, 2018
Summary:
Pull Request resolved: pytorch/pytorch#13900

Add c10 cuda library.

Right now, this is not used by anything, and only tests if the CUDA
headers are available (and not, e.g., that linking works.)

Extra changes:
- cmake/public/cuda.cmake now is correctly include guarded, so you
  can include it multiple times without trouble.

Signed-off-by: Edward Z. Yang <ezyang@fb.com>

Reviewed By: smessmer

Differential Revision: D13025313

fbshipit-source-id: fda85b4c35783ffb48ddd6bbb98dbd9154119d86
@ezyang ezyang added the merged label Jun 25, 2019
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