Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Reduce IL of lifted S.L.Expression boolean And, Or, AndAlso, OrElse#17535

Merged
VSadov merged 1 commit intodotnet:masterfrom
JonHanna:reduce_and_or_andalso_orelse
Mar 30, 2017
Merged

Reduce IL of lifted S.L.Expression boolean And, Or, AndAlso, OrElse#17535
VSadov merged 1 commit intodotnet:masterfrom
JonHanna:reduce_and_or_andalso_orelse

Conversation

@JonHanna
Copy link
Copy Markdown
Contributor

Produce smaller IL with fewer branches that doesn't call constructors, but returns either the left or right operand as appropriate.

Produce smaller IL with fewer branches that doesn't call constructors,
but returns either the left or right operand as appropriate.
@VSadov
Copy link
Copy Markdown
Member

VSadov commented Mar 30, 2017

Nice!!
I will check if this is better than what we have in Roslyn.

Copy link
Copy Markdown
Member

@VSadov VSadov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@VSadov VSadov merged commit 7f9562b into dotnet:master Mar 30, 2017
@karelz karelz modified the milestone: 2.0.0 Mar 31, 2017
@JonHanna JonHanna deleted the reduce_and_or_andalso_orelse branch July 21, 2017 20:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants