diff options
| author | Schuyler Eldridge | 2018-06-21 15:05:31 -0400 |
|---|---|---|
| committer | Jack Koenig | 2018-06-21 12:05:31 -0700 |
| commit | f86d72c9cd2526fc33528cc6a8023d1c313c4eb2 (patch) | |
| tree | b73f16fc2eca4e2935d965dde87e21f3c418df19 /src/main | |
| parent | 61cf31d35c5bf22bebdcd959eac2cb8ff3574f33 (diff) | |
--infer-rw should take no argument (#829)
This fixes --infer-rw to not expect an argument. After the annotations
refactor, no option was required, but some legacy code remained.
This also updates the test cases to be more correct and not specify an
option to --infer-rw.
Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>
Diffstat (limited to 'src/main')
| -rw-r--r-- | src/main/scala/firrtl/ExecutionOptionsManager.scala | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/main/scala/firrtl/ExecutionOptionsManager.scala b/src/main/scala/firrtl/ExecutionOptionsManager.scala index b359ba5e..f8caf842 100644 --- a/src/main/scala/firrtl/ExecutionOptionsManager.scala +++ b/src/main/scala/firrtl/ExecutionOptionsManager.scala @@ -406,9 +406,8 @@ trait HasFirrtlOptions { """Inline one or more module (comma separated, no spaces) module looks like "MyModule" or "MyModule.myinstance""" } - parser.opt[String]("infer-rw") + parser.opt[Unit]("infer-rw") .abbr("firw") - .valueName ("<circuit>") .foreach { x => firrtlOptions = firrtlOptions.copy( annotations = firrtlOptions.annotations :+ InferReadWriteAnnotation, @@ -585,4 +584,3 @@ class ExecutionOptionsManager(val applicationName: String) extends HasParser(app s"$directoryName$baseName$normalizedSuffix" } } - |