User Details
- User Since
- Sep 21 2020, 2:09 PM (17 w, 4 d)
Wed, Jan 20
Tue, Jan 19
Thu, Jan 14
This change was accepted and landed. (Github link: https://github.com/llvm/llvm-project/commit/a0b65a7bcd6065688189b3d678c42ed6af9603db#diff-16c8c6eb85e05438f5d6c60ff9869072a3a3b1618aa1481ac7a0cb049f06f51d )
Wed, Jan 13
fix phrasing in commit message
fixed phrasing in comment
fix the headerguards and add a comment explaining the undef in FuchsiaTest.h
add headerguards to the new Test.h and FuchsiaTest.h
Tue, Jan 12
add the functions to the spec in the correct places,
and add them to the entrypoints list for aarch64
Mon, Jan 11
LGTM
Fri, Jan 8
fix the previous patch, the macro was placed on the wrong line.
Add the macro definition for fuchsia (or others) to hook into for extra attributes on the libc functions
Thu, Jan 7
submit comment
update the macro in common.h to force the decltypes to use the definition in the __llvm_libc namespace
update fmaf.cpp to be in line with the rest of the functions.
Wed, Jan 6
fix the alias name in common.h.def
Dec 16 2020
accepted with nits
Dec 11 2020
LGTM
Dec 4 2020
final cleanup
Dec 2 2020
submit comments
added more tests
Dec 1 2020
remove references to targetos
after talking with Siva about it it looks like we're going to stick with this for the moment and not go with lit tests. The reason is that currently the wrappergen unit tests about as dependent on LLVM support as wrappergen itself, adding lit tests would make portability potentially harder.
add testapi.td so that the tests are no longer platform dependent and clean up the code a little.
Nov 30 2020
submit comments
address most of the comments
I'm not quite sure why a lit test would be better in this situation, does it have better support for testing complete binaries?
Nov 25 2020
Nov 20 2020
remove commented out include
Nov 19 2020
submit comments
This change finishes the work started in the previous one, all of the tests now pass.
Big thank you to abrachet for the comments.
Nov 18 2020
submit comments
(NOT QUITE WORKING) get the rest of the llvm dependencies out StreamWrapper and FDReader
Nov 17 2020
I guess I've got some more changes ahead of me. This new commit (linked here) is where I plan on putting the next set, and it currently has a few based on what you've said.
switch the tests to using the global headers to match the sources they're testing.
update: I was reminded of the other patch that I'm currently in the process of getting reviewed (linked here: https://reviews.llvm.org/D91394) that will make the string and math tests independent of headergen since they can use the system libc's header. This means if I can make the unit tests fully independent then you can bring up both the string and math parts of LLVM-libc without the whole project.
Do our tests depend on the generated headers? I thought they did, if so it will be very hard to remove our dependency on the rest of llvm.
Nov 16 2020
reset the #includes for everything not in src/string and src/math
Nov 12 2020
Nov 6 2020
lgtm with nits
LGTM
Nov 5 2020
submit messages (I really gotta remember to mark these before I run the arc command)
make error message more helpful
submit comments
add check for a void argument as well as other arguments in the same function.
submit comments
change continue to break
Nov 4 2020
Nov 3 2020
added header guard to llvmlibc_errno.h, and a comment on assert.h explaining the lack of header guard (thank you to abrachet).
fix missing newline at the end of a file.
move the errno macro to its own file to match assert.
update the assert files to also match
Nov 2 2020
Oct 19 2020
Oct 16 2020
remove unnecessary comment
fix commit message locally
Oct 15 2020
Fixed nits.
Minor formatting fixes.
Oct 14 2020
Fix formatting and move data path to PublicAPICommand
Remove the redundant new parser