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

Tweaks for EDG based compilers (Intel, nVidia, MCST/Elbrus, etc). #1738

Closed
wants to merge 1 commit into from

Conversation

erthink
Copy link
Contributor

@erthink erthink commented Jun 22, 2020

Using the __EDG_VERSION__ macro for checks related to EDG-based
compilers instead of a macros specific for each of ones.


I agree that my contributions are licensed under the {fmt} license, and agree to future changes to the licensing.

@erthink
Copy link
Contributor Author

erthink commented Jun 22, 2020

At the moment, I don't have exact information about the version number of the EDG-frontend, which fully supports the [[gnu::fallthrough]] for FMT_FALLTHROUGH macro.

Using the __EDG_VERSION__ macro for checks related to EDG-based
compilers instead of a macros specific for each of ones.
@vitaut
Copy link
Contributor

vitaut commented Jun 23, 2020

Merged with minor comment tweak in 8bef1c3. Thanks!

@vitaut vitaut closed this Jun 23, 2020
vitaut added a commit that referenced this pull request Jun 23, 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.

2 participants