[webkit-changes] [WebKit/WebKit] 4b7d24: [Apple-Ventura-AppleSilicon-Debug-JSC-Tests] ASSER...
Dan Hecht
noreply at github.com
Fri May 17 21:56:16 PDT 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 4b7d246d6d632f5297784b401e3d30153ac55bc7
https://github.com/WebKit/WebKit/commit/4b7d246d6d632f5297784b401e3d30153ac55bc7
Author: Dan Hecht <dan.hecht at apple.com>
Date: 2024-05-17 (Fri, 17 May 2024)
Changed paths:
M Source/JavaScriptCore/dfg/DFGOperations.cpp
Log Message:
-----------
[Apple-Ventura-AppleSilicon-Debug-JSC-Tests] ASSERTION FAILED: !m_needExceptionCheck in multiple tests in ChakraCore.yaml/ChakraCore/test
https://bugs.webkit.org/show_bug.cgi?id=274281
rdar://128228200
Reviewed by Keith Miller.
toStringOrNull() can throw an exception but no exception check occurs before calling
testInline() which can also throw. However, according to the code in the similar func
operationRegExpTestGeneric(), toStringOrNull() only throws when !input, so we are
actually safe and would always bail on that path anyway. Document this and placate
the exception checker by adding an assert.
* Source/JavaScriptCore/dfg/DFGOperations.cpp:
(JSC::DFG::JSC_DEFINE_JIT_OPERATION):
Canonical link: https://commits.webkit.org/278947@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list