There are two problems with the diamond implementation currently checked in:

*) the implementation is using some flag to suppress errors that doesn't seem to be used anymore (probably a leftover of the old algorithm)

*) diamond errors issued during constructor resolution does not go through standard channels (logResolveError) - instead, Resolve directly reports an error to Log.