kda added a comment.

In D127145#3561519 <https://reviews.llvm.org/D127145#3561519>, @eugenis wrote:

> The interceptor (dn_expand) is only defined on linux && !android, while this 
> code links libresolv on *bsd and others, too. I think it's ok, as long as all 
> platforms with the interceptor are covered.
>
> It would be great to mention the actual reason we do this in the commit 
> message: because otherwise link tests in ./configure scripts may be confused 
> by the presence of the dn_expand symbol even without -lresolv (because of the 
> interceptor) and get a runtime error later.

Updated summary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127145

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to