The main code for the FILE struct is only enabled on platforms that it
works on, but before this patch the tests were included unconditionally.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I thought a test was already skipped if it had missing deps. But, it looks like we skip a test only if it has skipped entrypoints: https://github.com/llvm/llvm-project/blob/main/libc/cmake/modules/LLVMLibCTestRules.cmake#L32.
While this change LGTM for now, I think a more robust way to skip tests would be by skipping if there are any missing deps. It can be done as a separate cleanup.