Generalized use of property list

[steffenlarsen]

Adds optional property_list parameters to all user-constructible SYCL
object constructors that did not have it before.

This is the wording related this proposal which has been discussed internally.

[keryell]

This is a great idea.
Is this something you need for 1.2.1 or can it wait fot the next SYCL version?

[keryell]

It looks like a good proposal.

[fraggamuffin]

You introduce an exception; current properties are specific; if we dont want it to be in this one, then I will have to change it; with SYCL NEXT constructing object with introp type is to use make function

I still want to remove this exception. I would have to change the wording because this is a restriction they have as well. Could be a separate issue. OK to be separate issue.

[keryell]

Some merge crumbles?

[CLAassistant]

All committers have signed the CLA.

[illuhad]

I wonder if we should move to a more type-based system for properties, such that passing invalid properties for one class will be caught by the type system. I expect that, if everything accepts properties, it opens the door for a lot of user error, and requires a lot of error handling..

[mkinsner]

In addition to #78 there are additional explicit constructors required before this change goes in, right? e.g. context?

[AerialMantis]

I have pushed another PR #81, which introduces a clarification to property_list constructors which removes the ambiguity in SYCL class constructors, allowing us to introduce this change without the worry of creating further ambiguities.

[fraggamuffin]

mar 19:

• ready to go, PR81 uses new trait to restrict property list construction to types which are registered as officla - vendor or standard to not trigger an ambiguity
• extend and clarify, hence 121
• async handler resolved by PR 81; this is an important change

[steffenlarsen]

I noticed that there were some changes missing to program, specifically:

• The constructor with linkOption should be split in two, similar to queue and context with async_handler.
• The documentation of the constructors for program didn't have the new property_list parameter.

This has been fixed with 69b362c.