[PATCH] D109977: LLVM Driver Multicall tool

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 5 14:21:58 PDT 2022


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

LGTM from my perspective, but this needs someone else's review.



================
Comment at: llvm/tools/llvm-driver/llvm-driver.cpp:50
+  bool ConsumeFirstArg = false;
+  if (LaunchedTool == "llvm") {
+    if (Argc < 2)
----------------
abrachet wrote:
> MaskRay wrote:
> > Some distributions may want to use something like llvm-15. See some binary utilities how the version is handled.
> Thank's I've taken this from objcopy's code
The format of `cl::PrintVersionMessage();` is not so good for user-facing tools. Consider omitting it.

`llvm-objcopy --version` should probably use a style similar to `clang --version` but the priority isn't high.


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

https://reviews.llvm.org/D109977



More information about the cfe-commits mailing list