-
Notifications
You must be signed in to change notification settings - Fork 133
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
Optimization training data (PGO/IBC) in source-build #247
Comments
/cc @brianrob @adiaaida |
Thanks for getting this conversation started. Is it correct to say this is about optimization performed by the native compiler. Is there something on the managed side as well? Do you have any idea of the performance impact of optimization training data? Is it 1% to 2% or significantly more?
Is this data platform, architecture or compiler dependent? If not, could it be documented, run once and then added to the repository per release?
That shouldn't be a problem as long as both builds are source-build based with the additional data file as a new input for the second build. |
Adding to General Prebuilt Removal. We can't have this binary data as a prebuilt, but need to determine if or how to deliver this with source-build. |
Spotted this comment about open sourcing dotnet-pgo, envisioned as a replacement for ibcmerge and ibc data sometime post-5.0. dotnet/runtime#34422 (comment). Text-based and/or reproducible training data seem like the next missing parts to get this into source-build. |
FSharp has enabled PGO and would benefit from this as well - see dotnet/fsharp#17513 |
PGO/IBC optimization training data is used in CoreCLR and CoreFX (maybe more that I don't know about). There are two issues with using that in source-build:
Is it possible to fix (1)?
If we can't accomplish this, what impact does it have on source-build users?
Currently the optimization data packages are dummied out (for the offline build):
source-build/tools-local/init-build.proj
Lines 149 to 152 in a680f36
@dotnet/source-build-contrib
The text was updated successfully, but these errors were encountered: