[PATCH] D30904: Allow suppressing host and target info in VersionPrinter

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 12:16:00 PDT 2017

chandlerc added a comment.

This seems like a totally nice direction, but I wonder if I could convince you to scope creep this a bit?

I feel like this shouldn't be guarded by a macro at all. Instead, for tools where it makes sense, we should have an API they can call (perhaps instead of or in addition to this) which provides this level of information in their version strings.

For example, I don't ever want clang-format or FileCheck to care about this. But I'd rather like opt, llc, and clang to always show it. And for those, it costs them nothing as this code is going to get linked in anyways.

What do you think?



More information about the llvm-commits mailing list