[PATCH] D73237: [CUDA] Fix order of memcpy arguments in __shfl_*(<64-bit type>).

2020-01-24 Thread Artem Belevich via Phabricator via cfe-commits
tra added a comment. In D73237#1839216 , @hans wrote: > In D73237#1837077 , @tra wrote: > > > Landed in > > https://github.com/llvm/llvm-project/commit/cc14de88da27a8178976972bdc8211c31f7ca9ae > > @hans -- can we

[PATCH] D73237: [CUDA] Fix order of memcpy arguments in __shfl_*(<64-bit type>).

2020-01-24 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. In D73237#1837077 , @tra wrote: > Landed in > https://github.com/llvm/llvm-project/commit/cc14de88da27a8178976972bdc8211c31f7ca9ae > @hans -- can we cherry-pick it into 10? Yes, go ahead and "git cherry-pick -x" it and push to the

[PATCH] D73237: [CUDA] Fix order of memcpy arguments in __shfl_*(<64-bit type>).

2020-01-23 Thread Artem Belevich via Phabricator via cfe-commits
tra added a comment. In D73237#1836978 , @timshen wrote: > What's the test situation for these headers? They are not testable in clang/llvm tree as they need CUDA SDK headers. I should be able to add some tests in the test-suite. Repository: rG LLVM

[PATCH] D73237: [CUDA] Fix order of memcpy arguments in __shfl_*(<64-bit type>).

2020-01-23 Thread Artem Belevich via Phabricator via cfe-commits
tra closed this revision. tra added a subscriber: hans. tra added a comment. Landed in https://github.com/llvm/llvm-project/commit/cc14de88da27a8178976972bdc8211c31f7ca9ae @hans -- can we cherry-pick it into 10? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.

[PATCH] D73237: [CUDA] Fix order of memcpy arguments in __shfl_*(<64-bit type>).

2020-01-23 Thread Tim Shen via Phabricator via cfe-commits
timshen accepted this revision. timshen added a comment. This revision is now accepted and ready to land. What's the test situation for these headers? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73237/new/ https://reviews.llvm.org/D73237 _

[PATCH] D73237: [CUDA] Fix order of memcpy arguments in __shfl_*(<64-bit type>).

2020-01-22 Thread Artem Belevich via Phabricator via cfe-commits
tra created this revision. tra added a reviewer: jlebar. Herald added subscribers: sanjoy.google, bixia. Herald added a project: clang. Wrong argument order resulted in broken shfl ops for 64-bit types. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73237 Files: clang/lib/He