Rename testing build configuration to release+assert, and add new testing build confi...
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Jan 2020 22:20:24 +0000 (22:20 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Jan 2020 22:20:24 +0000 (22:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=205954

Reviewed by Yusuke Suzuki.

.:

"make release-assert" builds a release build with ASSERT_ENABLED=1.
"make testing" builds a debug build with clang optimization level forced to -O3.

* Makefile:
* Makefile.shared:
* Source/Makefile:

Source/ThirdParty:

* Makefile:

Tools:

Added a --force-opt shortcut for the --force-optimization-level option of
set-webkit-configuration.  This makes it less painful to type this on the command
line for those of us who would like to use this option often in our local builds.

* Makefile:
* Scripts/set-webkit-configuration:

WebKitLibraries:

* Makefile:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@254227 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Makefile
Makefile.shared
Source/Makefile
Source/ThirdParty/ChangeLog
Source/ThirdParty/Makefile
Tools/ChangeLog
Tools/Makefile
Tools/Scripts/set-webkit-configuration
WebKitLibraries/ChangeLog
WebKitLibraries/Makefile

index 3247ab2..595ae9b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2020-01-08  Mark Lam  <mark.lam@apple.com>
+
+        Rename testing build configuration to release+assert, and add new testing build configuration.
+        https://bugs.webkit.org/show_bug.cgi?id=205954
+
+        Reviewed by Yusuke Suzuki.
+
+        "make release-assert" builds a release build with ASSERT_ENABLED=1.
+        "make testing" builds a debug build with clang optimization level forced to -O3.
+
+        * Makefile:
+        * Makefile.shared:
+        * Source/Makefile:
+
 2020-01-08  Keith Miller  <keith_miller@apple.com>
 
         Fix testing Makefile to inherit preprocessor definitions
index 4a6e61f..f215e30 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -12,6 +12,10 @@ release r:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+release+assert ra:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 testing t:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
index 80575e5..9016f9c 100644 (file)
@@ -101,10 +101,14 @@ release r deployment dep deploy: force
        $(SCRIPTS_PATH)/set-webkit-configuration --release $(ASAN_OPTION) $(WK_LTO_OPTION)
        ( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
 
-testing t: force
+release+assert ra: force
        $(SCRIPTS_PATH)/set-webkit-configuration --release $(ASAN_OPTION) $(WK_LTO_OPTION)
        ( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) GCC_PREPROCESSOR_DEFINITIONS='ASSERT_ENABLED=1 $$(inherited)' | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
 
+testing t: force
+       $(SCRIPTS_PATH)/set-webkit-configuration --debug $(ASAN_OPTION) $(WK_LTO_OPTION) --force-optimization-level=O3
+       ( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
+
 analyze:
        $(SCRIPTS_PATH)/set-webkit-configuration --debug $(ASAN_OPTION) $(WK_LTO_OPTION)
 ifndef PATH_TO_SCAN_BUILD
index 24f9846..ed335ce 100644 (file)
@@ -22,6 +22,10 @@ release r:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+release+assert ra:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 testing t:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
index ba2d324..d7ecb6e 100644 (file)
@@ -1,3 +1,12 @@
+2020-01-08  Mark Lam  <mark.lam@apple.com>
+
+        Rename testing build configuration to release+assert, and add new testing build configuration.
+        https://bugs.webkit.org/show_bug.cgi?id=205954
+
+        Reviewed by Yusuke Suzuki.
+
+        * Makefile:
+
 2020-01-07  Keith Miller  <keith_miller@apple.com>
 
         Add a testing target to make that is release + assertions
index a7fddf8..f28032c 100644 (file)
@@ -36,6 +36,10 @@ release r deployment dep deploy:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+release+assert ra:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 testing t:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
index 9a8f331..c65de6a 100644 (file)
@@ -1,3 +1,17 @@
+2020-01-08  Mark Lam  <mark.lam@apple.com>
+
+        Rename testing build configuration to release+assert, and add new testing build configuration.
+        https://bugs.webkit.org/show_bug.cgi?id=205954
+
+        Reviewed by Yusuke Suzuki.
+
+        Added a --force-opt shortcut for the --force-optimization-level option of
+        set-webkit-configuration.  This makes it less painful to type this on the command
+        line for those of us who would like to use this option often in our local builds.
+
+        * Makefile:
+        * Scripts/set-webkit-configuration:
+
 2020-01-08  Matt Lewis  <jlewis3@apple.com>
 
         Move EWS from macOS High Sierra to macOS Mojave
index 874ce2a..d147801 100644 (file)
@@ -32,6 +32,10 @@ release r deployment dep deploy:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+release+assert ra:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 testing t:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
index 818df3f..d7c68d2 100755 (executable)
@@ -54,7 +54,8 @@ if (!checkForArgumentAndRemoveFromARGVGettingValue("--lto-mode", \$ltoMode)) {
     $ltoMode="";
 }
 my $forceOptimizationLevel;
-if (!checkForArgumentAndRemoveFromARGVGettingValue("--force-optimization-level", \$forceOptimizationLevel)) {
+if (!checkForArgumentAndRemoveFromARGVGettingValue("--force-optimization-level", \$forceOptimizationLevel)
+    && !checkForArgumentAndRemoveFromARGVGettingValue("--force-opt", \$forceOptimizationLevel)) {
     $forceOptimizationLevel="";
 }
 
index 4e4414c..9220838 100644 (file)
@@ -1,3 +1,12 @@
+2020-01-08  Mark Lam  <mark.lam@apple.com>
+
+        Rename testing build configuration to release+assert, and add new testing build configuration.
+        https://bugs.webkit.org/show_bug.cgi?id=205954
+
+        Reviewed by Yusuke Suzuki.
+
+        * Makefile:
+
 2020-01-07  Keith Miller  <keith_miller@apple.com>
 
         Add a testing target to make that is release + assertions
index df203c6..6c42e2a 100644 (file)
@@ -22,10 +22,14 @@ release r:
        $(SCRIPTS_PATH)/set-webkit-configuration --release
        @$(MAKE) libs
 
-testing t:
+release+assert ra:
        $(SCRIPTS_PATH)/set-webkit-configuration --release
        @$(MAKE) libs
 
+testing t:
+       $(SCRIPTS_PATH)/set-webkit-configuration --debug --force-optimization-level=O3
+       @$(MAKE) libs
+
 analyze:
        $(SCRIPTS_PATH)/set-webkit-configuration --debug
        @$(MAKE) libs