Krzysztof Bieganski

This patch: ```diff diff --git a/src/V3LinkJump.cpp b/src/V3LinkJump.cpp index e777ac164..5687fa626 100644 --- a/src/V3LinkJump.cpp +++ b/src/V3LinkJump.cpp AstVar* const varp = new AstVar{nodep->fileline(), VVarType::BLOCKTEMP, name, nodep->findSigned32DType()}; varp->lifetime(VLifetime::AUTOMATIC); + varp->funcLocal(true); varp->usedLoopIdx(true); - m_modp->addStmtsp(varp); +...

> Improve V3Dead. I think most of the generated classes really aren't used, and could be not be generated Yes, though so far this defect has proven to be useful...

Ah, no, I've only been building without the GUI. I'll try it, and if it doesn't work, I'll add another condition that checks if `BUILD_GUI=off`.

| | Full build | Linking only | |:-|-:|-:| | No LTO | 8 min 23 s | 5 s | | With LTO | 9 min 23 s |...

If we're not going to enable it by default, there's no point in this PR. Closing it. For future reference: * If you want to build with LTO, use `-DCMAKE_INTERPROCEDURAL_OPTIMIZATION=ON`...

For sure, you don't want LTO while developing. I'll get back to this then.

I'm busy with other stuff at the moment, so if anyone is willing to take over, go ahead.

94% of the time is spent in: https://github.com/The-OpenROAD-Project/OpenROAD/blob/441581bc5af322bc122e6a018d8d1bbdab0649e5/src/rsz/src/RecoverPower.cc#L81-L226 It's doing ~4500 of iterations of this: https://github.com/The-OpenROAD-Project/OpenROAD/blob/441581bc5af322bc122e6a018d8d1bbdab0649e5/src/rsz/src/RecoverPower.cc#L126 Each iteration is about 1s, so it adds up. There are three parameters that...

@oharboe Can you try this again with the current `master`? Is the performance acceptable now?

> Fix the Surelog project definitions for each module. > They seem all wrong. Missing includes and all In some cases, yeah, but even modules that don't have missing includes...