-
Notifications
You must be signed in to change notification settings - Fork 7
Delegate implementation to std::experimental #12
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
Conversation
|
Will likely be able to add back some support once we have a switch and have alternative implementations. This compiler requirement is pretty crazy for llvm. |
|
Where are we at with this PR now given that the other PR has 20 as the base? |
|
@wusatosi I don't understand all the removals. We should just remove the compilers that do not support C++20. |
The removed compilers doesn't have std::experimental |
That PR is Irrelevant to this. |
JeffGarland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok this looks ready to go -- lets merge it
Delegate implementation detail to
std::experimental.Note that this removes support for
This is the complication I was complaining about in #3 .
This also means our project currently only support > C++20.