Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1240)

Unified Diff: runtime/vm/vm.gypi

Issue 11419230: Enable building with VTune support on Windows. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Ditto. Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/tools/gyp/runtime-configurations.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/vm.gypi
diff --git a/runtime/vm/vm.gypi b/runtime/vm/vm.gypi
index 937b1d0cc57c7ac6e0d75251b49dc6d8f76be514..530f7f0642f3d3a805f3d3b9d0069943c442b20e 100644
--- a/runtime/vm/vm.gypi
+++ b/runtime/vm/vm.gypi
@@ -58,20 +58,22 @@
['exclude', 'gdbjit.cc'],
],
}],
- ['dart_vtune_support == 0', {
+ ['dart_vtune_support==0', {
'sources/' : [
['exclude', 'vtune\\.(cc|h)$'],
],
}],
- ['OS=="linux" and dart_vtune_support == 1', {
- # Link in libjitprofiling.a.
- 'cflags': [
- '-DDART_VTUNE_SUPPORT',
- '-I<(dart_vtune_root)/include',
- ],
+ ['dart_vtune_support==1', {
+ 'include_dirs': ['<(dart_vtune_root)/include'],
+ 'defines': ['DART_VTUNE_SUPPORT'],
'link_settings': {
- 'libraries': [
- '-ljitprofiling',
+ 'conditions': [
+ ['OS=="linux"', {
+ 'libraries': ['-ljitprofiling'],
+ }],
+ ['OS=="win"', {
+ 'libraries': ['-ljitprofiling.lib'],
+ }],
],
},
}]],
« no previous file with comments | « runtime/tools/gyp/runtime-configurations.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698