Fix invalid PCH error

[eszpotanski]

This PR combines:

• Fix problem with precompiled headers and Verilator #736
• logging: quote arguments when there is a space in the arg when logging cmds #737

It changes the default CFLAGS for Verilator:

• -O1 changed to -Os as it is a default optimization used by Verilator, and using different ones can result in invalid PCH error (Verilator OPTs):

  cc1plus: error: one or more PCH files were found, but they were invalid
  cc1plus: note: use -Winvalid-pch for more information
  <command-line>: fatal error: VMockArrayTestbench__pch.h.fast: No such file or directory
  compilation terminated.
  

• -include V$topName.h removed, as it should not be used according to <command-line>: fatal error: VMockArrayTestbench__pch.h.fast: No such file or directory verilator/verilator#5244 (comment)

Moreover, PR improves formatting of displayed commands - e.g. values used for -CFLAGS are wrapped with quote.

[CLAassistant]

All committers have signed the CLA.

[oharboe]

@jackkoenig 🥳

[eszpotanski]

Unrelated CI fail, due to the timeout for newer Verilator version (like on main) and missing SBT on mac

[nndurj]

does this work with the latest verilator?

[eszpotanski]

Yes, I've tested it with verilator 5.026 and with master branch (Verilator 5.027 devel rev v5.026-77-g1600cc01a) using built-in test:

$ sbt "testOnly chiseltest.** -- -n RequiresVerilator"
[...]
[info] Run completed in 2 minutes, 1 second.
[info] Total number of tests run: 95
[info] Suites: completed 134, aborted 0
[info] Tests: succeeded 95, failed 0, canceled 0, ignored 1, pending 0
[info] All tests passed.
[success] Total time: 123 s (02:03), completed Aug 2, 2024, 11:11:44 AM

Changes were also tested on example from The-OpenROAD-Project/OpenROAD-flow-scripts#2120

[ekiwi]

Thanks!