Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Seed more methods with AotCompilerTools. #1574

Conversation

UltraNamahage
Copy link
Contributor

@UltraNamahage UltraNamahage commented Mar 10, 2021

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following matches the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

If you upcast a class that implements a virtual generic method to a superclass or interface, the AoT build may not work properly.
They must be seeded appropriately.
Details are explained here.

@CLAassistant
Copy link

CLAassistant commented Mar 10, 2021

CLA assistant check
All committers have signed the CLA.

@JimBobSquarePants
Copy link
Member

This all looks excellent @UltraNamahage thanks ever so much!

I'm happy to approve this so let me know if you want to add anything else before I merge.

@UltraNamahage UltraNamahage marked this pull request as ready for review March 13, 2021 19:13
@UltraNamahage
Copy link
Contributor Author

@JimBobSquarePants I'm sorry. There was still something missing.

@JimBobSquarePants
Copy link
Member

Thanks! I wasn't aware of the PreserveAttribute functionality. Very useful!

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.

None yet

3 participants