Title: [104298] branches/chromium/wpotest/Source
- Revision
- 104298
- Author
- [email protected]
- Date
- 2012-01-06 09:34:57 -0800 (Fri, 06 Jan 2012)
Log Message
Merge 103722 - [chromium] really enable wpo for WebCore libs and for WTF
https://bugs.webkit.org/show_bug.cgi?id=75264
Reviewed by Adam Barth.
Source/_javascript_Core:
* _javascript_Core.gyp/_javascript_Core.gyp: Enable WPO for wtf and yarr.
Source/WebCore:
* WebCore.gyp/WebCore.gyp: The variable was getting clobbered by the
value set in common.gypi. Use a target_defaults instead to set the
variable. I tested manually on my Windows machine and
WholeProgramOptimization is getting set when buildtype is Official.
[email protected]
Modified Paths
Diff
Modified: branches/chromium/wpotest/Source/_javascript_Core/ChangeLog (104297 => 104298)
--- branches/chromium/wpotest/Source/_javascript_Core/ChangeLog 2012-01-06 17:33:40 UTC (rev 104297)
+++ branches/chromium/wpotest/Source/_javascript_Core/ChangeLog 2012-01-06 17:34:57 UTC (rev 104298)
@@ -1,3 +1,12 @@
+2011-12-27 Tony Chang <[email protected]>
+
+ [chromium] really enable wpo for WebCore libs and for WTF
+ https://bugs.webkit.org/show_bug.cgi?id=75264
+
+ Reviewed by Adam Barth.
+
+ * _javascript_Core.gyp/_javascript_Core.gyp: Enable WPO for wtf and yarr.
+
2011-12-02 Sheriff Bot <[email protected]>
Unreviewed, rolling out r101801.
Modified: branches/chromium/wpotest/Source/_javascript_Core/_javascript_Core.gyp/_javascript_Core.gyp (104297 => 104298)
--- branches/chromium/wpotest/Source/_javascript_Core/_javascript_Core.gyp/_javascript_Core.gyp 2012-01-06 17:33:40 UTC (rev 104297)
+++ branches/chromium/wpotest/Source/_javascript_Core/_javascript_Core.gyp/_javascript_Core.gyp 2012-01-06 17:34:57 UTC (rev 104298)
@@ -104,6 +104,7 @@
'target_name': 'wtf',
'type': 'static_library',
'msvs_guid': 'AA8A5A85-592B-4357-BC60-E0E91E026AF6',
+ 'variables': { 'optimize': 'max' },
'dependencies': [
'wtf_config',
'<(chromium_src_dir)/third_party/icu/icu.gyp:icui18n',
@@ -205,6 +206,7 @@
}],
],
'msvs_guid': '49909552-0B0C-4C14-8CF6-DB8A2ADE0934',
+ 'variables': { 'optimize': 'max' },
'actions': [
{
'action_name': 'retgen',
Modified: branches/chromium/wpotest/Source/WebCore/ChangeLog (104297 => 104298)
--- branches/chromium/wpotest/Source/WebCore/ChangeLog 2012-01-06 17:33:40 UTC (rev 104297)
+++ branches/chromium/wpotest/Source/WebCore/ChangeLog 2012-01-06 17:34:57 UTC (rev 104298)
@@ -1,3 +1,15 @@
+2011-12-27 Tony Chang <[email protected]>
+
+ [chromium] really enable wpo for WebCore libs and for WTF
+ https://bugs.webkit.org/show_bug.cgi?id=75264
+
+ Reviewed by Adam Barth.
+
+ * WebCore.gyp/WebCore.gyp: The variable was getting clobbered by the
+ value set in common.gypi. Use a target_defaults instead to set the
+ variable. I tested manually on my Windows machine and
+ WholeProgramOptimization is getting set when buildtype is Official.
+
2011-12-02 Sheriff Bot <[email protected]>
Unreviewed, rolling out r101731.
Modified: branches/chromium/wpotest/Source/WebCore/WebCore.gyp/WebCore.gyp (104297 => 104298)
--- branches/chromium/wpotest/Source/WebCore/WebCore.gyp/WebCore.gyp 2012-01-06 17:33:40 UTC (rev 104297)
+++ branches/chromium/wpotest/Source/WebCore/WebCore.gyp/WebCore.gyp 2012-01-06 17:34:57 UTC (rev 104298)
@@ -259,6 +259,12 @@
],
}, # variables
+ 'target_defaults': {
+ 'variables': {
+ 'optimize': 'max',
+ },
+ },
+
'conditions': [
['OS!="win" and remove_webcore_debug_symbols==1', {
# Remove -g from all targets defined here.
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
