User Details
- User Since
- Jun 19 2014, 10:59 AM (418 w, 6 d)
Apr 5 2022
@ldionne PTAL
Jan 19 2022
Jan 18 2022
Jan 10 2022
OS_NAME MATCHES "Darwin|Linux|FreeBSD|Android|NetBSD" AND ANDROID_PLATFORM_LEVEL GREATER 23
Dec 10 2021
LGTM once the comments are addressed. (I'd also approve the alternative approach discussed in the thread of searching each older API level directory, but have no preference one way or the other)
Dec 7 2021
Dec 3 2021
Dec 2 2021
Dec 1 2021
Nov 30 2021
Doesn't seem like we use this for Android, but @pirama just in case.
Nov 19 2021
Nice, that's a lot of code cleaned up! LGTM, but probably should wait for someone from Apple to weigh in. I think the new formatting for those error messages is better for them too but that's not my call :)
Nov 17 2021
Fine by me if this is the default for LLD.
Nov 16 2021
Nov 10 2021
Nov 8 2021
Fixes LGTM.
Oct 1 2021
Odd. Thanks for confirming. We'll revert while we try to figure out what it is in that builders config that is uncovering the bug. I don't think the default config uses libc++, so probably that?
Are you sure that's caused by this change? We didn't see that error.
Sep 29 2021
LGTM. @vitalybuka, do you want to have one last look, or are we clear to merge?
Sep 28 2021
Sep 27 2021
Sep 17 2021
LGTM but we should wait for @vitalybuka.
Sep 16 2021
Sep 10 2021
I think this is probably fine from the NDK perspective. Even before Clang defaulted to C++11 or newer, our build system did that when using libc++. Other build systems may not have, but unfortunately I'm not currently able to get at the data to get a concrete answer.
Sep 2 2021
Sep 1 2021
(I'd still like to hear what vitalybuka/eugenis have to say, but LGTM)
Aug 19 2021
A couple stray pieces snuck in. Could you revert those and update the change? It looks like you also missed the mailing list, but we can fix that once the patch is fixed.
Aug 17 2021
Jul 30 2021
LGTM, but @rprichard knows this area better than I do so he should probably weigh in before submitting :)
May 19 2021
Fix remaining tests for the deferred test configuration.
May 13 2021
Adds XFAIL-BUILD to deal with the deferred-run cases that fail to build.
May 12 2021
Initial pass at my suggestion of deferring XFAIL handling to the bundler. This is incomplete because we'll need to differentiate XFAIL-BUILD and XFAIL-RUN in tests (running this as-is makes any test that is XFAIL because it doesn't _build_ turn to FAIL, but things that fail to _run_ are handled correctly), but it's enough to get feedback on the idea.
May 4 2021
Re problems 1 and 2 (xfail handling), my idea what that we'd add --xfail to the bundler interface and delegate to the test runner to handle that. For Android I need to fit these tests into tradefed, so our bundler would work by creating the test config files for tradefed and would let it handle xfails. We don't have much reason to interact with LIT again after the tests are built. This more or less matches how we've been running libc++ tests for years, though with some improvements (xfail info is currently manually tracked rather than being given by LIT). I've been carrying a local patch that adds a --build-only flag which just ignores xfail annotations and defers all the execution and interpretation to our test runner.
Mar 5 2021
Feb 19 2021
It sounds like everyone is happy here but the tools. Could we get a libunwind reviewer (preferably @compnerd, since his review is the red one) to LGTM this?
Feb 10 2021
Feb 3 2021
We've since added -fuse-ld=lld to the three CMAKE_*_FLAGS so I think we're likely set on our end.
Jan 29 2021
Jan 28 2021
Jan 27 2021
Jan 26 2021
Jan 21 2021
Thanks! Has been up for a while and I haven't heard any objections. Will submit and we can revert if issues are found.
Dec 14 2020
Ping.
Dec 2 2020
Nov 17 2020
Nov 6 2020
Ping
Oct 28 2020
Oct 27 2020
Oct 26 2020
Oct 12 2020
LGTM for Android.
Sep 23 2020
Aug 20 2020
Jun 17 2020
Jun 1 2020
May 21 2020
The NDK still supports linkers other than LLD, but we are changing the default to LLD in the next release. I'd prefer to keep this for the time being, but don't feel strongly about it.
Apr 29 2020
Apr 24 2020
Apr 23 2020
@srhines to confirm that this won't break our toolchain build, and @rprichard because he's the one looking more closely at libunwind these days.
Apr 16 2020
Option 4 was (at least on the surface) super easy: https://reviews.llvm.org/D78328. lmk if you'd prefer that approach. I'm slightly less confident in it since it affects non-Android platforms as well.
@MaskRay Any other ideas, or should I submit this? Reviewing all our options:
Apr 7 2020
Added missing copyright header and moved into the libcxx specific test directory.
Apr 6 2020
Mar 30 2020
Mar 26 2020
It seems I'd goofed something in my testing earlier (I think I still had -fuse-ld=lld force on in my build system). While Clang will find ld in the driver directory and prefer it, LLD defaults to the Darwin driver mode when argv[0] is ld when run on Darwin. We need GNU mode, and the best way to get this behavior is to have Clang invoke ld.lld instead.