Skip to content

Workaround for non-_Ugly attribute tokens - #3760

Merged
Stephan T. Lavavej (StephanTLavavej) merged 4 commits into
microsoft:mainfrom
frederick-vs-ja:push-pop-attributes
Jun 15, 2023
Merged

Stephan T. Lavavej (StephanTLavavej) merged 4 commits into
microsoft:mainfrom
frederick-vs-ja:push-pop-attributes

Conversation

@frederick-vs-ja

@frederick-vs-ja A. Jiang (frederick-vs-ja) commented Jun 10, 2023

Copy link
Copy Markdown
Contributor

This PR is trying to push and pop these attribute tokens like new. Considering [[msvc::lifetimebound]] which is about to be used (#3754).

Fixes #2645.

@frederick-vs-ja
A. Jiang (frederick-vs-ja) requested a review from a team as a code owner June 10, 2023 17:54
Comment thread stl/inc/xutility Outdated
@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
Stephan T. Lavavej (StephanTLavavej) merged commit 5226b57 into microsoft:main Jun 15, 2023
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for improving our macroization defenses! 🛡️ 🎉 ✨

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<yvals_core.h>: Conformance issue on [[msvc::known_semantics]]

3 participants