<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>239681</bug_id>
          
          <creation_ts>2022-04-22 18:09:46 -0700</creation_ts>
          <short_desc>[Xcode] Compute PGO profdata paths instead of searching for them at build time</short_desc>
          <delta_ts>2022-05-31 14:19:30 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>239795</dependson>
    
    <dependson>241065</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Elliott Williams">emw</reporter>
          <assigned_to name="Elliott Williams">emw</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1863831</commentid>
    <comment_count>0</comment_count>
    <who name="Elliott Williams">emw</who>
    <bug_when>2022-04-22 18:09:46 -0700</bug_when>
    <thetext>Currently, the &quot;Copy Profiling Data&quot; 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&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1863833</commentid>
    <comment_count>1</comment_count>
    <who name="Elliott Williams">emw</who>
    <bug_when>2022-04-22 18:23:19 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/367</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1863835</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-04-22 18:30:04 -0700</bug_when>
    <thetext>&lt;rdar://problem/92200351&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1863918</commentid>
    <comment_count>3</comment_count>
    <who name="Elliott Williams">emw</who>
    <bug_when>2022-04-23 13:34:56 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/369</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864565</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-04-26 12:21:12 -0700</bug_when>
    <thetext>Committed r293442 (249997@main): &lt;https://commits.webkit.org/249997@main&gt;

Reviewed commits have been landed. Closing PR #367 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864673</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2022-04-26 16:37:23 -0700</bug_when>
    <thetext>Re-opened since this is blocked by bug 239795</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1871889</commentid>
    <comment_count>6</comment_count>
    <who name="Elliott Williams">emw</who>
    <bug_when>2022-05-25 16:26:00 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/1031</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1872428</commentid>
    <comment_count>7</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-05-27 14:11:36 -0700</bug_when>
    <thetext>Committed r294952 (251060@main): &lt;https://commits.webkit.org/251060@main&gt;

Reviewed commits have been landed. Closing PR #1031 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1872865</commentid>
    <comment_count>8</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2022-05-30 19:13:47 -0700</bug_when>
    <thetext>This patch was reverted in https://commits.webkit.org/251100@main</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1872879</commentid>
    <comment_count>9</comment_count>
    <who name="Elliott Williams">emw</who>
    <bug_when>2022-05-30 20:09:05 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/1171</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1873134</commentid>
    <comment_count>10</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-05-31 14:19:29 -0700</bug_when>
    <thetext>Committed r295065 (251160@main): &lt;https://commits.webkit.org/251160@main&gt;

Reviewed commits have been landed. Closing PR #1171 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>