Skip to content

Commit 1ad7689

Browse files
Removed num_modules flag from RL training in demo (#149)
We refactored this flag a while ago to be specified in the gin config for the relevant problem to improve reproducibility, especially in the regalloc case, but I forgot to remove the flag from the demo. This patch removes the flag from the demo, making everything (hopefully) plug and play.
1 parent 33d2dcc commit 1ad7689

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

docs/demo/demo.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,6 @@ rm -rf $OUTPUT_DIR && \
308308
--data_path=$CORPUS \
309309
--gin_bindings=clang_path="'$LLVM_INSTALLDIR/bin/clang'" \
310310
--gin_bindings=llvm_size_path="'$LLVM_INSTALLDIR/bin/llvm-size'" \
311-
--num_modules=100 \
312311
--gin_files=compiler_opt/rl/inlining/gin_configs/ppo_nn_agent.gin \
313312
--gin_bindings=train_eval.warmstart_policy_dir=\"$WARMSTART_OUTPUT_DIR/saved_policy\"
314313
```

0 commit comments

Comments
 (0)