<?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>124749</bug_id>
          
          <creation_ts>2013-11-21 16:34:14 -0800</creation_ts>
          <short_desc>JSC: Removed unused opcodes from offline assembler</short_desc>
          <delta_ts>2013-11-21 16:43:49 -0800</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>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Saboff">msaboff</reporter>
          <assigned_to name="Michael Saboff">msaboff</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>953341</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2013-11-21 16:34:14 -0800</bug_when>
    <thetext>The opcodes peek and pokeq are not currently used by the LLInt and therefore are redundant.  The desire to remove these at this time is due to a limitation in size of regular expressions in ruby.  For the patch https://bugs.webkit.org/show_bug.cgi?id=124721 - &quot;ARM64: Implement push/pop equivalents in LLInt&quot; I&apos;d like to add two new opcodes.  By removing these two opcodes, we can add the opcodes in that patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>953342</commentid>
    <comment_count>1</comment_count>
      <attachid>217627</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2013-11-21 16:41:24 -0800</bug_when>
    <thetext>Created attachment 217627
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>953343</commentid>
    <comment_count>2</comment_count>
      <attachid>217627</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2013-11-21 16:42:34 -0800</bug_when>
    <thetext>Comment on attachment 217627
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>953344</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2013-11-21 16:43:49 -0800</bug_when>
    <thetext>Committed r159654: &lt;http://trac.webkit.org/changeset/159654&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>217627</attachid>
            <date>2013-11-21 16:41:24 -0800</date>
            <delta_ts>2013-11-21 16:42:34 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>124749.patch</filename>
            <type>text/plain</type>
            <size>2308</size>
            <attacher name="Michael Saboff">msaboff</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTU5NjUzKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBA
CisyMDEzLTExLTIxICBNaWNoYWVsIFNhYm9mZiAgPG1zYWJvZmZAYXBwbGUuY29tPgorCisgICAg
ICAgIEpTQzogUmVtb3ZlZCB1bnVzZWQgb3Bjb2RlcyBmcm9tIG9mZmxpbmUgYXNzZW1ibGVyCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMjQ3NDkKKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBSZW1vdmVkIHRo
ZSB1bnVzZWQsIFg4NiBvbmx5IHBlZWtxIGFuZCBwb2tlcS4KKworICAgICAgICAqIG9mZmxpbmVh
c20vaW5zdHJ1Y3Rpb25zLnJiOgorICAgICAgICAqIG9mZmxpbmVhc20veDg2LnJiOgorCiAyMDEz
LTExLTIxICBNaWNoYWVsIFNhYm9mZiAgPG1zYWJvZmZAYXBwbGUuY29tPgogCiAgICAgICAgIFJF
R1JFU1NJT04oMTU5Mzk1KSBGaXggYnJhbmNoOCjigKYsIEFic29sdXRlQWRkcmVzcywg4oCmKSBp
biBBUk02NCBNYWNyb0Fzc2VtYmxlcgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL29mZmxp
bmVhc20vaW5zdHJ1Y3Rpb25zLnJiCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29y
ZS9vZmZsaW5lYXNtL2luc3RydWN0aW9ucy5yYgkocmV2aXNpb24gMTU5NjUyKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL29mZmxpbmVhc20vaW5zdHJ1Y3Rpb25zLnJiCSh3b3JraW5nIGNvcHkp
CkBAIC0yMDYsOCArMjA2LDYgQEAgTUFDUk9fSU5TVFJVQ1RJT05TID0KICAgICAgInRxcyIsCiAg
ICAgICJ0cXoiLAogICAgICAidHFueiIsCi0gICAgICJwZWVrcSIsCi0gICAgICJwb2tlcSIsCiAg
ICAgICJicWVxIiwKICAgICAgImJxbmVxIiwKICAgICAgImJxYSIsCkluZGV4OiBTb3VyY2UvSmF2
YVNjcmlwdENvcmUvb2ZmbGluZWFzbS94ODYucmIKPT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFT
Y3JpcHRDb3JlL29mZmxpbmVhc20veDg2LnJiCShyZXZpc2lvbiAxNTk2NTIpCisrKyBTb3VyY2Uv
SmF2YVNjcmlwdENvcmUvb2ZmbGluZWFzbS94ODYucmIJKHdvcmtpbmcgY29weSkKQEAgLTEyODYs
MTUgKzEyODYsOSBAQCBjbGFzcyBJbnN0cnVjdGlvbgogICAgICAgICB3aGVuICJwZWVrIgogICAg
ICAgICAgICAgc3AgPSBSZWdpc3RlcklELm5ldyhuaWwsICJzcCIpCiAgICAgICAgICAgICAkYXNt
LnB1dHMgIm1vdiN7eDg2U3VmZml4KDpwdHIpfSAje29wZXJhbmRzWzBdLnZhbHVlICogeDg2Qnl0
ZXMoOnB0cil9KCN7c3AueDg2T3BlcmFuZCg6cHRyKX0pLCAje29wZXJhbmRzWzFdLng4Nk9wZXJh
bmQoOnB0cil9IgotICAgICAgICB3aGVuICJwZWVrcSIKLSAgICAgICAgICAgIHNwID0gUmVnaXN0
ZXJJRC5uZXcobmlsLCAic3AiKQotICAgICAgICAgICAgJGFzbS5wdXRzICJtb3Yje3g4NlN1ZmZp
eCg6cXVhZCl9ICN7b3BlcmFuZHNbMF0udmFsdWUgKiB4ODZCeXRlcyg6cXVhZCl9KCN7c3AueDg2
T3BlcmFuZCg6cHRyKX0pLCAje29wZXJhbmRzWzFdLng4Nk9wZXJhbmQoOnF1YWQpfSIKICAgICAg
ICAgd2hlbiAicG9rZSIKICAgICAgICAgICAgIHNwID0gUmVnaXN0ZXJJRC5uZXcobmlsLCAic3Ai
KQogICAgICAgICAgICAgJGFzbS5wdXRzICJtb3Yje3g4NlN1ZmZpeCg6cHRyKX0gI3tvcGVyYW5k
c1swXS54ODZPcGVyYW5kKDpwdHIpfSwgI3tvcGVyYW5kc1sxXS52YWx1ZSAqIHg4NkJ5dGVzKDpw
dHIpfSgje3NwLng4Nk9wZXJhbmQoOnB0cil9KSIKLSAgICAgICAgd2hlbiAicG9rZXEiCi0gICAg
ICAgICAgICBzcCA9IFJlZ2lzdGVySUQubmV3KG5pbCwgInNwIikKLSAgICAgICAgICAgICRhc20u
cHV0cyAibW92I3t4ODZTdWZmaXgoOnF1YWQpfSAje29wZXJhbmRzWzBdLng4Nk9wZXJhbmQoOnF1
YWQpfSwgI3tvcGVyYW5kc1sxXS52YWx1ZSAqIHg4NkJ5dGVzKDpxdWFkKX0oI3tzcC54ODZPcGVy
YW5kKDpwdHIpfSkiCiAgICAgICAgIHdoZW4gImNkcWkiCiAgICAgICAgICAgICAkYXNtLnB1dHMg
ImNkcSIKICAgICAgICAgd2hlbiAiaWRpdmkiCg==
</data>
<flag name="review"
          id="240798"
          type_id="1"
          status="+"
          setter="mhahnenberg"
    />
          </attachment>
      

    </bug>

</bugzilla>