User Details
- User Since
- Sep 21 2020, 2:09 PM (22 w, 2 d)
Mon, Feb 1
Fri, Jan 29
fix the comments and change the final test in RaiseAndTest
Thu, Jan 28
update all of the EXPECT_DEATH calls
clarify the type for the constant being shifted
documentation changes
Tue, Jan 26
This change was landed manually since it was too big for arcanist. The change is here and then the follow up change is here.
Jan 20 2021
Jan 19 2021
Jan 14 2021
This change was accepted and landed. (Github link: https://github.com/llvm/llvm-project/commit/a0b65a7bcd6065688189b3d678c42ed6af9603db#diff-16c8c6eb85e05438f5d6c60ff9869072a3a3b1618aa1481ac7a0cb049f06f51d )
Jan 13 2021
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
Jan 12 2021
add the functions to the spec in the correct places,
and add them to the entrypoints list for aarch64
Jan 11 2021
LGTM
Jan 8 2021
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
Jan 7 2021
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.
Jan 6 2021
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.