-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
<source_location>
: Remove EDG workaround for DevCom-10199227
#4939
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Should we add |
We have so little test coverage of We should alter the specific test to provide coverage. (But I don't think we need to alter the modules and header units tests.) |
You preserve the compile time value until runtime where you can examine it. In this case, the function names all have static storage duration like string literals. You can simply return the value from the function to initialize a |
Relabeling as this is a workaround removal in product code, purr! 😻 |
<source_location>
: Remove EDG workaround for DevCom-10199227
I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed. |
Thanks for the extensive workaround removal! 🧹 🪄 😺 |
Fixed In: Visual Studio 2022 version 17.11.2
DevCom-10199227