Skip to content

Remove unused #define Context_needs_padding from stack.h#11621

Merged
dra27 merged 1 commit intoocaml:trunkfrom
dra27:no-context
Oct 13, 2022
Merged

Remove unused #define Context_needs_padding from stack.h#11621
dra27 merged 1 commit intoocaml:trunkfrom
dra27:no-context

Conversation

@dra27
Copy link
Copy Markdown
Member

@dra27 dra27 commented Oct 13, 2022

Follows-up a point in #10831. Context_needs_padding no longer has meaning (and wasn't needed in #10972 for the arm64 backend). The use of the macro itself was completely removed in #11059 as part of the documentation updates.

Copy link
Copy Markdown
Contributor

@nojb nojb left a comment

Choose a reason for hiding this comment

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

LGTM

@dra27 dra27 merged commit 33e2d04 into ocaml:trunk Oct 13, 2022
@dra27 dra27 deleted the no-context branch October 13, 2022 12:42
dra27 added a commit that referenced this pull request Oct 13, 2022
Remove unused `#define Context_needs_padding` from `stack.h`

(cherry picked from commit 33e2d04)
@dra27
Copy link
Copy Markdown
Member Author

dra27 commented Oct 13, 2022

Ta!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants