Open nickdesaulniers opened 1 week ago
Hi!
This issue may be a good introductory issue for people new to working on LLVM. If you would like to work on this issue, your first steps are:
test/
create fine-grained testing targets, so you can e.g. use make check-clang-ast
to only run Clang's AST tests.git clang-format HEAD~1
to format your changes.If you have any further questions about this issue, don't hesitate to ask via a comment in the thread below.
@llvm/issue-subscribers-good-first-issue
Author: Nick Desaulniers (paternity leave) (nickdesaulniers)
@llvm/issue-subscribers-libc
Author: Nick Desaulniers (paternity leave) (nickdesaulniers)
via https://github.com/llvm/llvm-project/pull/96591#discussion_r1653050200 it seems like we don't have
-Wunused
set.We should enable it and clean up the instances it finds.
We should append that to
compile_options
in libc/cmake/modules/LLVMLibCObjectRules.cmake.