Title: [139765] trunk/Source/_javascript_Core
Revision
139765
Author
zandober...@gmail.com
Date
2013-01-15 11:41:02 -0800 (Tue, 15 Jan 2013)

Log Message

[Autotools] Unify _javascript_Core sources list, regardless of target OS
https://bugs.webkit.org/show_bug.cgi?id=106007

Patch by Zan Dobersek <zandober...@gmail.com> on 2013-01-15
Reviewed by Gustavo Noronha Silva.

Include the Source/_javascript_Core/jit/ExecutableAllocatorFixedVMPool.cpp target
in the general sources list as it is guarded by the ENABLE_EXECUTABLE_ALLOCATOR_FIXED
feature define. This define is only used on 64-bit architecture and indirectly depends
on enabling either JIT or YARR JIT feature. Both of these defines are disabled on
Windows OS when using 64-bit architecture so there's no need to add this target to
sources only when the target OS is Windows.

* GNUmakefile.list.am:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (139764 => 139765)


--- trunk/Source/_javascript_Core/ChangeLog	2013-01-15 19:35:07 UTC (rev 139764)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-01-15 19:41:02 UTC (rev 139765)
@@ -1,3 +1,19 @@
+2013-01-15  Zan Dobersek  <zandober...@gmail.com>
+
+        [Autotools] Unify _javascript_Core sources list, regardless of target OS
+        https://bugs.webkit.org/show_bug.cgi?id=106007
+
+        Reviewed by Gustavo Noronha Silva.
+
+        Include the Source/_javascript_Core/jit/ExecutableAllocatorFixedVMPool.cpp target
+        in the general sources list as it is guarded by the ENABLE_EXECUTABLE_ALLOCATOR_FIXED
+        feature define. This define is only used on 64-bit architecture and indirectly depends
+        on enabling either JIT or YARR JIT feature. Both of these defines are disabled on
+        Windows OS when using 64-bit architecture so there's no need to add this target to
+        sources only when the target OS is Windows.
+
+        * GNUmakefile.list.am:
+
 2013-01-11  Filip Pizlo  <fpi...@apple.com>
 
         DFG should not forget that it had proved something to be a constant during a merge just because it's merging against the empty value

Modified: trunk/Source/_javascript_Core/GNUmakefile.list.am (139764 => 139765)


--- trunk/Source/_javascript_Core/GNUmakefile.list.am	2013-01-15 19:35:07 UTC (rev 139764)
+++ trunk/Source/_javascript_Core/GNUmakefile.list.am	2013-01-15 19:41:02 UTC (rev 139765)
@@ -405,6 +405,7 @@
 	Source/_javascript_Core/jit/ClosureCallStubRoutine.h \
 	Source/_javascript_Core/jit/ExecutableAllocator.cpp \
 	Source/_javascript_Core/jit/ExecutableAllocator.h \
+	Source/_javascript_Core/jit/ExecutableAllocatorFixedVMPool.cpp \
 	Source/_javascript_Core/jit/GCAwareJITStubRoutine.cpp \
 	Source/_javascript_Core/jit/GCAwareJITStubRoutine.h \
 	Source/_javascript_Core/jit/HostCallReturnValue.cpp \
@@ -765,12 +766,6 @@
 	Source/_javascript_Core/yarr/YarrSyntaxChecker.cpp \
 	Source/_javascript_Core/yarr/YarrSyntaxChecker.h
 
-if TARGET_WIN32
-else
-_javascript_core_sources += \
-	Source/_javascript_Core/jit/ExecutableAllocatorFixedVMPool.cpp
-endif
-
 llint_nosources += \
 	Source/_javascript_Core/llint/LowLevelInterpreter32_64.asm \
 	Source/_javascript_Core/llint/LowLevelInterpreter64.asm \
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to