This is an archive of the discontinued LLVM Phabricator instance.

libc++abi: Don't print debugging messages to stdout
ClosedPublic

Authored by ed on Mar 9 2015, 6:55 AM.

Details

Summary

There is some debugging code in cxa_demangle.cpp that prints messages on stdout. In general this is not safe, as the program itself may use stdout to write its output. Change this code to write to stderr.

Testing has revealed that this code is actually not reachable. Maybe it would make sense to remove it instead?

Diff Detail

Repository
rL LLVM

Event Timeline

ed updated this revision to Diff 21485.Mar 9 2015, 6:55 AM
ed retitled this revision from to libc++abi: Don't print debugging messages to stdout.
ed updated this object.
ed edited the test plan for this revision. (Show Details)
ed added a reviewer: nicholas.
ed set the repository for this revision to rL LLVM.
ed added a subscriber: Unknown Object (MLST).
ed added a reviewer: EricWF.Mar 14 2015, 3:34 AM

Adding another reviewer.

This revision was automatically updated to reflect the committed changes.