-
Notifications
You must be signed in to change notification settings - Fork 29
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
Resolve compiler issues for common compiler configurations #42
Comments
Some of this may be impossible without either changing the call syntax or waiting until the library can support C++20. There are a couple alternatives:
This introduces a lot of duplication and isn't visually appealing.
Relevant links: Even still, this option may require a new syntax because operators can't be cleanly templated. E.g. you may need to write the syntax:
Here at least we could offer a new syntax:
This would maintain dual compatibility but would be sad to introduce a new call syntax. It's unclear to me if consteval could provide a consistent syntax to what exists today.
What's unfortunate is that the correct idx can be inferred at compile time at the call site, but not once the member method has been invoked. C++17 has everything it needs. I don't think this is trivial to resolve, so, I will leave this as an open problem, but not blocking for release. I tried every trick I had. |
Some common compilers (such as those bundled with the Android NDK on stock Bazel) have outdated or non-conformant clang implementations.
Some of these legal but unsupported syntaxes could be obviated with equivalent but supported implementations.
The text was updated successfully, but these errors were encountered: