Add -o option to gtkwave invocation when waveform type is VCD

It optimizes things out, and it's specially useful for large and complex projects.
machitgarha · Mar 28, 2023 · 3289df5 · 3289df5
1 parent 5eb070e
commit 3289df5
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/src/Console/Application.php b/src/Console/Application.php
@@ -7,7 +7,7 @@
 class Application extends \Symfony\Component\Console\Application
 {
     public const NAME = "Parvaj";
-    public const VERSION = "0.3.2";
+    public const VERSION = "0.4.0-alpha.1";
 
     public function __construct()
     {

diff --git a/src/Runner/Ghdl/GhdlRunner.php b/src/Runner/Ghdl/GhdlRunner.php
@@ -40,6 +40,9 @@ public function analyze(array $unitFilePaths): void
         ))->run();
     }
 
+    /**
+     * @todo Remove VCD file after this step, because of being huge in size.
+     */
     public function elabRun(string $testEntityName, ElabRunUserOptions $userOptions): string
     {
         (new Process(

diff --git a/src/Runner/Gtkwave/GtkwaveRunner.php b/src/Runner/Gtkwave/GtkwaveRunner.php
@@ -2,6 +2,8 @@
 
 namespace MAChitgarha\Parvaj\Runner\Gtkwave;
 
+use MAChitgarha\Parvaj\Runner\OptionBuilder;
+
 use MAChitgarha\Parvaj\WaveformType;
 
 use MAChitgarha\Parvaj\Util\Process;
@@ -15,9 +17,15 @@ public function __construct(
 
     public function open(string $waveformFilePath, string $waveformType): void
     {
+        $options = [];
+        if ($waveformType === WaveformType::VCD) {
+            $options["o"] = null;
+        }
+
         (new Process(
             [
                 $this->executable,
+                ...OptionBuilder::build($options),
                 $waveformFilePath,
             ]
         ))->run();