This is an archive of the discontinued LLVM Phabricator instance.

Hide dbgs() stream for when built with -fmodules.
ClosedPublic

Authored by marsupial on Jun 14 2017, 11:09 AM.

Details

Summary

Make DebugCounter::print and ump methods to be const correct.

Diff Detail

Repository
rL LLVM

Event Timeline

marsupial created this revision.Jun 14 2017, 11:09 AM
aprantl edited edge metadata.Jun 14 2017, 11:18 AM

I'm slightly confused why this works as a fix for the Debug build. Isn't this patch a no-op for a debug build?

aprantl accepted this revision.Jun 14 2017, 11:19 AM

That said, this patch is still obviously correct.

This revision is now accepted and ready to land.Jun 14 2017, 11:19 AM
marsupial edited the summary of this revision. (Show Details)

This should be more 'obviously' correct.
Sorry, some CMake confusion here....

Formatting.

Remove spurious newline.

This revision was automatically updated to reflect the committed changes.

LGTM, thanks!