Skip to content

Relax backend for default-constructed event - #582

Open
gmlueck wants to merge 1 commit into
KhronosGroup:mainfrom
gmlueck:gmlueck/default-event-backend
Open

Relax backend for default-constructed event#582
gmlueck wants to merge 1 commit into
KhronosGroup:mainfrom
gmlueck:gmlueck/default-event-backend

Conversation

@gmlueck

@gmlueck gmlueck commented Jul 12, 2024

Copy link
Copy Markdown
Contributor

Relax the requirements on the backend for a default-constructed event object. Previously, we required the backend for such an event to be the same as the backend for the default device. After some implementation experience, we decided that this is not practical. Requiring interoperation with a particular backend constrains the implementation too much. It also seems arbitrary to require the default-constructed event to have a particular backend. To address these concerns, we now say that the backend that is associated with a default-constructed event is implementation defined.

Relax the requirements on the backend for a default-constructed `event`
object.  Previously, we required the backend for such an event to be the
same as the backend for the default device.  After some implementation
experience, we decided that this is not practical.  Requiring
interoperation with a particular backend constrains the implementation
too much.  It also seems arbitrary to require the default-constructed
event to have a particular backend.  To address these concerns, we now
say that the backend that is associated with a default-constructed
event is implementation defined.

@illuhad illuhad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tomdeakin

Copy link
Copy Markdown
Contributor

Discussed previously and revisit.

@TApplencourt

Copy link
Copy Markdown
Contributor

( Maybe related to new change for free-function? Need update)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants