<?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>108292</bug_id>
          
          <creation_ts>2013-01-29 21:37:47 -0800</creation_ts>
          <short_desc>Be a little more conservative about emitting table-based switches</short_desc>
          <delta_ts>2013-01-31 07:52:57 -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>New Bugs</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>108487</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Geoffrey Garen">ggaren</reporter>
          <assigned_to name="Geoffrey Garen">ggaren</assigned_to>
          <cc>fpizlo</cc>
    
    <cc>oliver</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>819776</commentid>
    <comment_count>0</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2013-01-29 21:37:47 -0800</bug_when>
    <thetext>Be a little more conservative about emitting table-based switches</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>819778</commentid>
    <comment_count>1</comment_count>
      <attachid>185394</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2013-01-29 21:39:57 -0800</bug_when>
    <thetext>Created attachment 185394
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>819780</commentid>
    <comment_count>2</comment_count>
      <attachid>185395</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2013-01-29 21:40:48 -0800</bug_when>
    <thetext>Created attachment 185395
performance</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>819781</commentid>
    <comment_count>3</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2013-01-29 21:45:10 -0800</bug_when>
    <thetext>Committed r141222: &lt;http://trac.webkit.org/changeset/141222&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>185394</attachid>
            <date>2013-01-29 21:39:57 -0800</date>
            <delta_ts>2013-01-29 21:42:41 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-108292-20130129213643.patch</filename>
            <type>text/plain</type>
            <size>3333</size>
            <attacher name="Geoffrey Garen">ggaren</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTQxMjIxKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIwIEBA
CisyMDEzLTAxLTI5ICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CisKKyAgICAg
ICAgQmUgYSBsaXR0bGUgbW9yZSBjb25zZXJ2YXRpdmUgYWJvdXQgZW1pdHRpbmcgdGFibGUtYmFz
ZWQgc3dpdGNoZXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTEwODI5MgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgIFByb2ZpbGluZyBzaG93cyB3ZSdyZSB1c2luZyBvcF9zd2l0Y2ggaW4gY2FzZXMgd2hlcmUg
aXQncyBhIHJlZ3Jlc3Npb24uCisKKyAgICAgICAgKiBieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2Vu
LmNwcDoKKyAgICAgICAgKEpTQyk6CisgICAgICAgIChKU0M6Omxlbmd0aCk6CisgICAgICAgIChK
U0M6OkNhc2VCbG9ja05vZGU6OnRyeVRhYmxlU3dpdGNoKToKKyAgICAgICAgKEpTQzo6Q2FzZUJs
b2NrTm9kZTo6ZW1pdEJ5dGVjb2RlRm9yQmxvY2spOgorICAgICAgICAqIHBhcnNlci9Ob2Rlcy5o
OgorICAgICAgICAoQ2FzZUJsb2NrTm9kZSk6CisKIDIwMTMtMDEtMjkgIFNoZXJpZmYgQm90ICA8
d2Via2l0LnJldmlldy5ib3RAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIHJvbGxp
bmcgb3V0IHIxNDA5ODMuCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvYnl0ZWNvbXBpbGVy
L05vZGVzQ29kZWdlbi5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL2J5
dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4uY3BwCShyZXZpc2lvbiAxNDExOTEpCisrKyBTb3VyY2Uv
SmF2YVNjcmlwdENvcmUvYnl0ZWNvbXBpbGVyL05vZGVzQ29kZWdlbi5jcHAJKHdvcmtpbmcgY29w
eSkKQEAgLTE4NTEsOSArMTg1MSwyMiBAQCBzdGF0aWMgdm9pZCBwcm9jZXNzQ2xhdXNlTGlzdChD
bGF1c2VMaXN0CiAgICAgICAgIGJyZWFrOyAgICAgICAgCiAgICAgfQogfQotICAgIAotU3dpdGNo
SW5mbzo6U3dpdGNoVHlwZSBDYXNlQmxvY2tOb2RlOjp0cnlPcHRpbWl6ZWRTd2l0Y2goVmVjdG9y
PEV4cHJlc3Npb25Ob2RlKiwgOD4mIGxpdGVyYWxWZWN0b3IsIGludDMyX3QmIG1pbl9udW0sIGlu
dDMyX3QmIG1heF9udW0pCisKK3N0YXRpYyBpbmxpbmUgc2l6ZV90IGxlbmd0aChDbGF1c2VMaXN0
Tm9kZSogbGlzdDEsIENsYXVzZUxpc3ROb2RlKiBsaXN0MikKIHsKKyAgICBzaXplX3QgbGVuZ3Ro
ID0gMDsKKyAgICBmb3IgKENsYXVzZUxpc3ROb2RlKiBub2RlID0gbGlzdDE7IG5vZGU7IG5vZGUg
PSBub2RlLT5nZXROZXh0KCkpCisgICAgICAgICsrbGVuZ3RoOworICAgIGZvciAoQ2xhdXNlTGlz
dE5vZGUqIG5vZGUgPSBsaXN0Mjsgbm9kZTsgbm9kZSA9IG5vZGUtPmdldE5leHQoKSkKKyAgICAg
ICAgKytsZW5ndGg7CisgICAgcmV0dXJuIGxlbmd0aDsKK30KKworU3dpdGNoSW5mbzo6U3dpdGNo
VHlwZSBDYXNlQmxvY2tOb2RlOjp0cnlUYWJsZVN3aXRjaChWZWN0b3I8RXhwcmVzc2lvbk5vZGUq
LCA4PiYgbGl0ZXJhbFZlY3RvciwgaW50MzJfdCYgbWluX251bSwgaW50MzJfdCYgbWF4X251bSkK
K3sKKyAgICBpZiAobGVuZ3RoKG1fbGlzdDEsIG1fbGlzdDIpIDwgc190YWJsZVN3aXRjaE1pbmlt
dW0pCisgICAgICAgIHJldHVybiBTd2l0Y2hJbmZvOjpTd2l0Y2hOb25lOworCiAgICAgU3dpdGNo
S2luZCB0eXBlRm9yVGFibGUgPSBTd2l0Y2hVbnNldDsKICAgICBib29sIHNpbmdsZUNoYXJhY3Rl
clN3aXRjaCA9IHRydWU7CiAgICAgCkBAIC0xODg4LDcgKzE5MDEsNyBAQCBSZWdpc3RlcklEKiBD
YXNlQmxvY2tOb2RlOjplbWl0Qnl0ZWNvZGVGCiAgICAgVmVjdG9yPEV4cHJlc3Npb25Ob2RlKiwg
OD4gbGl0ZXJhbFZlY3RvcjsKICAgICBpbnQzMl90IG1pbl9udW0gPSBzdGQ6Om51bWVyaWNfbGlt
aXRzPGludDMyX3Q+OjptYXgoKTsKICAgICBpbnQzMl90IG1heF9udW0gPSBzdGQ6Om51bWVyaWNf
bGltaXRzPGludDMyX3Q+OjptaW4oKTsKLSAgICBTd2l0Y2hJbmZvOjpTd2l0Y2hUeXBlIHN3aXRj
aFR5cGUgPSB0cnlPcHRpbWl6ZWRTd2l0Y2gobGl0ZXJhbFZlY3RvciwgbWluX251bSwgbWF4X251
bSk7CisgICAgU3dpdGNoSW5mbzo6U3dpdGNoVHlwZSBzd2l0Y2hUeXBlID0gdHJ5VGFibGVTd2l0
Y2gobGl0ZXJhbFZlY3RvciwgbWluX251bSwgbWF4X251bSk7CiAKICAgICBpZiAoc3dpdGNoVHlw
ZSAhPSBTd2l0Y2hJbmZvOjpTd2l0Y2hOb25lKSB7CiAgICAgICAgIC8vIFByZXBhcmUgdGhlIHZh
cmlvdXMgbGFiZWxzCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvcGFyc2VyL05vZGVzLmgK
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL3BhcnNlci9Ob2Rlcy5oCShyZXZp
c2lvbiAxNDExOTEpCisrKyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvcGFyc2VyL05vZGVzLmgJKHdv
cmtpbmcgY29weSkKQEAgLTE1MDksNyArMTUwOSw4IEBAIG5hbWVzcGFjZSBKU0MgewogICAgICAg
ICBSZWdpc3RlcklEKiBlbWl0Qnl0ZWNvZGVGb3JCbG9jayhCeXRlY29kZUdlbmVyYXRvciYsIFJl
Z2lzdGVySUQqIGlucHV0LCBSZWdpc3RlcklEKiBkZXN0aW5hdGlvbik7CiAKICAgICBwcml2YXRl
OgotICAgICAgICBTd2l0Y2hJbmZvOjpTd2l0Y2hUeXBlIHRyeU9wdGltaXplZFN3aXRjaChWZWN0
b3I8RXhwcmVzc2lvbk5vZGUqLCA4PiYgbGl0ZXJhbFZlY3RvciwgaW50MzJfdCYgbWluX251bSwg
aW50MzJfdCYgbWF4X251bSk7CisgICAgICAgIFN3aXRjaEluZm86OlN3aXRjaFR5cGUgdHJ5VGFi
bGVTd2l0Y2goVmVjdG9yPEV4cHJlc3Npb25Ob2RlKiwgOD4mIGxpdGVyYWxWZWN0b3IsIGludDMy
X3QmIG1pbl9udW0sIGludDMyX3QmIG1heF9udW0pOworICAgICAgICBzdGF0aWMgY29uc3Qgc2l6
ZV90IHNfdGFibGVTd2l0Y2hNaW5pbXVtID0gMTA7CiAgICAgICAgIENsYXVzZUxpc3ROb2RlKiBt
X2xpc3QxOwogICAgICAgICBDYXNlQ2xhdXNlTm9kZSogbV9kZWZhdWx0Q2xhdXNlOwogICAgICAg
ICBDbGF1c2VMaXN0Tm9kZSogbV9saXN0MjsK
</data>
<flag name="review"
          id="204433"
          type_id="1"
          status="+"
          setter="fpizlo"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>185395</attachid>
            <date>2013-01-29 21:40:48 -0800</date>
            <delta_ts>2013-01-29 21:40:48 -0800</delta_ts>
            <desc>performance</desc>
            <filename>BASE_PATCH_Octane_garen_20130129_2055_report.txt</filename>
            <type>text/plain</type>
            <size>2857</size>
            <attacher name="Geoffrey Garen">ggaren</attacher>
            
              <data encoding="base64">QmVuY2htYXJrIHJlcG9ydCBmb3IgT2N0YW5lIG9uIGdhcmVuIChNYWNQcm81LDEpLgoKVk1zIHRl
c3RlZDoKIkJBU0UiIGF0IC9Wb2x1bWVzL0JpZy9nZ2FyZW4vYmFzZWxpbmUtd2Via2l0L1dlYktp
dEJ1aWxkL1JlbGVhc2UvRHVtcFJlbmRlclRyZWUgKHIxNDExOTEpCiJQQVRDSCIgYXQgL1ZvbHVt
ZXMvQmlnL2dnYXJlbi93ZWJraXQvV2ViS2l0QnVpbGQvUmVsZWFzZS9EdW1wUmVuZGVyVHJlZSAo
cjE0MTE5MSkKCkNvbGxlY3RlZCAxMiBzYW1wbGVzIHBlciBiZW5jaG1hcmsvVk0sIHdpdGggNCBW
TSBpbnZvY2F0aW9ucyBwZXIgYmVuY2htYXJrLiBFbWl0dGVkIGEgY2FsbCB0byBnYygpCmJldHdl
ZW4gc2FtcGxlIG1lYXN1cmVtZW50cy4gVXNlZCAxIGJlbmNobWFyayBpdGVyYXRpb24gcGVyIFZN
IGludm9jYXRpb24gZm9yIHdhcm0tdXAuIFVzZWQgdGhlCmpzYy1zcGVjaWZpYyBwcmVjaXNlVGlt
ZSgpIGZ1bmN0aW9uIHRvIGdldCBtaWNyb3NlY29uZC1sZXZlbCB0aW1pbmcuIFJlcG9ydGluZyBi
ZW5jaG1hcmsgZXhlY3V0aW9uCnRpbWVzIHdpdGggOTUlIGNvbmZpZGVuY2UgaW50ZXJ2YWxzIGlu
IG1pbGxpc2Vjb25kcy4KCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIEJBU0UgICAg
ICAgICAgICAgICAgICAgICBQQVRDSCAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgIApPY3RhbmUgYW5kIFY4djc6CiAgIGVuY3J5cHQgICAgICAgICAgICAgICAgIDAuNDQxMTYr
LTAuMDAwNzcgICAgPyAgICAgMC40NDE5NistMC4wMDA4MSAgICAgICA/CiAgIGRlY3J5cHQgICAg
ICAgICAgICAgICAgIDcuODc3NTUrLTAuMDE3MzggICAgISAgICAgNy45MjUwMCstMC4wMTkxMCAg
ICAgICAhIGRlZmluaXRlbHkgMS4wMDYweCBzbG93ZXIKICAgZGVsdGFibHVlICAgICAgeDIgICAg
ICAgMC41ODMzMSstMC4wMDU0NCAgICAgICAgICAwLjU3NzMyKy0wLjAwNDA2ICAgICAgICAgbWln
aHQgYmUgMS4wMTA0eCBmYXN0ZXIKICAgZWFybGV5ICAgICAgICAgICAgICAgICAgMC44ODI0Mist
MC4wMTMzMyAgICA/ICAgICAwLjg4ODc5Ky0wLjAwNTg0ICAgICAgID8KICAgYm95ZXIgICAgICAg
ICAgICAgICAgICAxMi4zNDI2MCstMC4wMjkwMyAgICAhICAgIDEyLjU4NDYyKy0wLjA1Mjc2ICAg
ICAgICEgZGVmaW5pdGVseSAxLjAxOTZ4IHNsb3dlcgogICByYXl0cmFjZSAgICAgICB4MiAgICAg
ICA0LjkxMDE0Ky0wLjI5MDcxICAgIF4gICAgIDQuNDk0NjQrLTAuMDM3MTYgICAgICAgXiBkZWZp
bml0ZWx5IDEuMDkyNHggZmFzdGVyCiAgIHJlZ2V4cCAgICAgICAgIHgyICAgICAgMjkuNzQ1NzUr
LTAuMDkwODkgICAgISAgICAyOS45OTgyNistMC4xMTE4OCAgICAgICAhIGRlZmluaXRlbHkgMS4w
MDg1eCBzbG93ZXIKICAgcmljaGFyZHMgICAgICAgeDIgICAgICAgMC4yODYyMystMC4wMDEzNCAg
ICAgICAgICAwLjI4NDcxKy0wLjAwMTQwICAgICAgIAogICBzcGxheSAgICAgICAgICB4MiAgICAg
ICAwLjY5Mzg1Ky0wLjAwNjk4ICAgID8gICAgIDAuNjk4MTIrLTAuMDIxNzAgICAgICAgPwogICBu
YXZpZXItc3Rva2VzICB4MiAgICAgIDExLjUwNzQ2Ky0wLjAyMjE2ICAgID8gICAgMTEuNTE1ODUr
LTAuMDIzNDkgICAgICAgPwogICBjbG9zdXJlICAgICAgICAgICAgICAgICAwLjUxMDQwKy0wLjAw
MTg2ICAgICEgICAgIDAuNTE3MDkrLTAuMDAzNjEgICAgICAgISBkZWZpbml0ZWx5IDEuMDEzMXgg
c2xvd2VyCiAgIGpxdWVyeSAgICAgICAgICAgICAgICAgIDYuNzU1MTcrLTAuMDczMTkgICAgPyAg
ICAgNi44NjAyOSstMC4xMjU2OCAgICAgICA/IG1pZ2h0IGJlIDEuMDE1Nnggc2xvd2VyCiAgIGdi
ZW11ICAgICAgICAgIHgyICAgICAyNjIuMjc5NjQrLTQuNjY3MjYgICAgXiAgIDI1MS42MDM3OCst
Mi42NDQ3NSAgICAgICBeIGRlZmluaXRlbHkgMS4wNDI0eCBmYXN0ZXIKICAgbWFuZHJlZWwgICAg
ICAgeDIgICAgIDE4My4wMjYxOSstMS4xNDg0NiAgICBeICAgMTcyLjMxODI5Ky0wLjYyNjkxICAg
ICAgIF4gZGVmaW5pdGVseSAxLjA2MjF4IGZhc3RlcgogICBwZGZqcyAgICAgICAgICB4MiAgICAg
MTMyLjMyNTMyKy0wLjYzODU4ICAgIF4gICAxMjcuNjY1ODQrLTAuMzc1NTkgICAgICAgXiBkZWZp
bml0ZWx5IDEuMDM2NXggZmFzdGVyCiAgIGJveDJkICAgICAgICAgIHgyICAgICAgMzcuMjA2ODYr
LTAuMTIyMDcgICAgICAgICAzNi45Mzc2MSstMC4yMTI1MSAgICAgICAKClY4djc6CiAgIDxhcml0
aG1ldGljPiAgICAgICAgICAgIDcuMzEyMzIrLTAuMDM5MzIgICAgICAgICAgNy4zMTExNCstMC4w
MTQyOCAgICAgICAgIG1pZ2h0IGJlIDEuMDAwMnggZmFzdGVyCiAgIDxnZW9tZXRyaWM+ICogICAg
ICAgICAgIDIuNDI0NDYrLTAuMDE5MzYgICAgICAgICAgMi40MDM1OSstMC4wMTExMCAgICAgICAg
IG1pZ2h0IGJlIDEuMDA4N3ggZmFzdGVyCiAgIDxoYXJtb25pYz4gICAgICAgICAgICAgIDAuOTEx
MTkrLTAuMDAzMjYgICAgICAgICAgMC45MDcxMistMC4wMDUxOCAgICAgICAgIG1pZ2h0IGJlIDEu
MDA0NXggZmFzdGVyCgpPY3RhbmUgaW5jbHVkaW5nIFY4djc6CiAgIDxhcml0aG1ldGljPiAgICAg
ICAgICAgNTIuMDc0NTcrLTAuMzc1NDIgICAgXiAgICA1MC4wNTQxMCstMC4yMjI2MiAgICAgICBe
IGRlZmluaXRlbHkgMS4wNDA0eCBmYXN0ZXIKICAgPGdlb21ldHJpYz4gKiAgICAgICAgICAgNy45
Njk4OCstMC4wMzM0MyAgICBeICAgICA3Ljg0ODUwKy0wLjAyNDIzICAgICAgIF4gZGVmaW5pdGVs
eSAxLjAxNTV4IGZhc3RlcgogICA8aGFybW9uaWM+ICAgICAgICAgICAgICAxLjMxNjE3Ky0wLjAw
NDAyICAgICAgICAgIDEuMzEyNjMrLTAuMDA3MDggICAgICAgICBtaWdodCBiZSAxLjAwMjd4IGZh
c3RlcgoKCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>