<?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>165981</bug_id>
          
          <creation_ts>2016-12-16 17:14:52 -0800</creation_ts>
          <short_desc>WebAssembly: We still have some incorrect parsing productions inside unreachable code</short_desc>
          <delta_ts>2016-12-16 19:20:32 -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>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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Saam Barati">saam</reporter>
          <assigned_to name="Saam Barati">saam</assigned_to>
          <cc>benjamin</cc>
    
    <cc>commit-queue</cc>
    
    <cc>fpizlo</cc>
    
    <cc>ggaren</cc>
    
    <cc>gskachkov</cc>
    
    <cc>jfbastien</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>oliver</cc>
    
    <cc>ticaiolima</cc>
    
    <cc>ysuzuki</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1260838</commentid>
    <comment_count>0</comment_count>
    <who name="Saam Barati">saam</who>
    <bug_when>2016-12-16 17:14:52 -0800</bug_when>
    <thetext>...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1260841</commentid>
    <comment_count>1</comment_count>
      <attachid>297378</attachid>
    <who name="Saam Barati">saam</who>
    <bug_when>2016-12-16 17:25:59 -0800</bug_when>
    <thetext>Created attachment 297378
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1260842</commentid>
    <comment_count>2</comment_count>
      <attachid>297379</attachid>
    <who name="Saam Barati">saam</who>
    <bug_when>2016-12-16 17:27:33 -0800</bug_when>
    <thetext>Created attachment 297379
patch

Hopefully this one builds.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1260849</commentid>
    <comment_count>3</comment_count>
      <attachid>297379</attachid>
    <who name="Keith Miller">keith_miller</who>
    <bug_when>2016-12-16 18:53:18 -0800</bug_when>
    <thetext>Comment on attachment 297379
patch

r=me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1260858</commentid>
    <comment_count>4</comment_count>
      <attachid>297379</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-12-16 19:20:25 -0800</bug_when>
    <thetext>Comment on attachment 297379
patch

Clearing flags on attachment: 297379

Committed r209955: &lt;http://trac.webkit.org/changeset/209955&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1260859</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-12-16 19:20:32 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>297378</attachid>
            <date>2016-12-16 17:25:59 -0800</date>
            <delta_ts>2016-12-16 17:27:33 -0800</delta_ts>
            <desc>patch</desc>
            <filename>b-backup.diff</filename>
            <type>text/plain</type>
            <size>2910</size>
            <attacher name="Saam Barati">saam</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMjA5OTUzKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBA
CisyMDE2LTEyLTE2ICBTYWFtIEJhcmF0aSAgPHNiYXJhdGlAYXBwbGUuY29tPgorCisgICAgICAg
IFdlYkFzc2VtYmx5OiBXZSBzdGlsbCBoYXZlIHNvbWUgaW5jb3JyZWN0IHBhcnNpbmcgcHJvZHVj
dGlvbnMgaW5zaWRlIHVucmVhY2hhYmxlIGNvZGUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTE2NTk4MQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgaGFyZGVucyBvdXIgcGFyc2luZyBmb3IgQ2FsbElu
ZGlyZWN0IGFuZCBMb29wL0Jsb2NrL0lmIHRvIGJlIGV4YWN0bHkgbGlrZSB0aGVpciByZWFjaGFi
bGUgdmFyaWFudC4KKyAgICAgICAgCisgICAgICAgIEl0IGFsc28gZml4ZXMgYSBtb3JlIG5lZmFy
aW91cyBidWcgaW4gd2hpY2ggd2Ugd2VyZSBkZWNvZGluZyBhbiBleHRyYSB2YXJ1aW50MzIKKyAg
ICAgICAgZm9yIEJyL0JySWYgaW5zaWRlIHVucmVhY2hhYmxlIGNvZGUuCisKKyAgICAgICAgKiB3
YXNtL1dhc21GdW5jdGlvblBhcnNlci5oOgorCiAyMDE2LTEyLTE2ICBTYWFtIEJhcmF0aSAgPHNi
YXJhdGlAYXBwbGUuY29tPgogCiAgICAgICAgIEIzOjpEb3VibGVUb0Zsb2F0UmVkdWN0aW9uIHdp
bGwgYWNjaWRlbnRhbGx5IGNvbnZpbmNlIGl0c2VsZiBpdCBjb252ZXJ0ZWQgYSBQaGkgZnJvbSBE
b3VibGUgdG8gRmxvYXQgYW5kIHRoZW4gY29udmVydCB1c2VzIG9mIHRoYXQgUGhpIGludG8gYSB1
c2Ugb2YgRmxvYXRUb0RvdWJsZShAUGhpKQpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL3dh
c20vV2FzbUZ1bmN0aW9uUGFyc2VyLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRD
b3JlL3dhc20vV2FzbUZ1bmN0aW9uUGFyc2VyLmgJKHJldmlzaW9uIDIwOTk1MykKKysrIFNvdXJj
ZS9KYXZhU2NyaXB0Q29yZS93YXNtL1dhc21GdW5jdGlvblBhcnNlci5oCSh3b3JraW5nIGNvcHkp
CkBAIC01MjcsOCArNTI3LDggQEAgYXV0byBGdW5jdGlvblBhcnNlcjxDb250ZXh0Pjo6cGFyc2VV
bnJlYQogICAgIGNhc2UgSWY6CiAgICAgY2FzZSBCbG9jazogewogICAgICAgICBtX3VucmVhY2hh
YmxlQmxvY2tzKys7Ci0gICAgICAgIGludDhfdCB1bnVzZWQ7Ci0gICAgICAgIFdBU01fUEFSU0VS
X0ZBSUxfSUYoIXBhcnNlSW50Nyh1bnVzZWQpLCAiY2FuJ3QgZ2V0IGlubGluZSB0eXBlIGZvciAi
LCBvcCwgIiBpbiB1bnJlYWNoYWJsZSBjb250ZXh0Iik7CisgICAgICAgIFR5cGUgdW51c2VkOwor
ICAgICAgICBXQVNNX1BBUlNFUl9GQUlMX0lGKCFwYXJzZVJlc3VsdFR5cGUodW51c2VkKSwgImNh
bid0IGdldCBpbmxpbmUgdHlwZSBmb3IgIiwgb3AsICIgaW4gdW5yZWFjaGFibGUgY29udGV4dCIp
OwogICAgICAgICByZXR1cm4geyB9OwogICAgIH0KIApAQCAtNTQ1LDEzICs1NDUsMTYgQEAgYXV0
byBGdW5jdGlvblBhcnNlcjxDb250ZXh0Pjo6cGFyc2VVbnJlYQogICAgICAgICByZXR1cm4geyB9
OwogICAgIH0KIAorICAgIGNhc2UgQ2FsbEluZGlyZWN0OiB7CisgICAgICAgIHVpbnQzMl90IHVu
dXNlZDsKKyAgICAgICAgV0FTTV9QQVJTRVJfRkFJTF9JRighcGFyc2VWYXJVSW50MzIodW51c2Vk
KSwgImNhbid0IGdldCBjYWxsX2luZGlyZWN0J3Mgc2lnbmF0dXJlIGluZGV4IGluIHVucmVhY2hh
YmxlIGNvbnRleHQiKTsKKyAgICAgICAgV0FTTV9QQVJTRVJfRkFJTF9JRighcGFyc2VWYXJVSW50
MSh1bnVzZWQpLCAiY2FuJ3QgZ2V0IGNhbGxfaW5kaXJlY3QncyByZXNlcnZlZCBieXRlIGluIHVu
cmVhY2hhYmxlIGNvbnRleHQiKTsKKyAgICAgICAgcmV0dXJuIHsgfTsKKyAgICB9CiAKICAgICAv
LyB0d28gaW1tZWRpYXRlIGNhc2VzCiAgICAgRk9SX0VBQ0hfV0FTTV9NRU1PUllfTE9BRF9PUChD
UkVBVEVfQ0FTRSkKLSAgICBGT1JfRUFDSF9XQVNNX01FTU9SWV9TVE9SRV9PUChDUkVBVEVfQ0FT
RSkKLSAgICBjYXNlIEJyOgotICAgIGNhc2UgQnJJZjoKLSAgICBjYXNlIENhbGxJbmRpcmVjdDog
eworICAgIEZPUl9FQUNIX1dBU01fTUVNT1JZX1NUT1JFX09QKENSRUFURV9DQVNFKSB7CiAgICAg
ICAgIHVpbnQzMl90IHVudXNlZDsKICAgICAgICAgV0FTTV9QQVJTRVJfRkFJTF9JRighcGFyc2VW
YXJVSW50MzIodW51c2VkKSwgImNhbid0IGdldCBmaXJzdCBpbW1lZGlhdGUgZm9yICIsIG9wLCAi
IGluIHVucmVhY2hhYmxlIGNvbnRleHQiKTsKICAgICAgICAgV0FTTV9QQVJTRVJfRkFJTF9JRigh
cGFyc2VWYXJVSW50MzIodW51c2VkKSwgImNhbid0IGdldCBzZWNvbmQgaW1tZWRpYXRlIGZvciAi
LCBvcCwgIiBpbiB1bnJlYWNoYWJsZSBjb250ZXh0Iik7CkBAIC01NjgsNiArNTcxLDggQEAgYXV0
byBGdW5jdGlvblBhcnNlcjxDb250ZXh0Pjo6cGFyc2VVbnJlYQogICAgIGNhc2UgVGVlTG9jYWw6
CiAgICAgY2FzZSBHZXRHbG9iYWw6CiAgICAgY2FzZSBTZXRHbG9iYWw6CisgICAgY2FzZSBCcjoK
KyAgICBjYXNlIEJySWY6CiAgICAgY2FzZSBDYWxsOiB7CiAgICAgICAgIHVpbnQzMl90IHVudXNl
ZDsKICAgICAgICAgV0FTTV9QQVJTRVJfRkFJTF9JRighcGFyc2VWYXJVSW50MzIodW51c2VkKSwg
ImNhbid0IGdldCBpbW1lZGlhdGUgZm9yICIsIG9wLCAiIGluIHVucmVhY2hhYmxlIGNvbnRleHQi
KTsK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>297379</attachid>
            <date>2016-12-16 17:27:33 -0800</date>
            <delta_ts>2016-12-16 19:20:25 -0800</delta_ts>
            <desc>patch</desc>
            <filename>b-backup.diff</filename>
            <type>text/plain</type>
            <size>2937</size>
            <attacher name="Saam Barati">saam</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMjA5OTUzKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBA
CisyMDE2LTEyLTE2ICBTYWFtIEJhcmF0aSAgPHNiYXJhdGlAYXBwbGUuY29tPgorCisgICAgICAg
IFdlYkFzc2VtYmx5OiBXZSBzdGlsbCBoYXZlIHNvbWUgaW5jb3JyZWN0IHBhcnNpbmcgcHJvZHVj
dGlvbnMgaW5zaWRlIHVucmVhY2hhYmxlIGNvZGUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTE2NTk4MQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgaGFyZGVucyBvdXIgcGFyc2luZyBmb3IgQ2FsbElu
ZGlyZWN0IGFuZCBMb29wL0Jsb2NrL0lmIHRvIGJlIGV4YWN0bHkgbGlrZSB0aGVpciByZWFjaGFi
bGUgdmFyaWFudC4KKyAgICAgICAgCisgICAgICAgIEl0IGFsc28gZml4ZXMgYSBtb3JlIG5lZmFy
aW91cyBidWcgaW4gd2hpY2ggd2Ugd2VyZSBkZWNvZGluZyBhbiBleHRyYSB2YXJ1aW50MzIKKyAg
ICAgICAgZm9yIEJyL0JySWYgaW5zaWRlIHVucmVhY2hhYmxlIGNvZGUuCisKKyAgICAgICAgKiB3
YXNtL1dhc21GdW5jdGlvblBhcnNlci5oOgorCiAyMDE2LTEyLTE2ICBTYWFtIEJhcmF0aSAgPHNi
YXJhdGlAYXBwbGUuY29tPgogCiAgICAgICAgIEIzOjpEb3VibGVUb0Zsb2F0UmVkdWN0aW9uIHdp
bGwgYWNjaWRlbnRhbGx5IGNvbnZpbmNlIGl0c2VsZiBpdCBjb252ZXJ0ZWQgYSBQaGkgZnJvbSBE
b3VibGUgdG8gRmxvYXQgYW5kIHRoZW4gY29udmVydCB1c2VzIG9mIHRoYXQgUGhpIGludG8gYSB1
c2Ugb2YgRmxvYXRUb0RvdWJsZShAUGhpKQpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL3dh
c20vV2FzbUZ1bmN0aW9uUGFyc2VyLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRD
b3JlL3dhc20vV2FzbUZ1bmN0aW9uUGFyc2VyLmgJKHJldmlzaW9uIDIwOTk1MykKKysrIFNvdXJj
ZS9KYXZhU2NyaXB0Q29yZS93YXNtL1dhc21GdW5jdGlvblBhcnNlci5oCSh3b3JraW5nIGNvcHkp
CkBAIC01MjcsOCArNTI3LDggQEAgYXV0byBGdW5jdGlvblBhcnNlcjxDb250ZXh0Pjo6cGFyc2VV
bnJlYQogICAgIGNhc2UgSWY6CiAgICAgY2FzZSBCbG9jazogewogICAgICAgICBtX3VucmVhY2hh
YmxlQmxvY2tzKys7Ci0gICAgICAgIGludDhfdCB1bnVzZWQ7Ci0gICAgICAgIFdBU01fUEFSU0VS
X0ZBSUxfSUYoIXBhcnNlSW50Nyh1bnVzZWQpLCAiY2FuJ3QgZ2V0IGlubGluZSB0eXBlIGZvciAi
LCBvcCwgIiBpbiB1bnJlYWNoYWJsZSBjb250ZXh0Iik7CisgICAgICAgIFR5cGUgdW51c2VkOwor
ICAgICAgICBXQVNNX1BBUlNFUl9GQUlMX0lGKCFwYXJzZVJlc3VsdFR5cGUodW51c2VkKSwgImNh
bid0IGdldCBpbmxpbmUgdHlwZSBmb3IgIiwgb3AsICIgaW4gdW5yZWFjaGFibGUgY29udGV4dCIp
OwogICAgICAgICByZXR1cm4geyB9OwogICAgIH0KIApAQCAtNTQ1LDEzICs1NDUsMTcgQEAgYXV0
byBGdW5jdGlvblBhcnNlcjxDb250ZXh0Pjo6cGFyc2VVbnJlYQogICAgICAgICByZXR1cm4geyB9
OwogICAgIH0KIAorICAgIGNhc2UgQ2FsbEluZGlyZWN0OiB7CisgICAgICAgIHVpbnQzMl90IHVu
dXNlZDsKKyAgICAgICAgdWludDhfdCB1bnVzZWQyOworICAgICAgICBXQVNNX1BBUlNFUl9GQUlM
X0lGKCFwYXJzZVZhclVJbnQzMih1bnVzZWQpLCAiY2FuJ3QgZ2V0IGNhbGxfaW5kaXJlY3QncyBz
aWduYXR1cmUgaW5kZXggaW4gdW5yZWFjaGFibGUgY29udGV4dCIpOworICAgICAgICBXQVNNX1BB
UlNFUl9GQUlMX0lGKCFwYXJzZVZhclVJbnQxKHVudXNlZDIpLCAiY2FuJ3QgZ2V0IGNhbGxfaW5k
aXJlY3QncyByZXNlcnZlZCBieXRlIGluIHVucmVhY2hhYmxlIGNvbnRleHQiKTsKKyAgICAgICAg
cmV0dXJuIHsgfTsKKyAgICB9CiAKICAgICAvLyB0d28gaW1tZWRpYXRlIGNhc2VzCiAgICAgRk9S
X0VBQ0hfV0FTTV9NRU1PUllfTE9BRF9PUChDUkVBVEVfQ0FTRSkKLSAgICBGT1JfRUFDSF9XQVNN
X01FTU9SWV9TVE9SRV9PUChDUkVBVEVfQ0FTRSkKLSAgICBjYXNlIEJyOgotICAgIGNhc2UgQnJJ
ZjoKLSAgICBjYXNlIENhbGxJbmRpcmVjdDogeworICAgIEZPUl9FQUNIX1dBU01fTUVNT1JZX1NU
T1JFX09QKENSRUFURV9DQVNFKSB7CiAgICAgICAgIHVpbnQzMl90IHVudXNlZDsKICAgICAgICAg
V0FTTV9QQVJTRVJfRkFJTF9JRighcGFyc2VWYXJVSW50MzIodW51c2VkKSwgImNhbid0IGdldCBm
aXJzdCBpbW1lZGlhdGUgZm9yICIsIG9wLCAiIGluIHVucmVhY2hhYmxlIGNvbnRleHQiKTsKICAg
ICAgICAgV0FTTV9QQVJTRVJfRkFJTF9JRighcGFyc2VWYXJVSW50MzIodW51c2VkKSwgImNhbid0
IGdldCBzZWNvbmQgaW1tZWRpYXRlIGZvciAiLCBvcCwgIiBpbiB1bnJlYWNoYWJsZSBjb250ZXh0
Iik7CkBAIC01NjgsNiArNTcyLDggQEAgYXV0byBGdW5jdGlvblBhcnNlcjxDb250ZXh0Pjo6cGFy
c2VVbnJlYQogICAgIGNhc2UgVGVlTG9jYWw6CiAgICAgY2FzZSBHZXRHbG9iYWw6CiAgICAgY2Fz
ZSBTZXRHbG9iYWw6CisgICAgY2FzZSBCcjoKKyAgICBjYXNlIEJySWY6CiAgICAgY2FzZSBDYWxs
OiB7CiAgICAgICAgIHVpbnQzMl90IHVudXNlZDsKICAgICAgICAgV0FTTV9QQVJTRVJfRkFJTF9J
RighcGFyc2VWYXJVSW50MzIodW51c2VkKSwgImNhbid0IGdldCBpbW1lZGlhdGUgZm9yICIsIG9w
LCAiIGluIHVucmVhY2hhYmxlIGNvbnRleHQiKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>