[PATCH] D95477: Replace use of deprecated gethostbyname by getaddrinfo.

Ahsan Saghir via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 14:20:47 PST 2021


saghir marked an inline comment as done.
saghir added inline comments.


================
Comment at: llvm/tools/llvm-jitlink/llvm-jitlink.cpp:694
+  addrinfo *AI;
+  addrinfo Hints = {};
+  Hints.ai_family = AF_INET;
----------------
rzurob wrote:
> IIRC, this syntax didn't build for me on Linux.  I suggest this instead:  addrinfo Hints{};
> 
Thanks for catching that. I was able to build with this on Ubuntu and RHEL(7.6 & 8.2). I believe the one that did not build was `addrinfo Hints = {0};`. I have changed this to `addrinfo Hints{};` now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95477



More information about the llvm-commits mailing list