[JSC] Strictly annotate pointers with TrustedImmPtr in CCallHelpers
Created attachment 456731 [details] Patch
Created attachment 456734 [details] Patch
Created attachment 456738 [details] Patch
Comment on attachment 456738 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456738&action=review r=me > Source/JavaScriptCore/dfg/DFGArrayifySlowPathGenerator.h:97 > + jit->callOperation(operationEnsureInt32, m_tempGPR, SpeculativeJIT::TrustedImmPtr(&vm), m_baseGPR); Why "SpeculativeJIT::" instead of "MacroAssembler::"? We use "MacroAssembler::" above. Would be good to be consistent everywhere. Ditto below. > Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:3528 > + jit.loadDouble(SpeculativeJIT::TrustedImmPtr(&zero), scratch); Why not use MacroAssembler:: qualifier instead?
Comment on attachment 456738 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456738&action=review >> Source/JavaScriptCore/dfg/DFGArrayifySlowPathGenerator.h:97 >> + jit->callOperation(operationEnsureInt32, m_tempGPR, SpeculativeJIT::TrustedImmPtr(&vm), m_baseGPR); > > Why "SpeculativeJIT::" instead of "MacroAssembler::"? We use "MacroAssembler::" above. Would be good to be consistent everywhere. Ditto below. Because SpeculativeJIT::TrustedImmPtr is different from MacroAssembler::TrustedImmPtr. It has some more extra checks. >> Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:3528 >> + jit.loadDouble(SpeculativeJIT::TrustedImmPtr(&zero), scratch); > > Why not use MacroAssembler:: qualifier instead? Ditto.
Committed r292445 (?): <https://commits.webkit.org/r292445>
<rdar://problem/91327271>