Skip to content

Untag dispatch <execution>#2662

Merged
StephanTLavavej merged 4 commits into
microsoft:mainfrom
AlexGuteniev:untag_dispatch_execution
May 1, 2022
Merged

Untag dispatch <execution>#2662
StephanTLavavej merged 4 commits into
microsoft:mainfrom
AlexGuteniev:untag_dispatch_execution

Conversation

@AlexGuteniev
Copy link
Copy Markdown
Contributor

Towards #189

@AlexGuteniev AlexGuteniev requested a review from a team as a code owner April 21, 2022 18:05
@CaseyCarter CaseyCarter added the enhancement Something can be improved label Apr 21, 2022
@StephanTLavavej StephanTLavavej added throughput Must compile faster and removed enhancement Something can be improved labels Apr 21, 2022
@StephanTLavavej StephanTLavavej self-assigned this Apr 27, 2022
@StephanTLavavej StephanTLavavej removed their assignment Apr 28, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member

Wow, goodbye duplicated code and expensive SFINAE! 😻

@StephanTLavavej StephanTLavavej self-assigned this Apr 30, 2022
@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 StephanTLavavej merged commit b0b9a58 into microsoft:main May 1, 2022
@StephanTLavavej
Copy link
Copy Markdown
Member

Thanks again for finding and if constexprizing this duplicated code! 😻 🎉 😻

@AlexGuteniev AlexGuteniev deleted the untag_dispatch_execution branch May 1, 2022 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

throughput Must compile faster

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants