Skip to content

Speculatively implement LWG-3867 - #3367

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
CaseyCarter:lwg3867
Feb 3, 2023
Merged

Speculatively implement LWG-3867#3367
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
CaseyCarter:lwg3867

Conversation

@CaseyCarter

Copy link
Copy Markdown
Contributor

Fixes #3355.

@StephanTLavavej

Copy link
Copy Markdown
Member

I pushed a commit: 🐿️ 🐿️ 🐿️

@CaseyCarter

Copy link
Copy Markdown
Contributor Author

Thanks for the de-squirrel-ification Stephan T. Lavavej (@StephanTLavavej)!

@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej

Copy link
Copy Markdown
Member

🐿️

🐿️

🐿️

@CaseyCarter

Copy link
Copy Markdown
Contributor Author

Truly a shame that you folks reviewed this so quickly or we could have speculatively merged the speculative implementation of LWG-3867.

@CaseyCarter

Copy link
Copy Markdown
Contributor Author

Truly a shame that you folks reviewed this so quickly or we could have speculatively merged the speculative implementation of LWG-3867.

With that much speculation, we'd need Spectre mitigation in our codeflow process. 👻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Something can be improved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<syncstream>: is_nothrow_move_assignable_v<osyncstream> is squirrelly 🐿️

3 participants