Refactor event_t similarly to the recent changes to stream_t
#8
Assignees
Labels
Comments
eyalroz
changed the title
event_t more like stream_tevent_t similarly to the recent changes to stream_t
eyalroz
pushed a commit
that referenced
this issue
Nov 18, 2016
`event_t`: * No longer caching the event flags in the wrapper; CUDA doesn't offer it - we don't offer it * Made `event_t` only constructible when an event id already exists - which allows the construction to have the event id as a constant field * Am caching the index of the device with which the stream is associated * Added `make()` and `wrap()` methods a-la-`stream_t` * Now destructing the `other` object on move-construction of an `event_`
|
Why aren't events closed by commit comments? Hrumph. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We've made
stream_tonly constructible with a stream id, and having a const id and device id - withstream::make()functions to call the creation code (which is outside of the class). Let's do the same forevent_t(as well as keep their device IDs explicitly).The text was updated successfully, but these errors were encountered: