<?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>99331</bug_id>
          
          <creation_ts>2012-10-15 08:34:11 -0700</creation_ts>
          <short_desc>We should avoid weakCompareAndSwap when parallel GC is disabled</short_desc>
          <delta_ts>2012-10-15 10:35:19 -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>528+ (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>Qt, QtTriaged</keywords>
          <priority>P1</priority>
          <bug_severity>Blocker</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>79668</blocked>
    
    <blocked>98792</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Hahnenberg">mhahnenberg</reporter>
          <assigned_to name="Mark Hahnenberg">mhahnenberg</assigned_to>
          <cc>fpizlo</cc>
    
    <cc>gaborb</cc>
    
    <cc>galpeter</cc>
    
    <cc>loki</cc>
    
    <cc>ossy</cc>
    
    <cc>rgabor</cc>
    
    <cc>zherczeg</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>742061</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-10-15 08:34:11 -0700</bug_when>
    <thetext>CopiedBlock::reportLiveBytes uses weakCompareAndSwap, which some platforms don&apos;t support. For platforms that don&apos;t have parallel GC enabled, we should just use a normal store.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742072</commentid>
    <comment_count>1</comment_count>
      <attachid>168718</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-10-15 08:39:28 -0700</bug_when>
    <thetext>Created attachment 168718
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742078</commentid>
    <comment_count>2</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-10-15 08:41:18 -0700</bug_when>
    <thetext>*** Bug 99328 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742099</commentid>
    <comment_count>3</comment_count>
      <attachid>168727</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-10-15 09:08:51 -0700</bug_when>
    <thetext>Created attachment 168727
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742158</commentid>
    <comment_count>4</comment_count>
      <attachid>168727</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2012-10-15 10:31:48 -0700</bug_when>
    <thetext>Comment on attachment 168727
Patch

You could have also said ENABLE(COMPARE_AND_SWAP)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742165</commentid>
    <comment_count>5</comment_count>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2012-10-15 10:35:19 -0700</bug_when>
    <thetext>Committed r131318: &lt;http://trac.webkit.org/changeset/131318&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>168718</attachid>
            <date>2012-10-15 08:39:28 -0700</date>
            <delta_ts>2012-10-15 09:08:49 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-99331-20121015083819.patch</filename>
            <type>text/plain</type>
            <size>2120</size>
            <attacher name="Mark Hahnenberg">mhahnenberg</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTMxMzAzKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBA
CisyMDEyLTEwLTE1ICBNYXJrIEhhaG5lbmJlcmcgIDxtaGFobmVuYmVyZ0BhcHBsZS5jb20+CisK
KyAgICAgICAgV2Ugc2hvdWxkIGF2b2lkIHdlYWtDb21wYXJlQW5kU3dhcCB3aGVuIHBhcmFsbGVs
IEdDIGlzIGRpc2FibGVkCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD05OTMzMQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg
ICAgICAgIENvcGllZEJsb2NrOjpyZXBvcnRMaXZlQnl0ZXMgYW5kIGRpZEV2YWN1YXRlQnl0ZXMg
dXNlcyB3ZWFrQ29tcGFyZUFuZFN3YXAsIHdoaWNoIHNvbWUgcGxhdGZvcm1zIAorICAgICAgICBk
b24ndCBzdXBwb3J0LiBGb3IgcGxhdGZvcm1zIHRoYXQgZG9uJ3QgaGF2ZSBwYXJhbGxlbCBHQyBl
bmFibGVkLCB3ZSBzaG91bGQganVzdCB1c2UgYSBub3JtYWwgc3RvcmUuCisKKyAgICAgICAgKiBo
ZWFwL0NvcGllZEJsb2NrLmg6CisgICAgICAgIChKU0M6OkNvcGllZEJsb2NrOjpyZXBvcnRMaXZl
Qnl0ZXMpOgorICAgICAgICAoSlNDOjpDb3BpZWRCbG9jazo6ZGlkRXZhY3VhdGVCeXRlcyk6CisK
IDIwMTItMTAtMTUgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgog
CiAgICAgICAgIFVucmV2aWV3ZWQuIEZpeCBtYWtlIGRpc3RjaGVjay4KSW5kZXg6IFNvdXJjZS9K
YXZhU2NyaXB0Q29yZS9oZWFwL0NvcGllZEJsb2NrLmgKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0ph
dmFTY3JpcHRDb3JlL2hlYXAvQ29waWVkQmxvY2suaAkocmV2aXNpb24gMTMxMzAzKQorKysgU291
cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvQ29waWVkQmxvY2suaAkod29ya2luZyBjb3B5KQpAQCAt
MTE5LDEyICsxMTksMTYgQEAgaW5saW5lIENvcGllZEJsb2NrOjpDb3BpZWRCbG9jayhSZWdpb24q
IAogCiBpbmxpbmUgdm9pZCBDb3BpZWRCbG9jazo6cmVwb3J0TGl2ZUJ5dGVzKHVuc2lnbmVkIGJ5
dGVzKQogeworI2lmIEVOQUJMRShQQVJBTExFTF9HQykKICAgICB1bnNpZ25lZCBvbGRWYWx1ZSA9
IDA7CiAgICAgdW5zaWduZWQgbmV3VmFsdWUgPSAwOwogICAgIGRvIHsKICAgICAgICAgb2xkVmFs
dWUgPSBtX2xpdmVCeXRlczsKICAgICAgICAgbmV3VmFsdWUgPSBvbGRWYWx1ZSArIGJ5dGVzOwog
ICAgIH0gd2hpbGUgKCFXVEY6OndlYWtDb21wYXJlQW5kU3dhcCgmbV9saXZlQnl0ZXMsIG9sZFZh
bHVlLCBuZXdWYWx1ZSkpOworI2Vsc2UKKyAgICBtX2xpdmVCeXRlcyArPSBieXRlczsKKyNlbmRp
ZgogfQogCiBpbmxpbmUgdm9pZCBDb3BpZWRCbG9jazo6ZGlkU3Vydml2ZUdDKCkKQEAgLTEzNiw2
ICsxNDAsNyBAQCBpbmxpbmUgdm9pZCBDb3BpZWRCbG9jazo6ZGlkU3Vydml2ZUdDKCkKIGlubGlu
ZSBib29sIENvcGllZEJsb2NrOjpkaWRFdmFjdWF0ZUJ5dGVzKHVuc2lnbmVkIGJ5dGVzKQogewog
ICAgIEFTU0VSVChtX2xpdmVCeXRlcyA+PSBieXRlcyk7CisjaWYgRU5BQkxFKFBBUkFMTEVMX0dD
KQogICAgIHVuc2lnbmVkIG9sZFZhbHVlID0gMDsKICAgICB1bnNpZ25lZCBuZXdWYWx1ZSA9IDA7
CiAgICAgZG8gewpAQCAtMTQ0LDYgKzE0OSw5IEBAIGlubGluZSBib29sIENvcGllZEJsb2NrOjpk
aWRFdmFjdWF0ZUJ5dGUKICAgICB9IHdoaWxlICghV1RGOjp3ZWFrQ29tcGFyZUFuZFN3YXAoJm1f
bGl2ZUJ5dGVzLCBvbGRWYWx1ZSwgbmV3VmFsdWUpKTsKICAgICBBU1NFUlQobV9saXZlQnl0ZXMg
PCBvbGRWYWx1ZSk7CiAgICAgcmV0dXJuICFuZXdWYWx1ZTsKKyNlbHNlCisgICAgbV9saXZlQnl0
ZXMgLT0gYnl0ZXM7CisjZW5kaWYKIH0KIAogaW5saW5lIGJvb2wgQ29waWVkQmxvY2s6OmNhbkJl
UmVjeWNsZWQoKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>168727</attachid>
            <date>2012-10-15 09:08:51 -0700</date>
            <delta_ts>2012-10-15 10:31:48 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-99331-20121015090742.patch</filename>
            <type>text/plain</type>
            <size>2147</size>
            <attacher name="Mark Hahnenberg">mhahnenberg</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTMxMzAzKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBA
CisyMDEyLTEwLTE1ICBNYXJrIEhhaG5lbmJlcmcgIDxtaGFobmVuYmVyZ0BhcHBsZS5jb20+CisK
KyAgICAgICAgV2Ugc2hvdWxkIGF2b2lkIHdlYWtDb21wYXJlQW5kU3dhcCB3aGVuIHBhcmFsbGVs
IEdDIGlzIGRpc2FibGVkCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD05OTMzMQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg
ICAgICAgIENvcGllZEJsb2NrOjpyZXBvcnRMaXZlQnl0ZXMgYW5kIGRpZEV2YWN1YXRlQnl0ZXMg
dXNlcyB3ZWFrQ29tcGFyZUFuZFN3YXAsIHdoaWNoIHNvbWUgcGxhdGZvcm1zIAorICAgICAgICBk
b24ndCBzdXBwb3J0LiBGb3IgcGxhdGZvcm1zIHRoYXQgZG9uJ3QgaGF2ZSBwYXJhbGxlbCBHQyBl
bmFibGVkLCB3ZSBzaG91bGQganVzdCB1c2UgYSBub3JtYWwgc3RvcmUuCisKKyAgICAgICAgKiBo
ZWFwL0NvcGllZEJsb2NrLmg6CisgICAgICAgIChKU0M6OkNvcGllZEJsb2NrOjpyZXBvcnRMaXZl
Qnl0ZXMpOgorICAgICAgICAoSlNDOjpDb3BpZWRCbG9jazo6ZGlkRXZhY3VhdGVCeXRlcyk6CisK
IDIwMTItMTAtMTUgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgog
CiAgICAgICAgIFVucmV2aWV3ZWQuIEZpeCBtYWtlIGRpc3RjaGVjay4KSW5kZXg6IFNvdXJjZS9K
YXZhU2NyaXB0Q29yZS9oZWFwL0NvcGllZEJsb2NrLmgKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0ph
dmFTY3JpcHRDb3JlL2hlYXAvQ29waWVkQmxvY2suaAkocmV2aXNpb24gMTMxMzAzKQorKysgU291
cmNlL0phdmFTY3JpcHRDb3JlL2hlYXAvQ29waWVkQmxvY2suaAkod29ya2luZyBjb3B5KQpAQCAt
MTE5LDEyICsxMTksMTYgQEAgaW5saW5lIENvcGllZEJsb2NrOjpDb3BpZWRCbG9jayhSZWdpb24q
IAogCiBpbmxpbmUgdm9pZCBDb3BpZWRCbG9jazo6cmVwb3J0TGl2ZUJ5dGVzKHVuc2lnbmVkIGJ5
dGVzKQogeworI2lmIEVOQUJMRShQQVJBTExFTF9HQykKICAgICB1bnNpZ25lZCBvbGRWYWx1ZSA9
IDA7CiAgICAgdW5zaWduZWQgbmV3VmFsdWUgPSAwOwogICAgIGRvIHsKICAgICAgICAgb2xkVmFs
dWUgPSBtX2xpdmVCeXRlczsKICAgICAgICAgbmV3VmFsdWUgPSBvbGRWYWx1ZSArIGJ5dGVzOwog
ICAgIH0gd2hpbGUgKCFXVEY6OndlYWtDb21wYXJlQW5kU3dhcCgmbV9saXZlQnl0ZXMsIG9sZFZh
bHVlLCBuZXdWYWx1ZSkpOworI2Vsc2UKKyAgICBtX2xpdmVCeXRlcyArPSBieXRlczsKKyNlbmRp
ZgogfQogCiBpbmxpbmUgdm9pZCBDb3BpZWRCbG9jazo6ZGlkU3Vydml2ZUdDKCkKQEAgLTEzNiw2
ICsxNDAsNyBAQCBpbmxpbmUgdm9pZCBDb3BpZWRCbG9jazo6ZGlkU3Vydml2ZUdDKCkKIGlubGlu
ZSBib29sIENvcGllZEJsb2NrOjpkaWRFdmFjdWF0ZUJ5dGVzKHVuc2lnbmVkIGJ5dGVzKQogewog
ICAgIEFTU0VSVChtX2xpdmVCeXRlcyA+PSBieXRlcyk7CisjaWYgRU5BQkxFKFBBUkFMTEVMX0dD
KQogICAgIHVuc2lnbmVkIG9sZFZhbHVlID0gMDsKICAgICB1bnNpZ25lZCBuZXdWYWx1ZSA9IDA7
CiAgICAgZG8gewpAQCAtMTQ0LDYgKzE0OSwxMCBAQCBpbmxpbmUgYm9vbCBDb3BpZWRCbG9jazo6
ZGlkRXZhY3VhdGVCeXRlCiAgICAgfSB3aGlsZSAoIVdURjo6d2Vha0NvbXBhcmVBbmRTd2FwKCZt
X2xpdmVCeXRlcywgb2xkVmFsdWUsIG5ld1ZhbHVlKSk7CiAgICAgQVNTRVJUKG1fbGl2ZUJ5dGVz
IDwgb2xkVmFsdWUpOwogICAgIHJldHVybiAhbmV3VmFsdWU7CisjZWxzZQorICAgIG1fbGl2ZUJ5
dGVzIC09IGJ5dGVzOworICAgIHJldHVybiAhbV9saXZlQnl0ZXM7CisjZW5kaWYKIH0KIAogaW5s
aW5lIGJvb2wgQ29waWVkQmxvY2s6OmNhbkJlUmVjeWNsZWQoKQo=
</data>
<flag name="review"
          id="181794"
          type_id="1"
          status="+"
          setter="fpizlo"
    />
          </attachment>
      

    </bug>

</bugzilla>