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

[ARM32] Simplify getting aligment value when remorphing #11709

Merged
merged 2 commits into from
May 19, 2017

Conversation

hseok-oh
Copy link

@hseok-oh hseok-oh commented May 18, 2017

Simplify getting aligment value in morphing phase when remorphing by using argument table.
It can be useful to get alignment value for struct morphed to FIELD_LIST in ARM32/RyuJIT.

Skip aligment computation in morphing phase when remorphing
It simplify getting alignment value for struct
morphed to FIELD_LIST in ARM32/RyuJIT
@hseok-oh
Copy link
Author

cc/ @dotnet/jit-contrib

@BruceForstall BruceForstall self-requested a review May 19, 2017 04:07
@@ -3207,10 +3207,17 @@ GenTreeCall* Compiler::fgMorphArgs(GenTreeCall* call)
}

// We don't use the "size" return value from InferOpSizeAlign().

Choose a reason for hiding this comment

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

Seems like this comment ought to remain next to the call to InferOpSizeAlign.

Copy link
Author

Choose a reason for hiding this comment

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

Thank you. I fixed it.

Move comment about `InferOpSizeAlign`
@BruceForstall BruceForstall merged commit 075a2d6 into dotnet:master May 19, 2017
@hseok-oh hseok-oh deleted the armjit/compute_aligment_remorph branch May 31, 2017 04:30
@karelz karelz modified the milestone: 2.1.0 Aug 28, 2017
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.

5 participants