-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add support for Pgo Mibc files to be used by the SDK #17769
Conversation
davidwrighton
commented
May 21, 2021
- A new ItemList (PublishReadyToRunPgoFiles) can be used to specify a custom list of Mibc files
- Any Mibc files defined in associated RuntimePacks via a RuntimeAsset with AssetType PgoData will also be included
- This behavior can be disabled by setting PublishReadyToRunUseRuntimePackOptimizationData to false
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
See related PR dotnet/arcade#7425 for logic around RuntimeList.xml generation |
<ItemGroup> | ||
<_ReadyToRunPgoFiles Include="@(PublishReadyToRunPgoFiles)" /> | ||
<_ReadyToRunPgoFiles Include="@(RuntimePackAsset)" | ||
Condition="'%(RuntimePackAsset.AssetType)' == 'pgodata' and '%(RuntimePackAsset.Extension)' == '.mibc' and ('$(PublishReadyToRunUseRuntimePackOptimizationData)' == 'true' or '$(PublishReadyToRunUseRuntimePackOptimizationData)' == '')" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Condition
is not aligned with Include
. I would either use '$(PublishReadyToRunUseRuntimePackOptimizationData)' != 'false'
or default that property to true
if not set.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
…uilds and pull optimization data details from RuntimeList.xml if present
3c2f478
to
0d5f55b
Compare
- A new ItemList (PublishReadyToRunPgoFiles) can be used to specify a custom list of Mibc files - Any Mibc files defined in associated RuntimePacks via a RuntimeAsset with AssetType PgoData will also be included This behavior can be disabled by setting PublishReadyToRunUseRuntimePackOptimizationData to false
…he SDK (#17930) Port #17769 to preview 5. - A new ItemList (PublishReadyToRunPgoFiles) can be used to specify a custom list of Mibc files - Any Mibc files defined in associated RuntimePacks via a RuntimeAsset with AssetType PgoData will also be included This behavior can be disabled by setting PublishReadyToRunUseRuntimePackOptimizationData to false