46083 2010-09-20 05:52:46 -0700 REGRESSION(67790): jsc tests are failed with YARR interpreter 2010-09-20 12:29:49 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) All All RESOLVED FIXED P2 Major --- 1 pvarga webkit-unassigned barraclough commit-queue joepeck msaboff ossy oldest_to_newest 281503 0 pvarga 2010-09-20 05:52:46 -0700 Some tests fail with YARR interpreter since r67790 (https://bugs.webkit.org/show_bug.cgi?id=45787): ecma_2/RegExp/exec-002.js ecma_3/RegExp/octal-001.js ecma_3/RegExp/perlstress-002.js ecma_3/RegExp/regress-103087.js ecma_3/RegExp/regress-165353.js ecma_3/RegExp/regress-169497.js ecma_3/RegExp/regress-191479.js ecma_3/RegExp/regress-216591.js ecma_3/RegExp/regress-24712.js ecma_3/RegExp/regress-85721.js js1_2/regexp/everything.js js1_2/regexp/interval.js js1_2/regexp/regress-9141.js 13 regressions found. These tests terminate by segmentation fault. I don't fully investigate the a problem yet but I think it is caused by the patch of the "Yarr JIT code checks BOL (^) each time through loop when in subexpression" bug. I created a simple testcase too: "a".match(/((a)*)/); 281511 1 68075 pvarga 2010-09-20 06:27:21 -0700 Created attachment 68075 proposed patch 281695 2 68075 ggaren 2010-09-20 11:31:43 -0700 Comment on attachment 68075 proposed patch r=me 281744 3 68075 commit-queue 2010-09-20 12:29:43 -0700 Comment on attachment 68075 proposed patch Clearing flags on attachment: 68075 Committed r67869: <http://trac.webkit.org/changeset/67869> 281745 4 commit-queue 2010-09-20 12:29:49 -0700 All reviewed patches have been landed. Closing bug. 68075 2010-09-20 06:27:21 -0700 2010-09-20 12:29:43 -0700 proposed patch 0001-fix-67790.patch text/plain 1672 pvarga RnJvbSAxZGY5MzM3NzhjNDZlZWQ0YmZkYTdkNmZlNGRkYTNlYzU4YTdmOTkwIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBQZXRlciBWYXJnYSA8cHZhcmdhQGluZi51LXN6ZWdlZC5odT4K RGF0ZTogTW9uLCAyMCBTZXAgMjAxMCAxNToyNjoxOSArMDIwMApTdWJqZWN0OiBbUEFUQ0hdIGZp eCA2Nzc5MAoKLS0tCiBKYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cgICAgICAgICAgICAgIHwgICAx MyArKysrKysrKysrKysrCiBKYXZhU2NyaXB0Q29yZS95YXJyL1JlZ2V4Q29tcGlsZXIuY3BwIHwg ICAgMiArLQogMiBmaWxlcyBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCAxIGRlbGV0aW9ucygt KQoKZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZyBiL0phdmFTY3JpcHRDb3Jl L0NoYW5nZUxvZwppbmRleCAzNDkwNzQ4Li5iZDgyMmE0IDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0 Q29yZS9DaGFuZ2VMb2cKKysrIGIvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEs MTYgQEAKKzIwMTAtMDktMjAgIFBldGVyIFZhcmdhICA8cHZhcmdhQGluZi51LXN6ZWdlZC5odT4K KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBSRUdSRVNT SU9OKDY3NzkwKToganNjIHRlc3RzIGFyZSBmYWlsZWQgd2l0aCBZQVJSIGludGVycHJldGVyCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00NjA4MworCisg ICAgICAgIEZpeCB0aGUgaW5pdGlhbGl6aW5nIG9mIHRoZSBsYXN0U3VicGF0dGVybklkIG1lbWJl ciBvZgorICAgICAgICBwYXJlbnRoZXNlcy4KKworICAgICAgICAqIHlhcnIvUmVnZXhDb21waWxl ci5jcHA6CisgICAgICAgIChKU0M6OllhcnI6OlJlZ2V4UGF0dGVybkNvbnN0cnVjdG9yOjphdG9t UGFyZW50aGVzZXNFbmQpOgorCiAyMDEwLTA5LTE5ICBHYXZpbiBCYXJyYWNsb3VnaCAgPGJhcnJh Y2xvdWdoQGFwcGxlLmNvbT4KIAogICAgICAgICBXaW5kb3dzIGJ1aWxkIGZpeCBwdCAyLgpkaWZm IC0tZ2l0IGEvSmF2YVNjcmlwdENvcmUveWFyci9SZWdleENvbXBpbGVyLmNwcCBiL0phdmFTY3Jp cHRDb3JlL3lhcnIvUmVnZXhDb21waWxlci5jcHAKaW5kZXggMzM0Mzc5Mi4uOWY5ZTAyOCAxMDA2 NDQKLS0tIGEvSmF2YVNjcmlwdENvcmUveWFyci9SZWdleENvbXBpbGVyLmNwcAorKysgYi9KYXZh U2NyaXB0Q29yZS95YXJyL1JlZ2V4Q29tcGlsZXIuY3BwCkBAIC0zNzUsNyArMzc1LDcgQEAgcHVi bGljOgogICAgICAgICBQYXR0ZXJuRGlzanVuY3Rpb24qIHBhcmVudGhlc2lzRGlzanVuY3Rpb24g PSBtX2FsdGVybmF0aXZlLT5tX3BhcmVudDsKICAgICAgICAgbV9hbHRlcm5hdGl2ZSA9IG1fYWx0 ZXJuYXRpdmUtPm1fcGFyZW50LT5tX3BhcmVudDsKIAotICAgICAgICBQYXR0ZXJuVGVybSBsYXN0 VGVybSA9IG1fYWx0ZXJuYXRpdmUtPmxhc3RUZXJtKCk7CisgICAgICAgIFBhdHRlcm5UZXJtJiBs YXN0VGVybSA9IG1fYWx0ZXJuYXRpdmUtPmxhc3RUZXJtKCk7CiAgICAgICAgIAogICAgICAgICB1 bnNpZ25lZCBudW1QYXJlbkFsdGVybmF0aXZlcyA9IHBhcmVudGhlc2lzRGlzanVuY3Rpb24tPm1f YWx0ZXJuYXRpdmVzLnNpemUoKTsKICAgICAgICAgdW5zaWduZWQgbnVtQk9MQW5jaG9yZWRBbHRz ID0gMDsKLS0gCjEuNy4yLjIKCg==