[PATCH] D137837: [Support] Move Target/CPU Printing out of CommandLine

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 04:08:00 PST 2022


lenary added a comment.

In D137837#3976344 <https://reviews.llvm.org/D137837#3976344>, @MaskRay wrote:

> I think this patch can drop `cl::AddExtraVersionPrinter(sys::printDefaultTargetAndDetectedCPU);` from most tools. Only add them case by case when there is consensus there is a significant value.
>
> Many tools get the default target and CPU information by the generic code and it seems that most don't care about the information from the non-flagship tools.

I have left the info in `llc` and `opt`. All other tools no longer have this info, but I hope it is clear how to re-add it if people want that in future.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137837/new/

https://reviews.llvm.org/D137837



More information about the llvm-commits mailing list