125727 2013-12-13 20:42:03 -0800 FTL should know the actual frame size of LLVM generated code 2017-04-24 19:12:03 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) All All REOPENED P2 Normal --- 113621 1 fpizlo fpizlo atrick barraclough basile_clement beidson ggaren mark.lam mhahnenberg msaboff nrotem oliver sam oldest_to_newest 959758 0 fpizlo 2013-12-13 20:42:03 -0800 For now, we'll just guess that it's 1000 slots (slots = 8 bytes). 973965 1 fpizlo 2014-01-30 13:29:18 -0800 This will just get fixed when we do the real stack check. *** This bug has been marked as a duplicate of bug 125650 *** 1123267 2 basile_clement 2015-09-03 18:49:52 -0700 DFGCommon::frameRegisterCount is still 1000. 1123459 3 260592 basile_clement 2015-09-04 09:37:23 -0700 Created attachment 260592 Patch 1300823 4 260592 beidson 2017-04-24 19:09:34 -0700 Comment on attachment 260592 Patch This patch has been pending review since 2015 with no recent activity. It seems unlikely that it would even still apply to trunk in its current form. Clearing from the review queue. Feel free to update and resubmit if the patch is still relevant. 260592 2015-09-04 09:37:23 -0700 2017-04-24 19:12:03 -0700 Patch file_125727.txt text/plain 1401 basile_clement ZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvSmF2 YVNjcmlwdENvcmUvQ2hhbmdlTG9nDQppbmRleCA3NGMwMDlmLi5hODFmZDNmIDEwMDY0NA0KLS0t IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZw0KKysrIGIvU291cmNlL0phdmFTY3Jp cHRDb3JlL0NoYW5nZUxvZw0KQEAgLTEsMyArMSwxMyBAQA0KKzIwMTUtMDktMDQgIEJhc2lsZSBD bGVtZW50ICA8YmFzaWxlX2NsZW1lbnRAYXBwbGUuY29tPg0KKw0KKyAgICAgICAgRlRMIHNob3Vs ZCBrbm93IHRoZSBhY3R1YWwgZnJhbWUgc2l6ZSBvZiBMTFZNIGdlbmVyYXRlZCBjb2RlDQorICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI1NzI3DQorDQor ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4NCisNCisgICAgICAgICogZnRsL0ZU TExpbmsuY3BwOg0KKyAgICAgICAgKEpTQzo6RlRMOjpsaW5rKToNCisNCiAyMDE1LTA5LTA0ICBC YXNpbGUgQ2xlbWVudCAgPGJhc2lsZV9jbGVtZW50QGFwcGxlLmNvbT4NCiANCiAgICAgICAgIFNp bmNlIHIxODkzNDEsIGVzNi55YW1sL2VzNi9hcnJvd19mdW5jdGlvbnNfbm9fcHJvdG90eXBlX3By b3BlcnR5LmpzIGlzIGV4cGVjdGVkIHRvIHN1Y2NlZWQNCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2 YVNjcmlwdENvcmUvZnRsL0ZUTExpbmsuY3BwIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Z0bC9G VExMaW5rLmNwcA0KaW5kZXggZmE5MzRjMy4uNDIzMmU3NCAxMDA2NDQNCi0tLSBhL1NvdXJjZS9K YXZhU2NyaXB0Q29yZS9mdGwvRlRMTGluay5jcHANCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29y ZS9mdGwvRlRMTGluay5jcHANCkBAIC01NCw5ICs1NCw3IEBAIHZvaWQgbGluayhTdGF0ZSYgc3Rh dGUpDQogICAgIC8vIExMVk0gd2lsbCBjcmVhdGUgaXRzIG93biBqdW1wIHRhYmxlcyBhcyBuZWVk ZWQuDQogICAgIGNvZGVCbG9jay0+Y2xlYXJTd2l0Y2hKdW1wVGFibGVzKCk7DQogICAgIA0KLSAg ICAvLyBGSVhNRTogTmVlZCB0byBrbm93IHRoZSByZWFsIGZyYW1lIHJlZ2lzdGVyIGNvdW50Lg0K LSAgICAvLyBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI1NzI3DQot ICAgIHN0YXRlLmppdENvZGUtPmNvbW1vbi5mcmFtZVJlZ2lzdGVyQ291bnQgPSAxMDAwOw0KKyAg ICBzdGF0ZS5qaXRDb2RlLT5jb21tb24uZnJhbWVSZWdpc3RlckNvdW50ID0gc3RhdGUuaml0Q29k ZS0+c3RhY2ttYXBzLnN0YWNrU2l6ZUZvckxvY2FscygpOw0KICAgICANCiAgICAgc3RhdGUuaml0 Q29kZS0+Y29tbW9uLnJlcXVpcmVkUmVnaXN0ZXJDb3VudEZvckV4aXQgPSBncmFwaC5yZXF1aXJl ZFJlZ2lzdGVyQ291bnRGb3JFeGl0KCk7DQogICAgIA0K