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
BruceForstall merged 2 commits into
dotnet:masterfrom
hseok-oh:armjit/compute_aligment_remorph
May 19, 2017
Merged

[ARM32] Simplify getting aligment value when remorphing#11709
BruceForstall merged 2 commits into
dotnet:masterfrom
hseok-oh:armjit/compute_aligment_remorph

Conversation

@hseok-oh
Copy link
Copy Markdown

@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
Copy Markdown
Author

cc/ @dotnet/jit-contrib

@BruceForstall BruceForstall self-requested a review May 19, 2017 04:07
Comment thread src/jit/morph.cpp Outdated
@@ -3207,10 +3207,17 @@ GenTreeCall* Compiler::fgMorphArgs(GenTreeCall* call)
}

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

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
Copy Markdown
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