search

acts-project / vecmem

Vectorised data model base and helper classes.
https://acts-project.github.io/vecmem/
Mozilla Public License 2.0
19 stars 13 forks source link

Multi-Alloc Fix, main branch (2023.10.31.) #245

Closed krasznaa closed 1 year ago

krasznaa commented 1 year ago

Made the vecmem::details::aligned_multiple_placement function work with other than 2 arguments. As it turns out, so far no client needed to use a different number of allocations through this function. :confused: But in my tests for an SoA based EDM for traccc (more on that once I have something usable to show...), I managed to run into this. :thinking:

Note that how much of the code style in vecmem::details::max comes from CoPilot, we may need to include authorship credits for it at one point. :stuck_out_tongue:

stephenswat commented 1 year ago

Also, either way - well spotted!

krasznaa commented 1 year ago

Huhh... Clearly I was google-ing badly. :confused: Since of course I was looking for a standard algorithm/utility for this at first.

Either way, let's just go with this then. I'll anyway have plenty of more variadic templates with the EDM developments...