...
Created attachment 440519 [details] patch
<rdar://83722151>
Created attachment 440520 [details] patch
Comment on attachment 440520 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=440520&action=review > JSTests/stress/reg-exp-exec-cant-prove-which-structure-it-produces-in-ai.js:6 > + let a = r.exec(/b/); Do you really want to match a RegExp (/b/) with r? Or does that somehow trigger the prior bug? > Source/JavaScriptCore/ChangeLog:9 > + Because of the RegExp.prototype.compile API, we don't know if the RegExp I think you want to change the last "RegExp" in this line to "Match result"
Comment on attachment 440520 [details] patch r=me
Comment on attachment 440520 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=440520&action=review >> JSTests/stress/reg-exp-exec-cant-prove-which-structure-it-produces-in-ai.js:6 >> + let a = r.exec(/b/); > > Do you really want to match a RegExp (/b/) with r? Or does that somehow trigger the prior bug? This is just the test the fuzzer came up with. >> Source/JavaScriptCore/ChangeLog:9 >> + Because of the RegExp.prototype.compile API, we don't know if the RegExp > > I think you want to change the last "RegExp" in this line to "Match result" Will fix.
Created attachment 440645 [details] patch for landing
Committed r283818 (242709@main): <https://commits.webkit.org/242709@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 440645 [details].
Comment on attachment 440520 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=440520&action=review >>> Source/JavaScriptCore/ChangeLog:9 >>> + Because of the RegExp.prototype.compile API, we don't know if the RegExp >> >> I think you want to change the last "RegExp" in this line to "Match result" > > Will fix. I forgot to do this 🤦♂️