Skip to content

[5.7] Add emission of Frontend parseable-output in WMO jobs. #58471

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
May 2, 2022

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Apr 27, 2022

Cherry-pick of #58422

Frontend emitting parsable output, as implemented, emitted began/finished messages on a per-primary basis. Which means that -frontend-parseable-output had no effect in contexts where no primary inputs are specified, such as WMO. This change fixes that by also emitting began/finished messages when no primary outputs are specified.

Resolves rdar://91999048

@artemcm artemcm requested a review from a team as a code owner April 27, 2022 22:36
@artemcm artemcm added the r5.7 label Apr 27, 2022
@artemcm artemcm changed the title Add emission of Frontend parseable-output in WMO jobs. [5.7] Add emission of Frontend parseable-output in WMO jobs. Apr 27, 2022
@artemcm
Copy link
Contributor Author

artemcm commented Apr 27, 2022

@swift-ci please test

…shed messages on a per-primary basis. Which means that `-frontend-parseable-output` had no effect in contexts where no primary inputs are specified, such as WMO. This change fixes that by also emitting began/finished messages when no primary outputs are specified.

Resolves rdar://91999048
@artemcm artemcm force-pushed the 57ParseableWMOOutput branch from 6a91900 to 96f8df2 Compare April 28, 2022 20:24
@artemcm
Copy link
Contributor Author

artemcm commented Apr 28, 2022

@swift-ci please test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants