Unverified Commit e46a4a36 authored by mergify[bot]'s avatar mergify[bot] Committed by GitHub
Browse files

Fix ChiselStage and Builder handling of logging (backport #3895) (#3898)

* Fix ChiselStage and Builder handling of logging (#3895)

Previously, object circt.stage.ChiselStage was ignoring the Logger.
Also, Chisel was not creating its own logger scope which could lead to
clobbering of the Console when running invoking Chisel in the same
process multiple times.

Fix various places we had to workaround this behavior and fix tests
checking --log-level debug.

(cherry picked from commit 88d147d9

)

# Conflicts:
#	src/main/scala/circt/stage/ChiselStage.scala

* Resolve backport conflicts

* Make logger annotations unserializable

Change logger annotations to mix-in the Unserializable trait so that they
will not emitted by a stage.  These annotations are not intended to be
seen by CIRCT and these should be stripped from the output FIRRTL text.

Signed-off-by: default avatarSchuyler Eldridge <schuyler.eldridge@sifive.com>

---------

Signed-off-by: default avatarSchuyler Eldridge <schuyler.eldridge@sifive.com>
Co-authored-by: default avatarJack Koenig <koenig@sifive.com>
Co-authored-by: default avatarSchuyler Eldridge <schuyler.eldridge@sifive.com>
parent 31c10343
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment