REOPENED224474
Reduce functionWithHellaArguments3()'s number of arguments from 5000 to 500. 
https://bugs.webkit.org/show_bug.cgi?id=224474
Summary Reduce functionWithHellaArguments3()'s number of arguments from 5000 to 500.
Mark Lam
Reported 2021-04-13 00:47:02 PDT
Using 5000 arguments seems excessive, and may blow out the stack on more resource constrained devices. 500 should be high enough. rdar://73614896
Attachments
proposed patch. (170.12 KB, patch)
2021-04-13 00:54 PDT, Mark Lam 		ysuzuki: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2021-04-13 00:54:38 PDT
Created attachment 425844 [details] proposed patch.
Yusuke Suzuki
Comment 2 2021-04-13 00:56:08 PDT
Comment on attachment 425844 [details] proposed patch. r=me
Mark Lam
Comment 3 2021-04-13 01:05:08 PDT
Thanks for the review. Landed in r275867: <http://trac.webkit.org/r275867>.
Saam Barati
Comment 4 2021-04-13 10:05:57 PDT
Comment on attachment 425844 [details] proposed patch. Why are we changing this? Did you go back in time to verify the test is still doing what it originally was testing?
Saam Barati
Comment 5 2021-04-13 10:07:09 PDT
(In reply to Saam Barati from comment #4) > Comment on attachment 425844 [details] > proposed patch. > > Why are we changing this? Did you go back in time to verify the test is > still doing what it originally was testing? I remember now something about some llvm bug. You could just JIT "functionWithHellaArguments" so that we're still testing what we intended to in B3/Air.
WebKit Commit Bot
Comment 6 2021-04-13 11:26:53 PDT
Re-opened since this is blocked by bug 224495
Note You need to log in before you can comment on or make changes to this bug.