Title: [189585] trunk/Source/_javascript_Core
Revision
189585
Author
[email protected]
Date
2015-09-10 12:36:42 -0700 (Thu, 10 Sep 2015)

Log Message

CodeBlock::codeType() doesn't need to compute anything
https://bugs.webkit.org/show_bug.cgi?id=149039

Reviewed by Michael Saboff.

CodeBlock already has an m_codeType data member.

* bytecode/CodeBlock.h:
(JSC::CodeBlock::codeType):
(JSC::CodeBlock::putByIdContext):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (189584 => 189585)


--- trunk/Source/_javascript_Core/ChangeLog	2015-09-10 19:34:40 UTC (rev 189584)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-09-10 19:36:42 UTC (rev 189585)
@@ -1,3 +1,16 @@
+2015-09-10  Geoffrey Garen  <[email protected]>
+
+        CodeBlock::codeType() doesn't need to compute anything
+        https://bugs.webkit.org/show_bug.cgi?id=149039
+
+        Reviewed by Michael Saboff.
+
+        CodeBlock already has an m_codeType data member.
+
+        * bytecode/CodeBlock.h:
+        (JSC::CodeBlock::codeType):
+        (JSC::CodeBlock::putByIdContext):
+
 2015-09-10  Sukolsak Sakshuwong  <[email protected]>
 
         Implement global variables in WebAssembly

Modified: trunk/Source/_javascript_Core/bytecode/CodeBlock.h (189584 => 189585)


--- trunk/Source/_javascript_Core/bytecode/CodeBlock.h	2015-09-10 19:34:40 UTC (rev 189584)
+++ trunk/Source/_javascript_Core/bytecode/CodeBlock.h	2015-09-10 19:36:42 UTC (rev 189585)
@@ -353,11 +353,7 @@
     
     CodeType codeType() const
     {
-#if ENABLE(WEBASSEMBLY)
-        if (m_ownerExecutable->isWebAssemblyExecutable())
-            return FunctionCode;
-#endif
-        return m_unlinkedCode->codeType();
+        return m_codeType;
     }
 
     PutPropertySlot::Context putByIdContext() const
@@ -1022,7 +1018,7 @@
     RefPtr<SourceProvider> m_source;
     unsigned m_sourceOffset;
     unsigned m_firstLineColumnOffset;
-    unsigned m_codeType;
+    CodeType m_codeType;
 
     Vector<LLIntCallLinkInfo> m_llintCallLinkInfos;
     SentinelLinkedList<LLIntCallLinkInfo, BasicRawSentinelNode<LLIntCallLinkInfo>> m_incomingLLIntCalls;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to