Andrew Stubbs wrote:
Maybe even adding 'requirement' on top as in:

# return 1 if the OpenMP requirement Unified Shared Memory
# is supported by at least one non-host device

This is a feature test, not a requirement at this stage, so I like it better without.

Still, the result is based on that handling that requirement. But in any case, the code after it is short and clear; thus, anyone wondering about the details just needs to read ten more lines to know the answer.

(In case of doubt: I am fine with the new wording.)

v2 patch attached.
OK for mainline?

LGTM from my side.

Thanks for the patch!

Tobias

Reply via email to