Changeset 239427 in webkit for trunk/Source/JavaScriptCore/dfg/DFGJITCode.cpp

Timestamp:

Dec 19, 2018, 8:41:11 PM (6 years ago)

Author:

Chris Dumez

Message:

wtf/Optional.h: move-constructor and move-assignment operator should disengage the value being moved from
​https://bugs.webkit.org/show_bug.cgi?id=192728
<rdar://problem/46746779>

Reviewed by Geoff Garen.

Source/JavaScriptCore:

• API/*:
• Scripts/*:
• assembler/*:
• b3/*:
• bytecode/*:
• bytecompiler/*:
• debugger/*:
• dfg/*:
• ftl/*:
• heap/*:
• inspector/*:
• jit/*:
• llint/*:
• parser/*:
• runtime/*:
• tools/*:
• wasm/*:
• yarr/*:

Source/WebCore:

• Modules/*:
• animation/*:
• bindings/*:
• crypto/*:
• css/*:
• dom/*:
• editing/*:
• fileapi/*:
• html/*:
• inspector/*:
• layout/*:
• loader/*:
• mathml/*:
• page/*:
• platform/*:
• plugins/*:
• rendering/*:
• testing/*:
• workers/*:
• xml/*:

Source/WebCore/PAL:

• pal/*:

Source/WebDriver:

• :

Source/WebKit:

• NetworkProcess/*:
• Platform/*:
• Scripts/*:
• Shared/*:
• UIProcess/*:
• WebProcess/*:

Source/WebKitLegacy/mac:

• DOM/*:
• Plugins/*:
• WebCoreSupport/*:
• WebView/*:

Source/WebKitLegacy/win:

• Plugins/*:
• WebCoreSupport/*:

Source/WTF:

Update optional's move-constructor and move-assignment operator to disengage the value being moved from.
Rename to optional to Optional, make_optional() to makeOptional(), and move class from std to WTF namespace.

Based on patch by David Kilzer.

• wtf/*:

Tools:

• DumpRenderTree/*:
• MiniBrowser/*:
• TestRunnerShared/*:
• TestWebKitAPI/*:
• WebGPUAPIStructure/*:
• WebKitTestRunner/*:

File:

• trunk/Source/JavaScriptCore/dfg/DFGJITCode.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/dfg/DFGJITCode.cpp

@@ -226 +226 @@
 }
 
-std::optional<CodeOrigin> JITCode::findPC(CodeBlock*, void* pc)
+Optional<CodeOrigin> JITCode::findPC(CodeBlock*, void* pc)
 {
     for (OSRExit& exit : osrExit) {
         if (ExecutableMemoryHandle* handle = exit.m_code.executableMemory()) {
             if (handle->start().untaggedPtr() <= pc && pc < handle->end().untaggedPtr())
-                return std::optional<CodeOrigin>(exit.m_codeOriginForExitProfile);
+                return Optional<CodeOrigin>(exit.m_codeOriginForExitProfile);
         }
     }
 
-    return std::nullopt;
+    return WTF::nullopt;
 }