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

Side by Side Diff: test/win/ml-safeseh/ml-safeseh.gyp

Issue 217813007: Pass the /safeseh option to ml.exe. (Closed) Base URL: https://chromium.googlesource.com/external/gyp.git@master
Patch Set: Add a unit test gyptest-ml-safeseh.py Created 6 years, 8 months 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 unified diff | Download patch
« test/win/gyptest-ml-safeseh.py ('K') | « test/win/ml-safeseh/hello.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 Google Inc. All rights reserved. 1 # Copyright (c) 2014 Google Inc. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'test_pdbname', 8 'target_name': 'ml_safeseh',
9 'type': 'executable', 9 'type': 'executable',
10 'sources': [ 10 'sources': [
11 'hello.cc', 11 'hello.cc',
12 'pdbname.cc', 12 'a.asm',
13 ], 13 ],
14 'msvs_settings': { 14 'msvs_settings': {
15 'VCCLCompilerTool': { 15 'VCLinkerTool': {
16 'DebugInformationFormat': '3', 16 'AdditionalOptions': [
17 '/SAFESEH'
wtc 2014/04/10 02:42:00 Is there a setting I can use instead of adding '/S
scottmg 2014/04/10 03:31:51 It should be 'ImageHasSafeExceptionHandlers': 'tru
wtc 2014/04/10 18:21:48 Done. Thanks for the hint.
18 ]
17 }, 19 },
18 'VCLinkerTool': { 20 'MASM': {
19 'GenerateDebugInformation': 'true', 21 'UseSafeExceptionHandlers': 'true',
20 }, 22 },
21 }, 23 },
22 }, 24 },
23 ] 25 ]
24 } 26 }
OLDNEW
« test/win/gyptest-ml-safeseh.py ('K') | « test/win/ml-safeseh/hello.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698