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

Fix Unix preallocatedArgCount #19276

Merged
merged 1 commit into from
Sep 10, 2018
Merged

Conversation

CarolEidt
Copy link

@CarolEidt CarolEidt commented Aug 3, 2018

This has been an existing ToDo for some time.
This results in 19 bytes reduction in code size for pmi diffs of tests and frameworks.

This has been an existing ToDo for some time.
This results in 19 bytes reduction in code size for pmi diffs of tests and frameworks.
@CarolEidt
Copy link
Author

@dotnet/jit-contrib PTAL
This change is motivated by my work to extract the argInfo building from fgMorphArgs. Duplicating the existing behavior is more complex than just doing it right.

Copy link

@briansull briansull left a comment

Choose a reason for hiding this comment

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

Deleting code is usually good.

@CarolEidt CarolEidt merged commit efd7220 into dotnet:master Sep 10, 2018
@CarolEidt CarolEidt deleted the FixUxArgSize branch September 10, 2018 23:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants