[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