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

Implement load(8 bytes) with a 64 bit read on x86 #87

Merged
merged 1 commit into from
Sep 11, 2019
Merged

Implement load(8 bytes) with a 64 bit read on x86 #87

merged 1 commit into from
Sep 11, 2019

Conversation

BillyONeal
Copy link
Member

@BillyONeal BillyONeal commented Sep 11, 2019

This depends on compiler support that will first ship in Visual Studio 2019 16.4, so it's guarded to allow the build to work with current preview releases.

Resolves https://developercommunity.visualstudio.com/content/problem/274938/index.html

Replays VSO PR 199513, this is the last substantial change I hope to do before we can work wholly in GitHub first. (barring fire drills)

This depends on compiler support that will first ship in Visual Studio 2019 Update 4, so it's guarded to allow the build to work with current preview releases.

Resolves https://developercommunity.visualstudio.com/content/problem/274938/index.html

Replays VSO PR 199513, this is the last substantial change I hope to do before we can work wholly in GitHub.
@CaseyCarter
Copy link
Member

CaseyCarter commented Sep 11, 2019

Visual Studio 2019 Update 4

Shouldn't this be "Visual Studio 2019 version 16.4"? I I thought we stopped using the "Update N" nomenclature.

@BillyONeal
Copy link
Member Author

Visual Studio 2019 Update 4

Shouldn't this be "Visual Studio 2019 version 16.4"? I I thought we stopped using the "Update N" nomenclature.

I admittedly am not positive what we're calling all the things now :/. It seems I turn around and they changed it again. Looks like Stephan already fixed it.

@StephanTLavavej
Copy link
Member

Casey's guidance is strictly correct for VS 2017 and VS 2019. I personally omit "version" whenever I can get away with it.

@BillyONeal BillyONeal merged commit 04cea50 into microsoft:master Sep 11, 2019
@BillyONeal BillyONeal deleted the replay_atomic_load branch September 11, 2019 02:46
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.

None yet

3 participants