Changes to comments

Author: ianayl

devops/benchmarking/config.ini

@@ -3,7 +3,8 @@
 ; benchmarking workflow in sycl-linux-run-tests.yml.
 ;
 ; DO NOT USE THE CONTENTS OF THIS FILE DIRECTLY -- Due to security concerns, The
-; contents of this file must be sanitized first before use. See: xxx.py
+; contents of this file must be sanitized first before use.
+; See: /devops/scripts/benchmarking/common.py
 ;
 
 ; Compute-benchmark compile/run options

devops/benchmarking/constants.ini

@@ -4,7 +4,8 @@
 ; workflow, you're likely looking for /devops/benchmarking/config.ini instead.
 ;
 ; DO NOT USE THE CONTENTS OF THIS FILE DIRECTLY -- Due to security concerns, The
-; contents of this file must be sanitized first before use. See: xxx.py
+; contents of this file must be sanitized first before use.
+; See: /devops/scripts/benchmarking/common.py
 ;
 
 ; Constants for compute-benchmarks
@@ -14,6 +15,13 @@ git_branch = master
 git_commit = 230a3db4d8d03c0e9a663988f7c3abbd1137a1e0
 ; path = ./compute-benchmarks
 
+; Constants for git repo storing benchmark performance results
+[perf_res]
+git_repo = intel/llvm-ci-perf-results
+git_branch = main
+; Path to clone performance result repo
+; path = ./llvm-ci-perf-results
+
 ; It was decided that paths should be hardcoded throughout this workflow for
 ; security reasons and ease of readability. Do not use paths as constants.
 
@@ -29,13 +37,6 @@ git_commit = 230a3db4d8d03c0e9a663988f7c3abbd1137a1e0
 ; ; Path (relative to artifact.path) to cache passing compute-benchmark results
 ; passing_cache = ./artifact/passing_tests
 
-; Constants for git repo storing benchmark performance results
-[perf_res]
-git_repo = intel/llvm-ci-perf-results
-git_branch = main
-; Path to clone performance result repo
-; path = ./llvm-ci-perf-results
-
 ; [timestamp]
 ; ; Timestamp format used for 
 ; format = %%Y%%m%%d_%%H%%M%%S

devops/scripts/benchmarking/common.py

@@ -163,21 +163,6 @@ def export_python_globals(self):
         all_opts.read(self.config_path)
         all_opts.read(self.constants_path)
 
-        # It was decided that paths should be hardcoded throughout this workflow
-        # for security reasons and ease of readability purposes:
-
-        # SanitizedConfig.PERF_RES_PATH = \
-        #     self.__sanitize(all_opts["perf_res"]["path"], "perf_res.path")
-        # SanitizedConfig.ARTIFACT_OUTPUT_CACHE = \
-        #     self.__sanitize(all_opts["artifact"]["output_cache"],
-        #                     "artifact.output_cache")
-        # SanitizedConfig.BENCHMARK_LOG_SLOW = \
-        #     self.__sanitize(all_opts["benchmark_log"]["slow"],
-        #                     "benchmark_log.slow")
-        # SanitizedConfig.BENCHMARK_LOG_ERROR = \
-        #     self.__sanitize(all_opts["benchmark_log"]["error"],
-        #                     "benchmark_log.error")
-
         # Fields that are supposed to be python objects need to be changed to
         # python objects manually: