Summary: | [Mesa-current] [clover] build failure in clvover/api/transfer.cpp | ||
---|---|---|---|
Product: | Mesa | Reporter: | Dmitry Cherkassov <dcherkassov> |
Component: | Other | Assignee: | mesa-dev |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | ||
Version: | git | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: | clover_simplify_is_zero.patch |
Description
Dmitry Cherkassov
2012-10-18 14:19:08 UTC
that dirty patch makes it at least compile diff --git a/src/gallium/state_trackers/clover/api/transfer.cpp b/src/gallium/state_trackers/clover/api/transfer.cpp index 8ea8f86..10b454b 100644 --- a/src/gallium/state_trackers/clover/api/transfer.cpp +++ b/src/gallium/state_trackers/clover/api/transfer.cpp @@ -39,8 +39,7 @@ namespace { if (!q) throw error(CL_INVALID_COMMAND_QUEUE); - if (bool(num_deps) != bool(deps) || - any_of(is_zero<cl_event>(), deps, deps + num_deps)) + if (bool(num_deps) != bool(deps)) throw error(CL_INVALID_EVENT_WAIT_LIST); if (any_of([&](const cl_event ev) { Created attachment 68790 [details] [review] clover_simplify_is_zero.patch (In reply to comment #0) > ../../../../src/gallium/state_trackers/clover/.libs/libclover.a(lt8- > libclover_la-kernel.o): In function `operator()': > /home/dc/src/mesa-cur/src/gallium/state_trackers/clover/./core/base.hpp:280: > multiple definition of `std::function<bool (_cl_event* const&)> > clover::is_zero<_cl_event*>()::{lambda(_cl_event* > const&)#1}::_FUN(_cl_event* const&)' > ../../../../src/gallium/state_trackers/clover/.libs/libclover.a(libclover_la- > transfer.o):/home/dc/src/mesa-cur/src/gallium/state_trackers/clover/./core/ > base.hpp:280: first defined here > ../../../../src/gallium/state_trackers/clover/.libs/libclover.a(lt8- > libclover_la-kernel.o): In function `std::function<bool (_cl_event* const&)> > clover::is_zero<_cl_event*>()::{lambda(_cl_event* const&)#1}::operator bool > (*)(_cl_event* const&)() const': > /home/dc/src/mesa-cur/src/gallium/state_trackers/clover/./core/base.hpp:278: > multiple definition of `std::function<bool (_cl_event* const&)> > clover::is_zero<_cl_event*>()::{lambda(_cl_event* const&)#1}::operator bool > (*)(_cl_event* const&)() const' > ../../../../src/gallium/state_trackers/clover/.libs/libclover.a(libclover_la- > transfer.o):/home/dc/src/mesa-cur/src/gallium/state_trackers/clover/./core/ > base.hpp:278: first defined here This looks like a compiler bug. Maybe [1]? In any case the attached patch should work around it, can you try it out please? [1] http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821 it compiles OK with your patch. thank you. will it be commited? (In reply to comment #3) > it compiles OK with your patch. thank you. > will it be commited? OK, pushed. Marking as fixed... |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.