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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 4 17:34:16 PST 2022


MaskRay added inline comments.


================
Comment at: llvm/lib/Support/HTTPClient.cpp:153
+    return;
+  assert((Curl = curl_easy_init()) && "Curl could not be initialized.");
+  // Set the callback hooks.
----------------
For non-Debug build types, `LLVM_ENABLE_ASSERTIONS=off` is the default and `assert` is a no-op.

I fixed this in 3dd2d4c0a239ce78421d51491456cb1a075ad2f3


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112753



More information about the llvm-commits mailing list