[PATCH] D53238: [Driver] Add -static= to unify -static-{libgcc,libstdc++}

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 5 22:32:42 PDT 2019


MaskRay added a comment.

In D53238#1660259 <https://reviews.llvm.org/D53238#1660259>, @pirama wrote:

> Does this regress from existing behavior for unused argument warnings?  Currently, `-static-libstdc++ -nostdlib` issues an unused argument warning for `-static-libstdc++`, while AFAICT `-static=c++stdlib -nostdlib` doesn't.
>
> I'm not exactly sure how/where to issue this warning, though.


This change will suppress some unused argument warnings but I'll not call that "regression":)

Various toolchains already do things like:

  Args.ClaimAllArgs(options::OPT_g_Group);
  Args.ClaimAllArgs(options::OPT_emit_llvm);
  Args.ClaimAllArgs(options::OPT_w);

Hexagon and Myriad even do `Args.ClaimAllArgs(options::OPT_static_libgcc);` This diagnostic IMO has very low value.


Repository:
  rC Clang

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

https://reviews.llvm.org/D53238





More information about the cfe-commits mailing list