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

[HIPIFY][#1021][#1022][build][MSVS][fix][partial] Fix for error C2365 occurred on VS 2019 with ToT clang 18.0.0git #1023

Merged
merged 1 commit into from
Sep 16, 2023

Conversation

emankov
Copy link
Collaborator

@emankov emankov commented Sep 16, 2023

…ror C2365` occurred on VS 2019 with ToT clang 18.0.0git

+ Widen the range of `MSVC_VERSION` from 1930 (initial version of VS 2022 17.0.0) to 1920 (initial version of VS 2019 16.0.0)
+ [Reason] Conforming preprocessor is needed for LLVM 18.0.0git on VS 2019 as well
+ [Details]
  https://learn.microsoft.com/en-us/cpp/preprocessor/preprocessor-experimental-overview?view=msvc-170
  https://learn.microsoft.com/en-us/cpp/preprocessor/preprocessor-experimental-overview?view=msvc-160
@emankov emankov added fix It fixes bug clang clang compiler related issue or change build Building issue/fix MSVS MS Visual Studio-related partial Partial fix or implementation labels Sep 16, 2023
@emankov emankov self-assigned this Sep 16, 2023
@emankov emankov merged commit 5682802 into ROCm:amd-staging Sep 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Building issue/fix clang clang compiler related issue or change fix It fixes bug MSVS MS Visual Studio-related partial Partial fix or implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant