|
| 1 | +2016-11-26 Yusuke Suzuki < [email protected]> |
| 2 | + |
| 3 | + [WTF] Import std::optional reference implementation as WTF::Optional |
| 4 | + https://bugs.webkit.org/show_bug.cgi?id=164199 |
| 5 | + |
| 6 | + Reviewed by Saam Barati and Sam Weinig. |
| 7 | + |
| 8 | + Previous WTF::Optional::operator= is not compatible to std::optional::operator=. |
| 9 | + std::optional::emplace has the same semantics to the previous one. |
| 10 | + So we change the code to use it. |
| 11 | + |
| 12 | + * Scripts/builtins/builtins_templates.py: |
| 13 | + * Scripts/tests/builtins/expected/JavaScriptCore-Builtin.Promise-Combined.js-result: |
| 14 | + * Scripts/tests/builtins/expected/JavaScriptCore-Builtin.Promise-Separate.js-result: |
| 15 | + * Scripts/tests/builtins/expected/JavaScriptCore-Builtin.prototype-Combined.js-result: |
| 16 | + * Scripts/tests/builtins/expected/JavaScriptCore-Builtin.prototype-Separate.js-result: |
| 17 | + * Scripts/tests/builtins/expected/JavaScriptCore-BuiltinConstructor-Combined.js-result: |
| 18 | + * Scripts/tests/builtins/expected/JavaScriptCore-BuiltinConstructor-Separate.js-result: |
| 19 | + * Scripts/tests/builtins/expected/JavaScriptCore-InternalClashingNames-Combined.js-result: |
| 20 | + * Scripts/tests/builtins/expected/WebCore-AnotherGuardedInternalBuiltin-Separate.js-result: |
| 21 | + * Scripts/tests/builtins/expected/WebCore-ArbitraryConditionalGuard-Separate.js-result: |
| 22 | + * Scripts/tests/builtins/expected/WebCore-GuardedBuiltin-Separate.js-result: |
| 23 | + * Scripts/tests/builtins/expected/WebCore-GuardedInternalBuiltin-Separate.js-result: |
| 24 | + * Scripts/tests/builtins/expected/WebCore-UnguardedBuiltin-Separate.js-result: |
| 25 | + * Scripts/tests/builtins/expected/WebCore-xmlCasingTest-Separate.js-result: |
| 26 | + * assembler/MacroAssemblerARM64.h: |
| 27 | + (JSC::MacroAssemblerARM64::commuteCompareToZeroIntoTest): |
| 28 | + * assembler/MacroAssemblerX86Common.h: |
| 29 | + (JSC::MacroAssemblerX86Common::commuteCompareToZeroIntoTest): |
| 30 | + * b3/B3CheckSpecial.cpp: |
| 31 | + (JSC::B3::CheckSpecial::forEachArg): |
| 32 | + (JSC::B3::CheckSpecial::shouldTryAliasingDef): |
| 33 | + * b3/B3CheckSpecial.h: |
| 34 | + * b3/B3LowerToAir.cpp: |
| 35 | + (JSC::B3::Air::LowerToAir::scaleForShl): |
| 36 | + (JSC::B3::Air::LowerToAir::effectiveAddr): |
| 37 | + (JSC::B3::Air::LowerToAir::tryAppendLea): |
| 38 | + * b3/B3Opcode.cpp: |
| 39 | + (JSC::B3::invertedCompare): |
| 40 | + * b3/B3Opcode.h: |
| 41 | + * b3/B3PatchpointSpecial.cpp: |
| 42 | + (JSC::B3::PatchpointSpecial::forEachArg): |
| 43 | + * b3/B3StackmapSpecial.cpp: |
| 44 | + (JSC::B3::StackmapSpecial::forEachArgImpl): |
| 45 | + * b3/B3StackmapSpecial.h: |
| 46 | + * b3/B3Value.cpp: |
| 47 | + (JSC::B3::Value::invertedCompare): |
| 48 | + * b3/air/AirArg.h: |
| 49 | + (JSC::B3::Air::Arg::isValidScale): |
| 50 | + (JSC::B3::Air::Arg::isValidAddrForm): |
| 51 | + (JSC::B3::Air::Arg::isValidIndexForm): |
| 52 | + (JSC::B3::Air::Arg::isValidForm): |
| 53 | + * b3/air/AirCustom.h: |
| 54 | + (JSC::B3::Air::PatchCustom::shouldTryAliasingDef): |
| 55 | + * b3/air/AirFixObviousSpills.cpp: |
| 56 | + * b3/air/AirInst.h: |
| 57 | + * b3/air/AirInstInlines.h: |
| 58 | + (JSC::B3::Air::Inst::shouldTryAliasingDef): |
| 59 | + * b3/air/AirIteratedRegisterCoalescing.cpp: |
| 60 | + * b3/air/AirSpecial.cpp: |
| 61 | + (JSC::B3::Air::Special::shouldTryAliasingDef): |
| 62 | + * b3/air/AirSpecial.h: |
| 63 | + * bytecode/BytecodeGeneratorification.cpp: |
| 64 | + (JSC::BytecodeGeneratorification::storageForGeneratorLocal): |
| 65 | + * bytecode/CodeBlock.cpp: |
| 66 | + (JSC::CodeBlock::findPC): |
| 67 | + (JSC::CodeBlock::bytecodeOffsetFromCallSiteIndex): |
| 68 | + * bytecode/CodeBlock.h: |
| 69 | + * bytecode/UnlinkedFunctionExecutable.cpp: |
| 70 | + (JSC::UnlinkedFunctionExecutable::link): |
| 71 | + * bytecode/UnlinkedFunctionExecutable.h: |
| 72 | + * bytecompiler/BytecodeGenerator.h: |
| 73 | + * bytecompiler/NodesCodegen.cpp: |
| 74 | + (JSC::PropertyListNode::emitPutConstantProperty): |
| 75 | + (JSC::ObjectPatternNode::bindValue): |
| 76 | + * debugger/Debugger.cpp: |
| 77 | + (JSC::Debugger::resolveBreakpoint): |
| 78 | + * debugger/DebuggerCallFrame.cpp: |
| 79 | + (JSC::DebuggerCallFrame::currentPosition): |
| 80 | + * debugger/DebuggerParseData.cpp: |
| 81 | + (JSC::DebuggerPausePositions::breakpointLocationForLineColumn): |
| 82 | + * debugger/DebuggerParseData.h: |
| 83 | + * debugger/ScriptProfilingScope.h: |
| 84 | + * dfg/DFGAbstractInterpreterInlines.h: |
| 85 | + (JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects): |
| 86 | + (JSC::DFG::AbstractInterpreter<AbstractStateType>::executeDoubleUnaryOpEffects): |
| 87 | + * dfg/DFGJITCode.cpp: |
| 88 | + (JSC::DFG::JITCode::findPC): |
| 89 | + * dfg/DFGJITCode.h: |
| 90 | + * dfg/DFGOperations.cpp: |
| 91 | + (JSC::DFG::operationPutByValInternal): |
| 92 | + * dfg/DFGSlowPathGenerator.h: |
| 93 | + (JSC::DFG::SlowPathGenerator::generate): |
| 94 | + * dfg/DFGSpeculativeJIT.cpp: |
| 95 | + (JSC::DFG::SpeculativeJIT::runSlowPathGenerators): |
| 96 | + (JSC::DFG::SpeculativeJIT::emitUntypedBitOp): |
| 97 | + (JSC::DFG::SpeculativeJIT::emitUntypedRightShiftBitOp): |
| 98 | + (JSC::DFG::SpeculativeJIT::compileMathIC): |
| 99 | + (JSC::DFG::SpeculativeJIT::compileArithDiv): |
| 100 | + (JSC::DFG::SpeculativeJIT::compileCallDOMGetter): |
| 101 | + * dfg/DFGSpeculativeJIT.h: |
| 102 | + * dfg/DFGSpeculativeJIT32_64.cpp: |
| 103 | + (JSC::DFG::SpeculativeJIT::compile): |
| 104 | + * dfg/DFGSpeculativeJIT64.cpp: |
| 105 | + (JSC::DFG::SpeculativeJIT::compileLogicalNot): |
| 106 | + (JSC::DFG::SpeculativeJIT::emitBranch): |
| 107 | + (JSC::DFG::SpeculativeJIT::compile): |
| 108 | + * dfg/DFGStrengthReductionPhase.cpp: |
| 109 | + (JSC::DFG::StrengthReductionPhase::handleNode): |
| 110 | + * ftl/FTLJITCode.cpp: |
| 111 | + (JSC::FTL::JITCode::findPC): |
| 112 | + * ftl/FTLJITCode.h: |
| 113 | + * heap/Heap.cpp: |
| 114 | + (JSC::Heap::collectAsync): |
| 115 | + (JSC::Heap::collectSync): |
| 116 | + (JSC::Heap::collectInThread): |
| 117 | + (JSC::Heap::requestCollection): |
| 118 | + (JSC::Heap::willStartCollection): |
| 119 | + (JSC::Heap::didFinishCollection): |
| 120 | + (JSC::Heap::shouldDoFullCollection): |
| 121 | + * heap/Heap.h: |
| 122 | + (JSC::Heap::collectionScope): |
| 123 | + * heap/HeapSnapshot.cpp: |
| 124 | + (JSC::HeapSnapshot::nodeForCell): |
| 125 | + (JSC::HeapSnapshot::nodeForObjectIdentifier): |
| 126 | + * heap/HeapSnapshot.h: |
| 127 | + * inspector/InspectorBackendDispatcher.cpp: |
| 128 | + (Inspector::BackendDispatcher::dispatch): |
| 129 | + (Inspector::BackendDispatcher::sendPendingErrors): |
| 130 | + (Inspector::BackendDispatcher::reportProtocolError): |
| 131 | + * inspector/InspectorBackendDispatcher.h: |
| 132 | + * inspector/agents/InspectorHeapAgent.cpp: |
| 133 | + (Inspector::InspectorHeapAgent::nodeForHeapObjectIdentifier): |
| 134 | + (Inspector::InspectorHeapAgent::getPreview): |
| 135 | + (Inspector::InspectorHeapAgent::getRemoteObject): |
| 136 | + * inspector/agents/InspectorHeapAgent.h: |
| 137 | + * inspector/remote/RemoteConnectionToTarget.h: |
| 138 | + * inspector/remote/RemoteConnectionToTarget.mm: |
| 139 | + (Inspector::RemoteConnectionToTarget::targetIdentifier): |
| 140 | + (Inspector::RemoteConnectionToTarget::setup): |
| 141 | + * inspector/remote/RemoteInspector.h: |
| 142 | + * inspector/remote/RemoteInspector.mm: |
| 143 | + (Inspector::RemoteInspector::updateClientCapabilities): |
| 144 | + * inspector/scripts/codegen/generate_cpp_protocol_types_header.py: |
| 145 | + (_generate_declarations_for_enum_conversion_methods): |
| 146 | + (_generate_declarations_for_enum_conversion_methods.return_type_with_export_macro): |
| 147 | + * inspector/scripts/codegen/generate_cpp_protocol_types_implementation.py: |
| 148 | + (CppProtocolTypesImplementationGenerator._generate_enum_conversion_methods_for_domain.generate_conversion_method_body): |
| 149 | + * inspector/scripts/tests/expected/commands-with-async-attribute.json-result: |
| 150 | + * inspector/scripts/tests/expected/commands-with-optional-call-return-parameters.json-result: |
| 151 | + * inspector/scripts/tests/expected/enum-values.json-result: |
| 152 | + * inspector/scripts/tests/expected/shadowed-optional-type-setters.json-result: |
| 153 | + * inspector/scripts/tests/expected/type-declaration-array-type.json-result: |
| 154 | + * inspector/scripts/tests/expected/type-declaration-enum-type.json-result: |
| 155 | + * inspector/scripts/tests/expected/type-requiring-runtime-casts.json-result: |
| 156 | + * jit/JITCode.h: |
| 157 | + (JSC::JITCode::findPC): |
| 158 | + * jit/JITDivGenerator.cpp: |
| 159 | + (JSC::JITDivGenerator::generateFastPath): |
| 160 | + * jit/JITOperations.cpp: |
| 161 | + * jit/PCToCodeOriginMap.cpp: |
| 162 | + (JSC::PCToCodeOriginMap::findPC): |
| 163 | + * jit/PCToCodeOriginMap.h: |
| 164 | + * jsc.cpp: |
| 165 | + (WTF::RuntimeArray::getOwnPropertySlot): |
| 166 | + * llint/LLIntSlowPaths.cpp: |
| 167 | + (JSC::LLInt::LLINT_SLOW_PATH_DECL): |
| 168 | + * parser/ModuleAnalyzer.cpp: |
| 169 | + (JSC::ModuleAnalyzer::exportVariable): |
| 170 | + * runtime/ConcurrentJSLock.h: |
| 171 | + (JSC::ConcurrentJSLocker::ConcurrentJSLocker): |
| 172 | + * runtime/DefinePropertyAttributes.h: |
| 173 | + (JSC::DefinePropertyAttributes::writable): |
| 174 | + (JSC::DefinePropertyAttributes::configurable): |
| 175 | + (JSC::DefinePropertyAttributes::enumerable): |
| 176 | + * runtime/GenericArgumentsInlines.h: |
| 177 | + (JSC::GenericArguments<Type>::getOwnPropertySlot): |
| 178 | + (JSC::GenericArguments<Type>::put): |
| 179 | + (JSC::GenericArguments<Type>::deleteProperty): |
| 180 | + (JSC::GenericArguments<Type>::defineOwnProperty): |
| 181 | + * runtime/HasOwnPropertyCache.h: |
| 182 | + (JSC::HasOwnPropertyCache::get): |
| 183 | + * runtime/HashMapImpl.h: |
| 184 | + (JSC::concurrentJSMapHash): |
| 185 | + * runtime/Identifier.h: |
| 186 | + (JSC::parseIndex): |
| 187 | + * runtime/JSArray.cpp: |
| 188 | + (JSC::JSArray::defineOwnProperty): |
| 189 | + * runtime/JSCJSValue.cpp: |
| 190 | + (JSC::JSValue::toNumberFromPrimitive): |
| 191 | + (JSC::JSValue::putToPrimitive): |
| 192 | + * runtime/JSCJSValue.h: |
| 193 | + * runtime/JSGenericTypedArrayView.h: |
| 194 | + (JSC::JSGenericTypedArrayView::toAdaptorNativeFromValueWithoutCoercion): |
| 195 | + * runtime/JSGenericTypedArrayViewConstructorInlines.h: |
| 196 | + (JSC::constructGenericTypedArrayViewWithArguments): |
| 197 | + (JSC::constructGenericTypedArrayView): |
| 198 | + * runtime/JSGenericTypedArrayViewInlines.h: |
| 199 | + (JSC::JSGenericTypedArrayView<Adaptor>::getOwnPropertySlot): |
| 200 | + (JSC::JSGenericTypedArrayView<Adaptor>::put): |
| 201 | + * runtime/JSModuleRecord.cpp: |
| 202 | + * runtime/JSModuleRecord.h: |
| 203 | + * runtime/JSObject.cpp: |
| 204 | + (JSC::JSObject::putDirectAccessor): |
| 205 | + (JSC::JSObject::deleteProperty): |
| 206 | + (JSC::JSObject::putDirectMayBeIndex): |
| 207 | + (JSC::JSObject::defineOwnProperty): |
| 208 | + * runtime/JSObject.h: |
| 209 | + (JSC::JSObject::getOwnPropertySlot): |
| 210 | + (JSC::JSObject::getPropertySlot): |
| 211 | + (JSC::JSObject::putOwnDataPropertyMayBeIndex): |
| 212 | + * runtime/JSObjectInlines.h: |
| 213 | + (JSC::JSObject::putInline): |
| 214 | + * runtime/JSString.cpp: |
| 215 | + (JSC::JSString::getStringPropertyDescriptor): |
| 216 | + * runtime/JSString.h: |
| 217 | + (JSC::JSString::getStringPropertySlot): |
| 218 | + * runtime/LiteralParser.cpp: |
| 219 | + (JSC::LiteralParser<CharType>::parse): |
| 220 | + * runtime/MathCommon.h: |
| 221 | + (JSC::safeReciprocalForDivByConst): |
| 222 | + * runtime/ObjectPrototype.cpp: |
| 223 | + (JSC::objectProtoFuncHasOwnProperty): |
| 224 | + * runtime/PropertyDescriptor.h: |
| 225 | + (JSC::toPropertyDescriptor): |
| 226 | + * runtime/PropertyName.h: |
| 227 | + (JSC::parseIndex): |
| 228 | + * runtime/SamplingProfiler.cpp: |
| 229 | + (JSC::SamplingProfiler::processUnverifiedStackTraces): |
| 230 | + * runtime/StringObject.cpp: |
| 231 | + (JSC::StringObject::put): |
| 232 | + (JSC::isStringOwnProperty): |
| 233 | + (JSC::StringObject::deleteProperty): |
| 234 | + * runtime/ToNativeFromValue.h: |
| 235 | + (JSC::toNativeFromValueWithoutCoercion): |
| 236 | + * runtime/TypedArrayAdaptors.h: |
| 237 | + (JSC::IntegralTypedArrayAdaptor::toNativeFromInt32WithoutCoercion): |
| 238 | + (JSC::IntegralTypedArrayAdaptor::toNativeFromUint32WithoutCoercion): |
| 239 | + (JSC::IntegralTypedArrayAdaptor::toNativeFromDoubleWithoutCoercion): |
| 240 | + (JSC::FloatTypedArrayAdaptor::toNativeFromInt32WithoutCoercion): |
| 241 | + (JSC::FloatTypedArrayAdaptor::toNativeFromDoubleWithoutCoercion): |
| 242 | + (JSC::Uint8ClampedAdaptor::toNativeFromInt32WithoutCoercion): |
| 243 | + (JSC::Uint8ClampedAdaptor::toNativeFromDoubleWithoutCoercion): |
| 244 | + |
1 | 245 | 2016-11-26 Sam Weinig < [email protected]>
|
2 | 246 |
|
3 | 247 | Convert IntersectionObserver over to using RuntimeEnabledFeatures so it can be properly excluded from script
|
|
0 commit comments