Skip to content

[libc] pipe(2) linux syscall wrapper and unittest #85514

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.

Sign up for GitHub

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
wants to merge 12 commits into from
Closed

[libc] pipe(2) linux syscall wrapper and unittest #85514

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6a75de8
pipe(2) linux syscall wrapper and unittest
muffpy Mar 16, 2024
a8afc5c
Format patch using git-clang-format
muffpy Mar 16, 2024
174676f
Fix typo in Pipe2BadFlags test
muffpy Mar 16, 2024
c6fab70
Set correct errors and test titles based on Linux standard
muffpy Mar 16, 2024
30d2385
Remove unwanted headers from pipe2_test and fix Pipe2InvalidFlags test
muffpy Mar 16, 2024
53c43e3
Pass unsupported flags to pipe(2) in Pipe2InvalidFlags test
muffpy Mar 16, 2024
6100033
Add fcntl.h to access file control flags for Pipe2InvalidFlags test
muffpy Mar 16, 2024
7b87696
Fix header ordering and pipe(2) failure convention
muffpy Mar 18, 2024
863c2ab
pipe(2) functionality and error handling tests + watch queue uapi macros
muffpy Mar 19, 2024
63d9955
Clang formatting
muffpy Mar 19, 2024
4cf1101
Fix pipe(2) function spec + remove fallback to pipe()
muffpy Mar 19, 2024
95f743d
Heading comment formatting + ArgSpec TODO
muffpy Mar 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions libc/include/llvm-libc-macros/linux/watch-queue-macros.h
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
//===-- Definition of macros from watch-queue.h
//---------------------------------===//
//===-- Definition of macros from watch-queue.h ---------------------------------===//
Copy link
Author

@muffpy muffpy Mar 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

git clang-format does not like this. Please ignore for now as this file may not even be required.

Copy link
Contributor

@jhuber6 jhuber6 Mar 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That header is way too long, shorten it to 80 characters and it will stop doing this.

//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
Expand Down
2 changes: 1 addition & 1 deletion libc/spec/linux.td
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@ def Linux : StandardSpec<"Linux"> {
FunctionSpec<
"pipe2",
RetValSpec<IntType>,
[ArgSpec<IntPtr>, ArgSpec<IntType>]
[ArgSpec<IntPtr>, ArgSpec<IntType>] // TODO: use int[2] array for the first ArgSpec when possible
>,
]
>;
Expand Down
3 changes: 1 addition & 2 deletions libc/src/unistd/linux/pipe2.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
//===-- Linux implementation of pipe2
//---------------------------------------===//
//===-- Linux implementation of pipe2 -------------------------------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
Expand Down
3 changes: 1 addition & 2 deletions libc/src/unistd/pipe2.h
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
//===-- Implementation header for pipe2 ---------------------------*- C++
//-*-===//
//===-- Implementation header for pipe2 -------------------------*- C++ -*-===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
Expand Down
3 changes: 1 addition & 2 deletions libc/test/src/unistd/pipe2_test.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
//===-- Unittests for pipe2
//-------------------------------------------------===//
//===-- Unittests for pipe2 -----------------------------------------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
Expand Down