Skip to content

ARROW-1387: [C++] Set up GPU leaf library, add unit test module for CUDA tests - #982

Closed
wesm wants to merge 2 commits into
apache:masterfrom
wesm:arrow-gpu-lib
Closed

ARROW-1387: [C++] Set up GPU leaf library, add unit test module for CUDA tests#982
wesm wants to merge 2 commits into
apache:masterfrom
wesm:arrow-gpu-lib

Conversation

@wesm

@wesmwesm commented Aug 20, 2017

Copy link
Copy Markdown
Member

This is an optional leaf library for users who want to use Arrow data on graphics cards. See parent JIRA ARROW-1055 for a roadmap for some basic GPU extensions

@wesm

wesm commented Aug 20, 2017

Copy link
Copy Markdown
MemberAuthor

cc @sklam

wesm added 2 commits August 20, 2017 19:53
…vice
Change-Id: I40e6b5d77f895d7803dfad64dd9fc2d9f882c0f3
Change-Id: Ib07611064711a320527325224e8005e7142f062e
@wesm

wesm commented Aug 21, 2017

Copy link
Copy Markdown
MemberAuthor

+1

@wesm
wesm deleted the arrow-gpu-lib branch August 22, 2017 01:24
typename std::conditional<std::is_base_of<DataType, T>::value, T, \
typename detail::GetAttr_TypeClass<T>::type>::type; \
using c_type = typename detail::GetAttr_c_type<TypeClass>::type;
using c_type = typename detail::GetAttr_c_type<TypeClass>::type

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Accidential deletion of the semicolon?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

This was on purpose because this macro is used like a function below (PRIMITIVE_TRAITS(...);). When this file gets touched by nvcc it caused clang to spew a bunch of warnings

Sign up for freeto 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

@wesm@xhochy