Skip to content

Commit 36c1d56

Browse files
committed
Remove hack that’s causing cperf failures
Previously, because we chdir()ed before parsing arguments, the --output argument would be relative to the swift-source-compat-suite directory, which necessitated a hack to write the comment file to the CI-expected path. Commit 6952162 accidentally fixed that bug. Unfortutnately, it also broke the hack, which caused all compiler performance tests to fail. This commit deletes the hack, which should restore correct behavior.
1 parent 0ec1874 commit 36c1d56

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

run_cperf

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -134,18 +134,6 @@ def main():
134134
execute_runner(instance, workspace, configs, args)
135135

136136
regressions = analyze_results(configs, args)
137-
138-
# Crude hack to write output to workspace when in CI,
139-
# regardless of --output passed.
140-
if ws_comment is not None:
141-
out = os.path.abspath(os.path.join(os.getcwd(),
142-
args.output.name))
143-
print("Output written to: %s" % out)
144-
if out != ws_comment:
145-
print("Copying %s to %s" % (out, ws_comment))
146-
args.output.close()
147-
shutil.copyfile(out, ws_comment)
148-
149137
return regressions
150138

151139

0 commit comments

Comments
 (0)