Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ROCm: Performance tests not building #1038

Closed
chipfreitag opened this issue Aug 14, 2017 · 2 comments
Closed

ROCm: Performance tests not building #1038

chipfreitag opened this issue Aug 14, 2017 · 2 comments
Assignees
Labels
Enhancement Improve existing capability; will potentially require voting

Comments

@chipfreitag
Copy link
Contributor

Fails building PerfTest_CustomReduction.cpp.

/home/chipf/kokkos/core/perf_test/PerfTest_CustomReduction.cpp:66:7: error: no
matching function for call to 'parallel_reduce'
Kokkos::parallel_reduce(Kokkos::ThreadVectorRange(team,32), [&] (c...
plus much more.
I think the constructor at :
/home/chipf/kokkos/core/src/ROCm/Kokkos_ROCm_Parallel.hpp:1133:6: note:
candidate function [with iType = int, Lambda = (lambda at
/home/chipf/kokkos/core/perf_test/PerfTest_CustomReduction.cpp:66:67),
ValueType = Kokkos::Experimental::Max<double, Kokkos::HostSpace>] not
viable: expects an l-value for 3rd argument
void parallel_reduce(const Impl::ThreadVectorRangeBoundariesStruct<iType...
should be the right one, but the compiler disagrees.

@chipfreitag chipfreitag added bug - experimental feature Compiler Issue An issue that Kokkos cannot / should not fix; Kokkos must communicate to relevant vendor labels Aug 14, 2017
@chipfreitag chipfreitag self-assigned this Aug 14, 2017
@chipfreitag chipfreitag added this to Bugs in ROCm Maturation Aug 16, 2017
@chipfreitag
Copy link
Contributor Author

Issue reported to ROCm tools team

@chipfreitag chipfreitag moved this from Bugs to In Progress in ROCm Maturation Aug 16, 2017
@chipfreitag
Copy link
Contributor Author

Further analysis shows this was not a compiler issue. The CustomReduction tests call parallel_reduce with a reducer instead of a value. That case was not implemented, but has been added in the issue branch.
The performance test case now builds and runs, but not confident of the coverage this provides.
Will be looking at unit tests to see which ones provide coverage.

@crtrott crtrott removed bug - experimental feature Compiler Issue An issue that Kokkos cannot / should not fix; Kokkos must communicate to relevant vendor labels Sep 25, 2017
@crtrott crtrott added Enhancement Improve existing capability; will potentially require voting InDevelop labels Oct 4, 2017
@crtrott crtrott closed this as completed Oct 28, 2017
@crtrott crtrott moved this from In Progress to Done in ROCm Maturation May 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improve existing capability; will potentially require voting
Projects
No open projects
Development

No branches or pull requests

2 participants