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

[simdjson] Fix error LNK2001 when compile with /fsanitize=address in MSVC #11495

Merged
merged 6 commits into from
May 27, 2020
Merged

[simdjson] Fix error LNK2001 when compile with /fsanitize=address in MSVC #11495

merged 6 commits into from
May 27, 2020

Conversation

Cheney-W
Copy link
Contributor

Describe the pull request
In VS2019 16.7 preview 2, simdjson build failed with error LNK2001: unresolved external symbol __asan_memset when compile with the option -fsanitize=address.
This option was added into the compile command as a gcc compile option.
For fixing this issue, I changed the variable that sets the compile option to take effect only in a GCC environment.

@Cheney-W Cheney-W added the info:internal This PR or Issue was filed by the vcpkg team. label May 21, 2020
@Cheney-W Cheney-W requested a review from NancyLi1013 May 21, 2020 10:33
@BillyONeal
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

ports/simdjson/portfile.cmake Outdated Show resolved Hide resolved
ports/simdjson/portfile.cmake Outdated Show resolved Hide resolved
Cheney-W and others added 2 commits May 22, 2020 13:53
Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
ports/simdjson/portfile.cmake Outdated Show resolved Hide resolved
ports/simdjson/portfile.cmake Outdated Show resolved Hide resolved
ports/simdjson/portfile.cmake Outdated Show resolved Hide resolved
@NancyLi1013 NancyLi1013 added category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines labels May 22, 2020
@dan-shaw dan-shaw merged commit 9dde75b into microsoft:master May 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants