[libcxx-commits] [PATCH] D153384: [libcxx] Include <sys/time.h> in posix_compat.h

[Petr Hosek via Phabricator via libcxx-commits]

phosek added a comment.

We noticed this in our downstream CI after D152377 <https://reviews.llvm.org/D152377> landed where we started getting build errors like these:

  In file included from /b/s/w/ir/x/w/llvm-llvm-project/libcxx/src/filesystem/directory_entry.cpp:16:
  /b/s/w/ir/x/w/llvm-llvm-project/libcxx/src/filesystem/time_utils.h:263:16: error: incomplete result type 'TimeVal' (aka 'std::filesystem::detail::(anonymous namespace)::timeval') in function definition
    263 | inline TimeVal make_timeval(TimeSpec const& ts) {
        |                ^


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153384