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

Add support Microsoft Visual Studio 2022. #2960

Merged
merged 3 commits into from
Jul 4, 2022

Conversation

phprus
Copy link
Contributor

@phprus phprus commented Jul 3, 2022

Fix for #2958 (On MSVC 2022 with /std:c++20: FMT_CPLUSPLUS == 202002L).

New CI tasks, Replace __cplusplus with FMT_CPLUSPLUS (for MSVC) and Add workaround for MSVC 2022 ICE.

Additional Information:
Internal compiler error message for commit "Workaround to Microsoft Visual Studio 2022 Internal compiler error.":

D:\a\fmt\fmt\test\compile-test.cc(362,3): fatal error C1001: Internal compiler error. [D:\a\fmt\build\test\compile-test.vcxproj]
  (compiler file 'D:\a\_work\1\s\src\vctools\Compiler\CxxFE\sl\p1\c\constexpr\constexpr.cpp', line 8635)
   To work around this problem, try simplifying or changing the program near the locations listed above.
  If possible please provide a repro here: https://developercommunity.visualstudio.com
  Please choose the Technical Support command on the Visual C++
   Help menu, or open the Technical Support help file for more information

@@ -294,7 +294,14 @@ TEST(compile_test, compile_format_string_literal) {
}
#endif

#if FMT_CPLUSPLUS >= 202002L || \
// MSVS 2019 19.29.30145.0 - Support C++20 and OK.
// MSVS 2022 19.32.31332.0 - compile-test.cc(362,3): fatal error C1001: Internal
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@phprus Have you checked whether a corresponding Visual Studio 2022 issue exists for this problem?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

phprus added 3 commits July 3, 2022 23:27
Signed-off-by: Vladislav Shchapov <[email protected]>
D:\a\fmt\fmt\test\compile-test.cc(362,3): fatal error C1001: Internal compiler error. [D:\a\fmt\build\test\compile-test.vcxproj]
(compiler file 'D:\a\_work\1\s\src\vctools\Compiler\CxxFE\sl\p1\c\constexpr\constexpr.cpp', line 8635)

Signed-off-by: Vladislav Shchapov <[email protected]>
@phprus phprus force-pushed the msvc-cplusplus-1 branch from 1faed00 to 2f3f86e Compare July 3, 2022 18:27
@phprus
Copy link
Contributor Author

phprus commented Jul 3, 2022

Rebased.

@vitaut vitaut merged commit 8311322 into fmtlib:master Jul 4, 2022
@vitaut
Copy link
Contributor

vitaut commented Jul 4, 2022

Thank you!

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.

3 participants