264488 2023-11-09 07:46:48 -0800 Use SIMD in HasConstantValue() in BiquadDSPKernel 2023-11-13 14:00:16 -0800 1 1 1 Unclassified WebKit Web Audio Other Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 uioptt24 cdumez cdumez uioptt24 webkit-bug-importer oldest_to_newest 1991154 0 uioptt24 2023-11-09 07:46:48 -0800 Use SIMD to optimize this. This would speed up processing by a factor of 4 because we can process 4 floats at a time. 1991165 1 cdumez 2023-11-09 08:07:29 -0800 Pull request: https://github.com/WebKit/WebKit/pull/20233 1991166 2 uioptt24 2023-11-09 08:14:57 -0800 (In reply to Chris Dumez from comment #1) > Pull request: https://github.com/WebKit/WebKit/pull/20233 Thank you for taking my patch from Blink. I originally wanted to implement it in Webkit myself, but you have done it for me, so I won't submit a new PR on this issue for the time being lol 1991170 3 cdumez 2023-11-09 08:17:09 -0800 (In reply to Ho Cheung from comment #2) > (In reply to Chris Dumez from comment #1) > > Pull request: https://github.com/WebKit/WebKit/pull/20233 > > Thank you for taking my patch from Blink. I originally wanted to implement > it in Webkit myself, but you have done it for me, so I won't submit a new PR > on this issue for the time being lol Sorry, I realized too late you were the patch author on Blink side. It wasn't clear when you filed the bug that you intended on actually making the code change yourself. If you end up making more contributions to WebKit (and I hope you do), I encourage you to make it clearer in the bug description that a patch is incoming. 1991173 4 uioptt24 2023-11-09 08:21:39 -0800 (In reply to Chris Dumez from comment #3) > (In reply to Ho Cheung from comment #2) > > (In reply to Chris Dumez from comment #1) > > > Pull request: https://github.com/WebKit/WebKit/pull/20233 > > > > Thank you for taking my patch from Blink. I originally wanted to implement > > it in Webkit myself, but you have done it for me, so I won't submit a new PR > > on this issue for the time being lol > > Sorry, I realized too late you were the patch author on Blink side. It > wasn't clear when you filed the bug that you intended on actually making the > code change yourself. > > If you end up making more contributions to WebKit (and I hope you do), I > encourage you to make it clearer in the bug description that a patch is > incoming. This is my first time submitting a bug report to WebKit. Thank you for your suggestions and help. Another question I would like to ask is that on the Blink side, I wrote a unit test for this modification. Does this also apply to WebKit? https://chromium-review.googlesource.com/c/chromium/src/+/4622707 1991181 5 cdumez 2023-11-09 08:31:33 -0800 (In reply to Ho Cheung from comment #4) > (In reply to Chris Dumez from comment #3) > > (In reply to Ho Cheung from comment #2) > > > (In reply to Chris Dumez from comment #1) > > > > Pull request: https://github.com/WebKit/WebKit/pull/20233 > > > > > > Thank you for taking my patch from Blink. I originally wanted to implement > > > it in Webkit myself, but you have done it for me, so I won't submit a new PR > > > on this issue for the time being lol > > > > Sorry, I realized too late you were the patch author on Blink side. It > > wasn't clear when you filed the bug that you intended on actually making the > > code change yourself. > > > > If you end up making more contributions to WebKit (and I hope you do), I > > encourage you to make it clearer in the bug description that a patch is > > incoming. > > This is my first time submitting a bug report to WebKit. Thank you for your > suggestions and help. > > Another question I would like to ask is that on the Blink side, I wrote a > unit test for this modification. Does this also apply to WebKit? > > https://chromium-review.googlesource.com/c/chromium/src/+/4622707 Oh, I missed that. In general, yes but API tests are located in a different place in WebKit: - Tools/TestWebKitAPI/Tests/ or Tools/TestWebKitAPI/Tests/WebCore in this case. I'll look into updating my PR to include your test soon. 1992259 6 ews-feeder 2023-11-13 13:59:54 -0800 Committed 270669@main (0a313d9769ba): <https://commits.webkit.org/270669@main> Reviewed commits have been landed. Closing PR #20233 and removing active labels. 1992260 7 webkit-bug-importer 2023-11-13 14:00:16 -0800 <rdar://problem/118354762>