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

Build src/tests tree in globalization invariant mode #111335

Closed
wants to merge 1 commit into from

Conversation

MichalStrehovsky
Copy link
Member

Saves 5% in build time (and in size) for Pri1 test builds. Globalization tests live in src/libraries; these are not runtime concerns.

@dotnet/jit-contrib could someone look at the JIT test changes? Neither of the modified tests look very interesting (I actually wonder if we should just delete both - I don't know what they're testing - the one I'm leaving will succeed even if it prints "Test failed").

Saves 5% in build time (and in size) for Pri1 test builds. Globalization tests live in src/libraries; these are not runtime concerns.
Copy link
Contributor

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

@MichalStrehovsky
Copy link
Member Author

/azp run runtime-nativeaot-outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MichalStrehovsky
Copy link
Member Author

/azp run runtime-nativeaot-outerloop

I did a pri-1 run with native AOT on Windows locally, but running pri-0 outerloop to cover non-Windows.

Copy link
Member

@EgorBo EgorBo 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 it's fine to drop b04914.cs and b15307.cs

@MichalStrehovsky
Copy link
Member Author

This needs more work and we don't currently need it (I managed to make the tests fit in a ZIP and then not OOM during upload in other ways).

@MichalStrehovsky MichalStrehovsky deleted the invarianttest branch March 17, 2025 12:35
@jkotas
Copy link
Member

jkotas commented Mar 18, 2025

I think it's fine to drop b04914.cs and b15307.cs

Extracted to #113633

@jkotas jkotas mentioned this pull request Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants