Skip to content

NVCC: Fix Compile with C++14 Constexpr#71

Closed
ax3l wants to merge 1 commit intompark:masterfrom
ax3l:fix-variadicUsing
Closed

NVCC: Fix Compile with C++14 Constexpr#71
ax3l wants to merge 1 commit intompark:masterfrom
ax3l:fix-variadicUsing

Conversation

@ax3l
Copy link
Copy Markdown
Contributor

@ax3l ax3l commented Nov 26, 2019

Out of the many C++14 constexpr constructs, one does not compile with NVCC - even when using -std=c++14 --expt-relaxed-constexpr.

Seen with all versions of NVCC so far (latest tests with 10.1 and 10.2), occurs even if simply included for host-side code.

Fix #70

Update: uh, it helps to replace the brackets with an explicit constructor of an initializer list :) does not work

@ax3l ax3l changed the base branch from master to dev November 26, 2019 01:59
@ax3l ax3l force-pushed the fix-variadicUsing branch from f1c495c to 5e89cfc Compare November 26, 2019 01:59
@ax3l ax3l changed the base branch from dev to master November 26, 2019 02:01
@ax3l ax3l force-pushed the fix-variadicUsing branch 2 times, most recently from f30853a to 715758b Compare November 26, 2019 02:13
@ax3l ax3l changed the title NVCC: Avoid Specific Constexpr Construct NVCC: Fix Compile with C++14 Constexpr Nov 26, 2019
@ax3l ax3l force-pushed the fix-variadicUsing branch from 0c23bf5 to 9c9d6fa Compare November 26, 2019 09:36
Out of the many C++14 `constexpr` constructs, one does not compile
with NVCC - even when using `-std=c++14 --expt-relaxed-constexpr`.
Seen with all versions of NVCC so far (latest tests with 10.1 and
10.2).
@ax3l ax3l force-pushed the fix-variadicUsing branch from 9c9d6fa to 443d945 Compare November 26, 2019 10:04
@ax3l
Copy link
Copy Markdown
Contributor Author

ax3l commented Nov 26, 2019

@mpark CI seams to fail in master for unrelated reasons

@gridley
Copy link
Copy Markdown
Contributor

gridley commented Dec 31, 2019

So, I think that #73 may fix this. You should take a look. Simplifying all-not to not-any seems to do the trick.

@mpark
Copy link
Copy Markdown
Owner

mpark commented Jan 6, 2020

@ax3l sorry for the delay, been busy for a while. do you know if #73 solves the problem?

@ax3l
Copy link
Copy Markdown
Contributor Author

ax3l commented Jan 6, 2020

Thanks for the hint, I'll try to test this these days again.
Is #73 already ready to be tested? CI does not pass on it.

@gridley
Copy link
Copy Markdown
Contributor

gridley commented Jan 6, 2020

I was unable to figure out why CI would fail on #73 myself. I didn’t try to fix it since it had been mentioned CI was failing in master anyways.

@mpark
Copy link
Copy Markdown
Owner

mpark commented Jan 6, 2020

The configs for which the CI is failing is due to libc++ tests failures, from not having implemented https://wg21.link/P0608 yet. I'm working on fixing it.

@ax3l
Copy link
Copy Markdown
Contributor Author

ax3l commented Jan 8, 2020

Fixed by #73 as well, closing in favour of it :)
Thanks a lot!

@ax3l ax3l closed this Jan 8, 2020
@ax3l ax3l mentioned this pull request Jan 8, 2020
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.

NVCC & C++14 Constexpr

3 participants