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

Unified Diff: syzygy/integration_tests/make_integration_tests_clang.py

Issue 2981443002: Revert of Fixed some nits based on reviewer comments. (Closed)
Patch Set: Created 3 years, 5 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: syzygy/integration_tests/make_integration_tests_clang.py
diff --git a/syzygy/integration_tests/make_integration_tests_clang.py b/syzygy/integration_tests/make_integration_tests_clang.py
index b9d7ade77a158cc5844f77358b280ecd8abe9045..0e1731fc6efef0a73a4b2669bf39cdc9fdc32177 100644
--- a/syzygy/integration_tests/make_integration_tests_clang.py
+++ b/syzygy/integration_tests/make_integration_tests_clang.py
@@ -56,8 +56,9 @@
compile_command_base.extend(compiler_flags)
for source_file, object_file in zip(source_files, object_files):
- compile_command = list(compile_command_base)
+ compile_command = compile_command_base
compile_command.extend([source_file, '-o', object_file])
+ print compile_command
ret = subprocess.call(compile_command)
if ret != 0:
print 'ERROR: Failed compiling %s using clang-cl.' % target_name
@@ -83,8 +84,8 @@
os.path.join(build_dir, target_name + '.dll'),
'/link',
'/dll',
- os.path.join(build_dir, 'export_dll.dll.lib'),
- os.path.join(build_dir, 'syzyasan_rtl.dll.lib'),
+ build_dir + '\export_dll.dll.lib',
+ build_dir + '\syzyasan_rtl.dll.lib',
'-defaultlib:libcmt'
]
@@ -117,8 +118,8 @@
def get_object_file_location(source_file,
output_dir, target_name):
return os.path.join(output_dir, 'obj',
- os.path.dirname(os.path.relpath(source_file,
- _SRC_DIR)),
+ os.path.split(os.path.relpath(source_file,
+ _SRC_DIR))[0],
'%s.%s.obj' % (target_name,
os.path.splitext(os.path.basename(source_file))[0]))
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698