Changeset 195362 in webkit

Timestamp:

Jan 20, 2016 11:58:31 AM (8 years ago)

Author:

peavo@outlook.com

Message:

[B3][Win64] Compile fix.
​https://bugs.webkit.org/show_bug.cgi?id=153278

Reviewed by Filip Pizlo.

MSVC does not accept that a class declared as exported also have members declared as exported.

• b3/B3Const32Value.h:
• b3/B3ControlValue.h:

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• b3/B3Const32Value.h (modified) (2 diffs)
• b3/B3ControlValue.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2016-01-20  Per Arne Vollan  <peavo@outlook.com>
+
+        [B3][Win64] Compile fix.
+        https://bugs.webkit.org/show_bug.cgi?id=153278
+
+        Reviewed by Filip Pizlo.
+
+        MSVC does not accept that a class declared as exported also have members declared as exported.
+
+        * b3/B3Const32Value.h:
+        * b3/B3ControlValue.h:
+
 2016-01-19  Keith Miller  <keith_miller@apple.com>
 

trunk/Source/JavaScriptCore/b3/B3Const32Value.h

@@ -37 +37 @@
     static bool accepts(Opcode opcode) { return opcode == Const32; }
     
-    JS_EXPORT_PRIVATE ~Const32Value();
+    ~Const32Value();
     
     int32_t value() const { return m_value; }
@@ -72 +72 @@
 
 protected:
-    JS_EXPORT_PRIVATE void dumpMeta(CommaPrinter&, PrintStream&) const override;
+    void dumpMeta(CommaPrinter&, PrintStream&) const override;
 
     friend class Procedure;

trunk/Source/JavaScriptCore/b3/B3ControlValue.h

@@ -57 +57 @@
     typedef Vector<FrequentedBlock, 2> SuccessorList;
 
-    JS_EXPORT_PRIVATE ~ControlValue();
+    ~ControlValue();
 
     unsigned numSuccessors() const { return m_successors.size(); }
@@ -92 +92 @@
 
 protected:
-    JS_EXPORT_PRIVATE void dumpMeta(CommaPrinter&, PrintStream&) const override;
+    void dumpMeta(CommaPrinter&, PrintStream&) const override;
 
     // Use this for subclasses.