<?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>270699</bug_id>
          
          <creation_ts>2024-03-08 07:30:20 -0800</creation_ts>
          <short_desc>[JSC] Add arbitrary label alignment functionality to offlineasm</short_desc>
          <delta_ts>2024-04-08 13:45:46 -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>JavaScriptCore</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>REOPENED</bug_status>
          <resolution></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>271065</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Max Rottenkolber">maximilian</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>justin_michaud</cc>
    
    <cc>mark.lam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2019614</commentid>
    <comment_count>0</comment_count>
    <who name="Max Rottenkolber">maximilian</who>
    <bug_when>2024-03-08 07:30:20 -0800</bug_when>
    <thetext>To express some programs like the InPlaceInterpreter we need to align instructions to specific boundaries (such as 256).

We currently to do this via `emit &quot;.balign 256, 0xcc&quot;&apos; (for instance) but this is not reliable on armv7 as its globaladdr implementation can insert code unpredictably (from the point of view of the offlineasm user), hence breaking the alignment.

We should add functionality in offlineasm to address this usecase reliably.

Possibly related to rdar://113594783</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2020991</commentid>
    <comment_count>1</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-03-14 06:01:23 -0700</bug_when>
    <thetext>Committed 276085@main (71f803e6ee62): &lt;https://commits.webkit.org/276085@main&gt;

Reviewed commits have been landed. Closing PR #25628 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2020992</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-03-14 06:02:15 -0700</bug_when>
    <thetext>&lt;rdar://problem/124613758&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2021383</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2024-03-15 10:41:36 -0700</bug_when>
    <thetext>Re-opened since this is blocked by bug 271065</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026147</commentid>
    <comment_count>4</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2024-04-04 17:43:04 -0700</bug_when>
    <thetext>rdar://113594783</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026149</commentid>
    <comment_count>5</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2024-04-04 17:44:00 -0700</bug_when>
    <thetext>*** Bug 272200 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026150</commentid>
    <comment_count>6</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2024-04-04 17:47:40 -0700</bug_when>
    <thetext>Pull request for re-land: https://github.com/WebKit/WebKit/pull/26877</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026355</commentid>
    <comment_count>7</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-04-05 13:15:43 -0700</bug_when>
    <thetext>Committed 277134@main (75713dba1787): &lt;https://commits.webkit.org/277134@main&gt;

Reviewed commits have been landed. Closing PR #26877 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026839</commentid>
    <comment_count>8</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-04-08 13:27:18 -0700</bug_when>
    <thetext>Committed 277212@main (885441cde700): &lt;https://commits.webkit.org/277212@main&gt;

Reviewed commits have been landed. Closing PR #26986 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026843</commentid>
    <comment_count>9</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2024-04-08 13:33:38 -0700</bug_when>
    <thetext>We&apos;re going to revert this because it&apos;s causing crashes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2026932</commentid>
    <comment_count>10</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2024-04-08 13:45:46 -0700</bug_when>
    <thetext>This was reverted by Justin Michaud in https://commits.webkit.org/277212@main.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>