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

Unified Diff: chrome/tools/build/win/syzygy_instrument.py

Issue 25890006: Copy asan_rtl.* and agent_logger.exe to the syzygy directory for the ASan builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move the dependency to copy_syzyasan_binaries outside of the 'action' section. Created 7 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/chrome_syzygy.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/tools/build/win/syzygy_instrument.py
diff --git a/chrome/tools/build/win/syzygy_instrument.py b/chrome/tools/build/win/syzygy_instrument.py
index 6c6fcfe685cae8dd10da331bacd89d35c2716bab..ae6558e7cf297f6967a2a58f92b85bd239e482eb 100755
--- a/chrome/tools/build/win/syzygy_instrument.py
+++ b/chrome/tools/build/win/syzygy_instrument.py
@@ -22,10 +22,6 @@ _DEFAULT_SYZYGY_DIR = os.path.abspath(os.path.join(
# Basenames of various tools.
_INSTRUMENT_EXE = 'instrument.exe'
_GENFILTER_EXE = 'genfilter.exe'
-_ASAN_AGENT_DLL = 'syzyasan_rtl.dll'
-
-# Default agents for known modes.
-_DEFAULT_AGENT_DLLS = { 'asan': _ASAN_AGENT_DLL }
_LOGGER = logging.getLogger()
@@ -87,24 +83,6 @@ def _InstrumentBinary(syzygy_dir, mode, executable, symbol, dst_dir,
return _Shell(*cmd)
-def _CopyAgentDLL(agent_dll, destination_dir):
- """Copy the agent DLL and PDB to the destination directory."""
- dirname, agent_name = os.path.split(agent_dll);
- agent_dst_name = os.path.join(destination_dir, agent_name);
- shutil.copyfile(agent_dll, agent_dst_name)
-
- # Search for the corresponding PDB file. We use this approach because
- # the naming convention for PDBs has changed recently (from 'foo.pdb'
- # to 'foo.dll.pdb') and we want to support both conventions during the
- # transition.
- agent_pdbs = glob.glob(os.path.splitext(agent_dll)[0] + '*.pdb')
- if len(agent_pdbs) != 1:
- raise RuntimeError('Failed to locate PDB file for %s' % agent_name)
- agent_pdb = agent_pdbs[0]
- agent_dst_pdb = os.path.join(destination_dir, os.path.split(agent_pdb)[1])
- shutil.copyfile(agent_pdb, agent_dst_pdb)
-
-
def main(options):
# Make sure the destination directory exists.
if not os.path.isdir(options.destination_dir):
@@ -128,9 +106,6 @@ def main(options):
options.destination_dir,
options.output_filter_file)
- # Copy the agent DLL and PDB to the destination directory.
- _CopyAgentDLL(options.agent_dll, options.destination_dir);
-
def _ParseOptions():
option_parser = optparse.OptionParser()
@@ -140,9 +115,6 @@ def _ParseOptions():
help='The path to the input symbol file.')
option_parser.add_option('--mode',
help='Specifies which instrumentation mode is to be used.')
- option_parser.add_option('--agent_dll',
- help='The agent DLL used by this instrumentation. If not specified a '
- 'default will be searched for.')
option_parser.add_option('--syzygy-dir', default=_DEFAULT_SYZYGY_DIR,
help='Instrumenter executable to use, defaults to "%default".')
option_parser.add_option('-d', '--destination_dir',
@@ -166,14 +138,6 @@ def _ParseOptions():
if options.filter and not options.output_filter_file:
option_parser.error('You must provide a filter output file.')
- if not options.agent_dll:
- if not options.mode in _DEFAULT_AGENT_DLLS:
- option_parser.error('No known default agent DLL for mode "%s".' %
- options.mode)
- options.agent_dll = os.path.abspath(os.path.join(options.syzygy_dir,
- _DEFAULT_AGENT_DLLS[options.mode]))
- _LOGGER.info('Using default agent DLL: %s' % options.agent_dll)
-
return options
« no previous file with comments | « chrome/chrome_syzygy.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698