Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make structName() and recordCoordTags() constexpr and use in Trace with CUDA #521

Merged

Conversation

bernhardmgruber
Copy link
Member

No description provided.

@bernhardmgruber bernhardmgruber force-pushed the constexpr_structname branch 2 times, most recently from 647f035 to 98849ff Compare June 7, 2022 16:32
@codecov
Copy link

codecov bot commented Jun 7, 2022

Codecov Report

Merging #521 (e841db7) into develop (e841db7) will not change coverage.
The diff coverage is n/a.

❗ Current head e841db7 differs from pull request most recent head 271643a. Consider uploading reports for the commit 271643a to get more accurate results

@@           Coverage Diff            @@
##           develop     #521   +/-   ##
========================================
  Coverage    98.74%   98.74%           
========================================
  Files           63       63           
  Lines         5888     5888           
========================================
  Hits          5814     5814           
  Misses          74       74           

@bernhardmgruber bernhardmgruber force-pushed the constexpr_structname branch 5 times, most recently from ec1853d to 9fcbb03 Compare June 9, 2022 10:39
@bernhardmgruber
Copy link
Member Author

Unfortunately, I cannot reproduce the Windows build failure locally :/

@bernhardmgruber bernhardmgruber force-pushed the constexpr_structname branch 3 times, most recently from c7f2f3f to 82b1aa2 Compare June 20, 2022 14:22
This allows to use recordCoordTags() in Trace::printFieldHits() with CUDA
@bernhardmgruber bernhardmgruber merged commit 211037c into alpaka-group:develop Jun 21, 2022
@bernhardmgruber bernhardmgruber deleted the constexpr_structname branch June 21, 2022 21:33
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.

1 participant