search

celerity / celerity-runtime

High-level C++ for Accelerator Clusters
https://celerity.github.io
MIT License
139 stars 18 forks source link

Stabilize experimental::fence #225

Closed fknorr closed 9 months ago

fknorr commented 10 months ago

celerity::experimental::fence and celerity::experimental::buffer_snapshot are now celerity::distr_queue::fence (a member function) and celerity::buffer_snapshot. The experimental declarations are aliases of the new names, and a deprecation test ensures they still work.

I have decided to keep the fence header since the definitions of buffer_snapshot and *_fence_promise don't fit into what I believe should be distr_queue.h.

Closes celerity/meta#69.

github-actions[bot] commented 10 months ago

Check-perf-impact results: (b003273516680ef3e6ca0110b3678f5e)

:question: No new benchmark data submitted. :question:
Please re-run the microbenchmarks and include the results if your commit could potentially affect performance.