[PATCH] D27805: [llvm-config] Print --system-libs only when static linking
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 15 08:41:58 PST 2016
mgorny added inline comments.
================
Comment at: tools/llvm-config/llvm-config.cpp:700
+ OS << LLVM_SYSTEM_LIBS << '\n';
+ }
+ }
----------------
compnerd wrote:
> The braces are unnecessary. You can probably even just use a ternary.
>
> OS << (LinkMode == LinkModeShared ? "" : LLVM_SYSTEM_LIBS) << '\n';
>
Yeah, that makes sense.
https://reviews.llvm.org/D27805
More information about the llvm-commits
mailing list