[PATCH] D59333: ELF: Rename Configuration::Static to Bstatic. NFCI.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 17 08:20:48 PDT 2019


MaskRay added a comment.

In D59333#1432433 <https://reviews.llvm.org/D59333#1432433>, @grimar wrote:

> In D59333#1428331 <https://reviews.llvm.org/D59333#1428331>, @ruiu wrote:
>
> > Maybe we should also update the helptext of Bstatic? It currently says that "Do not link against shared libraries" which doesn't sound like it captures the exact meaning of the option.
>
>
> "Do not search for dynamic libraries when linking"?


Is it worth mentioning the positional behavior? "Do not search for shared libraries mentioned on the command line after the option"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59333





More information about the llvm-commits mailing list