Skip to content

Restore impl of DerivedRecordSize in RDAT#4842

Merged
tex3d merged 1 commit intomicrosoft:mainfrom
tex3d:rdat-derived-record-size
Dec 5, 2022
Merged

Restore impl of DerivedRecordSize in RDAT#4842
tex3d merged 1 commit intomicrosoft:mainfrom
tex3d:rdat-derived-record-size

Conversation

@tex3d
Copy link
Copy Markdown
Contributor

@tex3d tex3d commented Dec 2, 2022

This default method implementation is necessary for the base->derived size version mechanism to work. This is only specialized when a base has a derived, and it relies on the newest version to use the default implementation (no specialization).

The implementation was removed in commit 8ee6ed1.

This default method implementation is necessary for the base->derived size version mechanism to work.
This is only specialized when a base has a derived, and it relies on the newest version to use the default implementation (no specialization).

The implementation was removed in commit 8ee6ed1.
@tex3d tex3d requested a review from llvm-beanz December 2, 2022 02:46
Copy link
Copy Markdown
Collaborator

@llvm-beanz llvm-beanz left a comment

Choose a reason for hiding this comment

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

LGTM.

@AppVeyorBot
Copy link
Copy Markdown

@tex3d tex3d merged commit af3b9d4 into microsoft:main Dec 5, 2022
@tex3d tex3d deleted the rdat-derived-record-size branch December 9, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants