Closed krasznaa closed 11 months ago
Only noticed with a delay that the constructors of vecmem::tuple
should be simplified a bit. Just as for vecmem::data::vector_view
and friends, if we want to use such types directly as parameters of a SYCL kernel, they have to have "proper default constructors".
And this change seemingly also brought a build failure forward. :thinking: Interesting that Clang (which is what I tested this change with) didn't complain about that variable. :thinking:
This is step 1 in getting #246 into the repository.
Introduced
vecmem::tuple
,vecmem::details::disjunction
andvecmem::details::negation
. All in preparation for the SoA EDM base classes.vecmem::details::disjunction
is a stand-in for std::disjunction with C++14. (Which is still our minimum requirement for device code!)vecmem::details::negation
is a stand-in for std::negation in device code.vecmem::tuple
is a shameless copy of detray::tuple. With very slight modifications for C++14 compatibility.Not sure yet how I'll make the rest of #246 digestible, but for now let's just get this reviewed. Then we'll see about the rest. :wink: