fix lint; fix gin file comment (#143)

* fix lint; fix gin file comment

* fix lint; fix gin file comment; fix test failure

Author: kshiteejm

compiler_opt/rl/corpus.py

@@ -183,7 +183,7 @@ def _build_modulespecs_from_datapath(
       logging.warning('Additional flags are specified together with override.')
     if len(delete_flags) > 0:
       logging.warning('Delete flags are specified together with override.')
-    if len(replace_flags) > 0:
+    if replace_flags:
       logging.warning('Replace flags are specified together with override.')
 
   module_specs: List[ModuleSpec] = []

compiler_opt/rl/inlining/gin_configs/common.gin

@@ -10,6 +10,6 @@ runners.InliningRunner.launcher_path=%launcher_path
 
 problem_config.flags_to_add.add_flags=()
 problem_config.flags_to_delete.delete_flags=('-split-dwarf-file','-split-dwarf-output',)
-# For AFDO profile reinjection elide overlapping flags from delete_flags and set
+# For AFDO profile reinjection set:
 # problem_config.flags_to_replace.replace_flags={'-fprofile-sample-use':'/path/to/gwp.afdo','-fprofile-remapping-file':'/path/to/prof_remap.txt'}
 problem_config.flags_to_replace.replace_flags={}

compiler_opt/rl/regalloc/gin_configs/common.gin

@@ -8,6 +8,6 @@ runners.RegAllocRunner.launcher_path=%launcher_path
 
 problem_config.flags_to_add.add_flags=()
 problem_config.flags_to_delete.delete_flags=('-split-dwarf-file','-split-dwarf-output',)
-# For AFDO profile reinjection elide overlapping flags from delete_flags and set
+# For AFDO profile reinjection set:
 # problem_config.flags_to_replace.replace_flags={'-fprofile-sample-use':'/path/to/gwp.afdo','-fprofile-remapping-file':'/path/to/prof_remap.txt'}
 problem_config.flags_to_replace.replace_flags={}

compiler_opt/rl/train_locally.py

@@ -100,10 +100,10 @@ def train_eval(agent_name=constant.AgentName.PPO,
 
   logging.info('Loading module specs from corpus at %s.', FLAGS.data_path)
   cps = corpus.Corpus(
-    FLAGS.data_path,
-    additional_flags=problem_config.flags_to_add(),
-    delete_flags=problem_config.flags_to_delete(),
-    replace_flags=problem_config.flags_to_replace())
+      FLAGS.data_path,
+      additional_flags=problem_config.flags_to_add(),
+      delete_flags=problem_config.flags_to_delete(),
+      replace_flags=problem_config.flags_to_replace())
   logging.info('Done loading module specs from corpus.')
 
   dataset_fn = data_reader.create_sequence_example_dataset_fn(