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

update to c++17 #38

Closed
wants to merge 1 commit into from
Closed

update to c++17 #38

wants to merge 1 commit into from

Conversation

nomaddo
Copy link
Collaborator

@nomaddo nomaddo commented Feb 26, 2018

Related:
doe300/VC4CL#23
doe300/VC4CL#17

After migration to gcc 6.3, we can use features of c++14 and the subset of c++17.
Should we use c++17 in addition to c++14?

@nomaddo nomaddo requested a review from doe300 February 26, 2018 01:21
@doe300
Copy link
Owner

doe300 commented Mar 1, 2018

The reason why I did not jump to C++17 is that gcc 6.3 (which is native to the Raspbian) does not yet fully support C++17 (see here) and especially lacks support for a lot of C++17 standard-library functions (see here).
So, we would always have to check, if the particular C++17 feature we use is supported by gcc 6.3...

@nomaddo nomaddo closed this Mar 5, 2018
@nomaddo
Copy link
Collaborator Author

nomaddo commented Mar 5, 2018

This pullreq is solved in 980e691.
Update to c++14

@nomaddo nomaddo deleted the update_to_17 branch April 2, 2018 07:38
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.

2 participants