[PATCH] D35949: Guard print() functions only used by dump() functions.

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 13:23:52 PDT 2017


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D35949#822975, @MatzeB wrote:

> So I guess those print() functions are not publicly accessible (I guess we wouldn't warn about them otherwise). If the print functions aren't used otherwise, then why not merge them into the dump() functions?


Ah that makes sense. Go ahead then for those cases.


https://reviews.llvm.org/D35949





More information about the llvm-commits mailing list