Co-authored-by: Albert Magyar <albert.magyar@gmail.com>
Co-authored-by: Chick Markley <chick@qrhino.com>

Fixed problem creating Interval literals with full ranges

The BitPat.parse factory though did not remove these from the returned count.
This fixes that adds whitespace and underscores to the unit tests
This is an updated vesion of Chisel PR #1069

This is just a band aid until an Aggregate `isLit()` method (for which work has begun) is implemented.

This removes a dead line where a WriteEmitted phase is constructed.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

- boundary testing was not taking binary point into account correctly
- add tests to show where things work and where they are supposed to fail

 Bug fixes to support code for Interval

  - getLowestPossibleValue
  - getHighestPossibleValue
  - getPossibleValues

- Fixed computation in getHighestPossibleValue, erroneously was using lower intead of upper

This reverts commit 85fe90d5.

Update README to reflect CCC20 Extension

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

Data.isLit called Data.litArg which would trigger a Chisel runtime
deprecation warning in user code with source locator Data.scala:488

* Use macro to materialize CompileOptions in Chisel._

This switches from using an implicit val that required awkward
suppression (as illustrated in CompileOptionsSpec) to allowing
overriding in the same way as done in "import chisel3._" via the
creation of an implicit val in lexical scope.

* Deprecate Chisel.defaultCompileOptions

* Add CCC20 Info at README top

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

* Add Upcoming Events h2 Heading to README.md

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

* Add brief description of (current) chisel versioning and version recommendations.

* Respond to comments.
Replace "semantic versioning" with "a form of semantic versioning".

* Respond to comments - replace mmddyy with yyyymmdd.

* Mention change in SNAPSHOT naming.

Don't use MuxLookup default for full mapping

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>
Co-authored-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>
Co-authored-by: Albert Magyar <albert.magyar@gmail.com>

This modifies MuxLookup to not use the 'default' mapping argument if a
"full" mapping is provided. A "full" mapping enumerates all possible
cases for a 'key' argument of a known size. This will check literal
values to ensure exhaustiveness holds.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>
Co-authored-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>
Co-authored-by: Albert Magyar <albert.magyar@gmail.com>

Accomplished by changing the code gen for casting literals to
aggregates. Rather than connecting the literal to a wire that is then
bit selected from, just bit select from the literal which saves the
creation of an intermediate wire and matches FIRRTL's semantics for
legal async reset initial values.

Improve naming of anonymous/class-in-function Modules

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

This changes the desired name of a Module to provide non-numeric
naming for anonymous Modules and Modules defined inside function
bodies.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

Make ChiselStage targets not private

This enables users to use the nice run method of `ChiselStage` with their own set of phases.

Internal BoringUtils.bore Bug Fix

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>

This fixes a bug where internal boring using BoringUtils.bore would
fail because it was using instanceName which cannot be called before
the module closes. Previously, this meant that BoringUtils.bore would
work for boring instances (which are closed in a parent), but not for
boring signals in the current, unclosed module.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@ibm.com>