Bug 239681 - [Xcode] Compute PGO profdata paths instead of searching for them at build time
Summary: [Xcode] Compute PGO profdata paths instead of searching for them at build time
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Elliott Williams
URL:
Keywords: InRadar
Depends on: 239795 241065
Blocks:
  Show dependency treegraph
 
Reported: 2022-04-22 18:09 PDT by Elliott Williams
Modified: 2022-05-31 14:19 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Elliott Williams 2022-04-22 18:09:46 PDT
Currently, the "Copy Profiling Data" script phases find profdata to use by checking BUILT_PRODUCTS_DIR, then the SDK, and finally falling back to the default profdata stub in SRCROOT. This can cause problems if stale profdata is present in BUILT_PRODUCTS_DIR and a production build is updating SDK content. It also confuses XCBuild's dependency validation on open-source builds, because the build system expects the SDK and WebKitAdditions paths to always exist.

Instead of this dynamic approach, we can compute the expected path in xcconfigs, based on the active build settings.
Comment 1 Elliott Williams 2022-04-22 18:23:19 PDT
Pull request: https://github.com/WebKit/WebKit/pull/367
Comment 2 Radar WebKit Bug Importer 2022-04-22 18:30:04 PDT
<rdar://problem/92200351>
Comment 3 Elliott Williams 2022-04-23 13:34:56 PDT
Pull request: https://github.com/WebKit/WebKit/pull/369
Comment 4 EWS 2022-04-26 12:21:12 PDT
Committed r293442 (249997@main): <https://commits.webkit.org/249997@main>

Reviewed commits have been landed. Closing PR #367 and removing active labels.
Comment 5 WebKit Commit Bot 2022-04-26 16:37:23 PDT
Re-opened since this is blocked by bug 239795
Comment 6 Elliott Williams 2022-05-25 16:26:00 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1031
Comment 7 EWS 2022-05-27 14:11:36 PDT
Committed r294952 (251060@main): <https://commits.webkit.org/251060@main>

Reviewed commits have been landed. Closing PR #1031 and removing active labels.
Comment 8 Alexey Proskuryakov 2022-05-30 19:13:47 PDT
This patch was reverted in https://commits.webkit.org/251100@main
Comment 9 Elliott Williams 2022-05-30 20:09:05 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1171
Comment 10 EWS 2022-05-31 14:19:29 PDT
Committed r295065 (251160@main): <https://commits.webkit.org/251160@main>

Reviewed commits have been landed. Closing PR #1171 and removing active labels.