search

llvm / circt

Circuit IR Compilers and Tools
https://circt.org
Other
1.57k stars 277 forks source link

[FIRRTL] add default case for EventControl instantiation #7210

Closed sequencer closed 6 days ago

sequencer commented 1 week ago

This patch fix the compiler warning maybe-uninitialized

dtzSiFive commented 1 week ago

https://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations

If it's covered please don't add a default.