Skip to content

Workaround for non-_Ugly attribute tokens#3760

Merged
StephanTLavavej merged 4 commits intomicrosoft:mainfrom
frederick-vs-ja:push-pop-attributes
Jun 15, 2023
Merged

Workaround for non-_Ugly attribute tokens#3760
StephanTLavavej merged 4 commits intomicrosoft:mainfrom
frederick-vs-ja:push-pop-attributes

Conversation

@frederick-vs-ja
Copy link
Contributor

@frederick-vs-ja frederick-vs-ja commented Jun 10, 2023

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 frederick-vs-ja requested a review from a team as a code owner June 10, 2023 17:54
@StephanTLavavej StephanTLavavej added the bug Something isn't working label Jun 11, 2023
@StephanTLavavej StephanTLavavej self-assigned this Jun 11, 2023
@StephanTLavavej StephanTLavavej self-assigned this Jun 14, 2023
@StephanTLavavej
Copy link
Member

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

@StephanTLavavej StephanTLavavej merged commit 5226b57 into microsoft:main Jun 15, 2023
@StephanTLavavej
Copy link
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