Skip to content

[libc][POSIX] implement poll from poll.h #124647

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
nickdesaulniers opened this issue Jan 27, 2025 · 2 comments · Fixed by #125118
Closed

[libc][POSIX] implement poll from poll.h #124647

nickdesaulniers opened this issue Jan 27, 2025 · 2 comments · Fixed by #125118
Assignees
Labels

Comments

@nickdesaulniers
Copy link
Member

building clang against llvm-libc fails with:

llvm-project-main/llvm/lib/Support/raw_socket_stream.cpp:24:10: fatal error: 'poll.h' file not found.
   24 | #include <poll.h>
      |           ^~~~~~~~

it looks like poll is used.

@llvmbot
Copy link
Member

llvmbot commented Jan 27, 2025

@llvm/issue-subscribers-libc

Author: Nick Desaulniers (nickdesaulniers)

building clang against llvm-libc fails with: ``` llvm-project-main/llvm/lib/Support/raw_socket_stream.cpp:24:10: fatal error: 'poll.h' file not found. 24 | #include <poll.h> | ^~~~~~~~ ``` it looks like `poll` is used.

@nickdesaulniers
Copy link
Member Author

nickdesaulniers commented Jan 30, 2025

libc/test/UnitTest/ExecuteFunctionUnix.cpp also depends on poll, the definition of struct pollfd and POLLHUP, so this is also blocking making the unit tests hermetic. #125092

@nickdesaulniers nickdesaulniers self-assigned this Jan 30, 2025
nickdesaulniers added a commit to nickdesaulniers/llvm-project that referenced this issue Jan 30, 2025
Icohedron pushed a commit to Icohedron/llvm-project that referenced this issue Feb 11, 2025

Verified

This commit was signed with the committer’s verified signature.
Icohedron Deric C.
Simple syscall.

Fixes: llvm#124647
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants