[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