Skip to content

utils: change handling of generic specializations in analyze_code_size.py #29382

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

Merged
merged 1 commit into from
Jan 23, 2020

Conversation

eeckstein
Copy link
Contributor

Two changes:

  • If a function is a function signature (or any other) specialization of a generic specialization, count it is generic specialization.
    Function signature speializations are just small modifications of the original function, whereas generic specializations are real copies. So it makes more sense to "prioritize" generic specializations.

  • Create a separate category of generic specializations of stdlib functions.

…e.py

Two changes:

* If a function is a function signature (or any other) specialization of a generic specialization, count it is generic specialization.
  Function signature speializations are just small modifications of the original function, whereas generic specializations are real copies. So it makes more sense to "prioritize" generic specializations.

* Create a separate category of generic specializations of stdlib functions.
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test

Copy link
Contributor

@aschwaighofer aschwaighofer left a comment

Choose a reason for hiding this comment

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

I think that makes sense.

@eeckstein eeckstein merged commit bcade5b into swiftlang:master Jan 23, 2020
@eeckstein eeckstein deleted the analyze_code_size branch January 23, 2020 18:51
@theblixguy
Copy link
Collaborator

theblixguy commented Jan 23, 2020

I think this seems to be causing python_lint.swift test to fail: https://ci.swift.org/job/swift-PR-osx-smoke-test/19993/consoleFull

19:34:16 ./utils/analyze_code_size.py:90:80: E501 line too long (103 > 79 characters)
19:34:16 ./utils/analyze_code_size.py:91:80: E501 line too long (80 > 79 characters)

edit: should be fixed by #29395

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