Skip to content

Valarray loops expansion - #1165

Merged
Stephan T. Lavavej (StephanTLavavej) merged 19 commits into
microsoft:masterfrom
AlexGuteniev:valarray_loops
Aug 26, 2020
Merged

Valarray loops expansion#1165
Stephan T. Lavavej (StephanTLavavej) merged 19 commits into
microsoft:masterfrom
AlexGuteniev:valarray_loops

Conversation

@AlexGuteniev

@AlexGuteniev Alex Guteniev (AlexGuteniev) commented Aug 8, 2020

Copy link
Copy Markdown
Contributor

Resolves #991, resolves #1169.

Note that _MOP macro was not expanded directly as is, it was simplified to reduce number of lines.

@AlexGuteniev
Alex Guteniev (AlexGuteniev) requested a review from a team as a code owner August 8, 2020 14:38
Comment thread stl/inc/valarray Outdated
Comment thread stl/inc/valarray
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the enhancement Something can be improved label Aug 8, 2020
where the Standard mandates according to @statementreply observation
Comment thread stl/inc/valarray Outdated
Comment thread stl/inc/valarray
Comment thread stl/inc/valarray Outdated
Comment thread stl/inc/valarray
Comment thread stl/inc/valarray
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for making this code clearer, improving debug codegen, and finding and fixing the magic static that eluded us! 😺

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.

<valarray>: magic static <valarray>: Avoid control flow in macro

4 participants