Merge pull request #320 from g-r-a-n-t/fix-optimize

Fix optimize flag.
ethereum · Mar 19, 2021 · 1e5ce10 · 1e5ce10
2 parents f5e10e1 + e6c348e
commit 1e5ce10
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/newsfragments/320.bugfix.md b/newsfragments/320.bugfix.md
@@ -0,0 +1,11 @@
+The CLI was using the overwrite flag to enable Yul optimization.
+
+i.e.
+
+```
+# Would both overwite output files and run the Yul optimizer. 
+$ fe my_contract.fe --overwrite
+```
+
+
+Using the overwrite flag now only overwrites and optimization is enabled with the optimize flag.
diff --git a/src/main.rs b/src/main.rs
@@ -76,7 +76,7 @@ pub fn main() {
     let input_file = matches.value_of("input").unwrap();
     let output_dir = matches.value_of("output-dir").unwrap();
     let overwrite = matches.is_present("overwrite");
-    let optimize = matches.is_present("overwrite");
+    let optimize = matches.is_present("optimize");
     let targets =
         values_t!(matches.values_of("emit"), CompilationTarget).unwrap_or_else(|e| e.exit());