Re: PATCH: replace cpu_trace_stream subclass with rdbuf swap

Hi -


blandy wrote:

> Below is a patch against the current SID sources to make
> sidutil::basic_cpu::trace_stream into an ordinary std::ostream object,
> which fixes the bug described in my earlier message, but does so by
> removing hair from sidcputil.h (and a bit elsewhere), instead of
> adding to it.

Looks like an improvement, thanks!

> It builds, but I haven't tested it.  The original patch was developed
> on an internal Red Hat branch and tested against an internal Red Hat
> cpu.

Does this version of the code run on that internal port?

> The full SID takes an awful long time to build, and I'm not sure what
> toolchain and BSP to match it against.  Do folks here have any advice
> on how to pare down the SID build and which architecture to target to
> test this with as little fuss as possible?

The usual tricks are to configure sid with a single simple --target (say
m32r-elf), with --disable-shared to omit shared library building, and
build with CXXFLAGS excluding -O2 for a quicker compilation run.


- FChE