[PATCH] D112753: [llvm] [Support] Add CURL HTTP Client.

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/lib/Support/InitLLVM.cpp:63
+
+  if (HTTPClient::isAvailable())
+    HTTPClient::initialize();
----------------
Does "isAvailable" need to be checked? The non-curl implementation doesn't fail on initialize or cleanup, I think? It's just a no-op there. So maybe calling these without the conditionality would be OK?


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

https://reviews.llvm.org/D112753