Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Invoke result extended lambdas support #284

Merged

Conversation

gevtushenko
Copy link
Collaborator

This PR addresses the following issue. The implementation of result_of and invoke_result was complimented with a static assert omitted in device code or compilers different than nvcc. In order not to force users to rewrite device lambdas as function objects, cuda::proclaim_return_type was added. Since the main use case for cuda::proclaim_return_type is device lambdas (which type is unknown in any way) I decided to hide the actual wrapper type in detailed namespace.

Questions:

  1. docs/serve failed to start locally without adding gem "webrick" dependency. I'm not sure if this has to be merged.
  2. I've just learned about lit and I'm not sure if there's a way to check that the fail test emits text in static asserts. Please advise me on this matter if there's a portable way to do so.

@jrhemstad
Copy link
Collaborator

2. I've just learned about lit and I'm not sure if there's a way to check that the fail test emits text in static asserts. Please advise me on this matter if there's a portable way to do so.

Not that I'm aware of. @griwes or @wmaxey may know better than me.

Adding the .fail to the file "extension" is all I've ever done.

@gevtushenko gevtushenko force-pushed the enh-main/github/invoke_result_ext_lambdas branch 3 times, most recently from 465e6c4 to 941c3cc Compare June 13, 2022 19:09
@gevtushenko gevtushenko force-pushed the enh-main/github/invoke_result_ext_lambdas branch from 941c3cc to 6101e39 Compare June 16, 2022 10:40
@gevtushenko gevtushenko force-pushed the enh-main/github/invoke_result_ext_lambdas branch from 6101e39 to a243b84 Compare June 16, 2022 13:30
@gevtushenko gevtushenko force-pushed the enh-main/github/invoke_result_ext_lambdas branch from a243b84 to c7bd640 Compare June 21, 2022 10:20
@gevtushenko gevtushenko force-pushed the enh-main/github/invoke_result_ext_lambdas branch from c7bd640 to 1cfa273 Compare June 21, 2022 10:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants