Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

danalbert (Dan Albert)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 19 2014, 10:59 AM (484 w, 3 d)

Recent Activity

Aug 31 2023

danalbert added a reviewer for D159292: [driver] Conditionally include installed libc++ headers for Android: rprichard.
Aug 31 2023, 11:26 AM · Restricted Project, Restricted Project

Jun 20 2023

danalbert added a comment to D139147: [libc++][Android] Enable libc++ testing on Android.

I see that libcxx only supports the last two stable releases of Clang (15 and 16), but the latest NDK is r25c, which is between Clang 13 and 14. NDK r26 should be released sometime this year with a compiler newer than Clang 16.

How often are Android NDKs released? I would like to know how often we Android uses compilers we no longer officially support.

Jun 20 2023, 4:09 PM · Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project

May 5 2023

danalbert accepted D149980: Remove unused basic_android_tree/mipsel-linux-android.
May 5 2023, 1:40 PM · Restricted Project, Restricted Project

Apr 17 2023

danalbert accepted D147849: [Android] Use ELF TLS for Android API level 29+.

The rest of us are just echoing rprichard since he's the expert, but done :)

Apr 17 2023, 2:24 PM · Restricted Project, Restricted Project

Apr 10 2023

danalbert added a comment to D147849: [Android] Use ELF TLS for Android API level 29+.

+1, but I'd like @rprichard to be the one to actually sign off since he'll be the one to know if there are any caveats.

Apr 10 2023, 2:13 PM · Restricted Project, Restricted Project

Feb 15 2023

danalbert added a comment to D63904: [Android] Use ELF TLS for Android API level 29+.

+1, but like @eugenis says, it needs a test.

Feb 15 2023, 3:06 PM · Restricted Project, Restricted Project

Dec 7 2022

danalbert accepted D139502: [libc++][Android] Disable pbump2gig for Android.

LGTM, but note that rprichard is going to be a better POC for Android libc++ in the future :)

Dec 7 2022, 10:28 AM · Restricted Project, Restricted Project

Dec 6 2022

danalbert added a comment to D139380: [libc++][Android] Rename user NS to User to avoid conflict with struct.

Would you be able to report this as a bug to Android? Perhaps the struct can be removed when a macro is provided and they can start some sort of deprecation path?

Dec 6 2022, 3:58 PM · Restricted Project, Restricted Project

Oct 14 2022

danalbert added a comment to D135925: Fix LazyInitialization in tsan.

BIonic supports it, but the sanitizer cannot use it for Android because sanitizers are shared libraries on Android, and preinit array only works for excutables.

Oct 14 2022, 1:30 PM · Restricted Project, Restricted Project

Oct 13 2022

danalbert added a comment to D135402: [LLD] Enable --no-undefined-version by default..

What would you suggest for solving [differing exported symbols between debug and release builds]?

Oct 13 2022, 3:27 PM · Restricted Project, Restricted Project
danalbert added a comment to D135402: [LLD] Enable --no-undefined-version by default..

Okay. Unfortunately it's unrealistic that I'll ever have the time to chase every build bot, so unless someone else can volunteer to do that I don't think this default will ever improve.

Oct 13 2022, 2:22 PM · Restricted Project, Restricted Project
danalbert added a comment to D135402: [LLD] Enable --no-undefined-version by default..

Why was an LLD patch reverted as a result of GNU ld not supporting a flag?

Oct 13 2022, 2:15 PM · Restricted Project, Restricted Project
danalbert accepted D135421: Driver: Change default Android linker to lld..
Oct 13 2022, 12:25 PM · Restricted Project, Restricted Project

Oct 12 2022

danalbert committed rG7ec8b0d162e3: [LLD] Enable --no-undefined-version by default. (authored by danalbert).
[LLD] Enable --no-undefined-version by default.
Oct 12 2022, 12:30 PM · Restricted Project
danalbert closed D135402: [LLD] Enable --no-undefined-version by default..
Oct 12 2022, 12:29 PM · Restricted Project, Restricted Project

Oct 6 2022

danalbert updated the diff for D135402: [LLD] Enable --no-undefined-version by default..

Remove man doc for default behavior.

Oct 6 2022, 3:22 PM · Restricted Project, Restricted Project
danalbert added a comment to D135402: [LLD] Enable --no-undefined-version by default..

Please update lld/docs/ReleaseNotes.rst and flip lld/docs/ld.lld.1 to use --undefined-version instead.

Oct 6 2022, 2:43 PM · Restricted Project, Restricted Project
danalbert updated the diff for D135402: [LLD] Enable --no-undefined-version by default..

Update docs.

Oct 6 2022, 2:42 PM · Restricted Project, Restricted Project
danalbert published D135402: [LLD] Enable --no-undefined-version by default. for review.
Oct 6 2022, 2:16 PM · Restricted Project, Restricted Project

Aug 12 2022

danalbert accepted D131792: [libunwind] Remove __ANDROID_API__ < 18 workaround.
Aug 12 2022, 10:45 AM · Restricted Project, Restricted Project, Restricted Project

Apr 5 2022

danalbert added a comment to D122593: [libcxx] locale_bionic.h: skip ndk-version.h on Android platform.

@ldionne PTAL

Apr 5 2022, 2:47 PM · Restricted Project, Restricted Project

Jan 19 2022

danalbert accepted D108394: add tsan shared library.
Jan 19 2022, 4:13 PM · Restricted Project, Restricted Project
danalbert added inline comments to D108394: add tsan shared library.
Jan 19 2022, 12:55 PM · Restricted Project, Restricted Project

Jan 18 2022

danalbert added inline comments to D108394: add tsan shared library.
Jan 18 2022, 6:17 PM · Restricted Project, Restricted Project

Jan 10 2022

danalbert requested changes to D108394: add tsan shared library.

OS_NAME MATCHES "Darwin|Linux|FreeBSD|Android|NetBSD" AND ANDROID_PLATFORM_LEVEL GREATER 23

Jan 10 2022, 12:37 PM · Restricted Project, Restricted Project

Dec 10 2021

danalbert accepted D115049: Fall back on Android triple w/o API level for runtimes search.

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 10 2021, 1:58 PM · Restricted Project
danalbert added inline comments to D115058: Add tsan shared library in Android version.
Dec 10 2021, 1:25 PM · Restricted Project, Restricted Project

Dec 7 2021

danalbert accepted D115254: Use VersionTuple for parsing versions in Triple.
Dec 7 2021, 2:31 PM · Restricted Project, Restricted Project

Dec 3 2021

danalbert added inline comments to D115058: Add tsan shared library in Android version.
Dec 3 2021, 12:22 PM · Restricted Project, Restricted Project

Dec 2 2021

danalbert accepted D114885: Use VersionTuple for parsing versions in Triple, fixing issues that caused the original change to be reverted. This makes it possible to distinguish between "16" and "16.0" after parsing, which previously was not possible..
Dec 2 2021, 1:08 PM · Restricted Project, Restricted Project

Dec 1 2021

danalbert accepted D114885: Use VersionTuple for parsing versions in Triple, fixing issues that caused the original change to be reverted. This makes it possible to distinguish between "16" and "16.0" after parsing, which previously was not possible..
Dec 1 2021, 5:33 PM · Restricted Project, Restricted Project

Nov 30 2021

danalbert accepted D114473: [runtimes] Remove support for GCC-style 32 bit multilib builds.

Doesn't seem like we use this for Android, but @pirama just in case.

Nov 30 2021, 2:21 PM · Restricted Project, Restricted Project, Restricted Project

Nov 19 2021

danalbert accepted D114163: Use VersionTuple for parsing versions in Triple. This makes it possible to distinguish between "16" and "16.0" after parsing, which previously was not possible..

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 19 2021, 11:35 AM · Restricted Project, Restricted Project

Nov 17 2021

danalbert accepted D113840: [Driver][Android] Remove unneeded isNoExecStackDefault.

Fine by me if this is the default for LLD.

Nov 17 2021, 12:44 PM · Restricted Project

Nov 16 2021

danalbert accepted D114036: Add Android test case for -Wpartial-availability. Also update Android availability tests to match on the whole string, so we can distinguish between "Android 16" and "Android 16.0.0" at the end of warning messages..
Nov 16 2021, 2:53 PM · Restricted Project

Nov 10 2021

danalbert accepted D113370: [Driver] Change Linux::isPIEDefault to true for all Android versions.
Nov 10 2021, 10:21 PM · Restricted Project

Nov 8 2021

danalbert accepted D108394: add tsan shared library.

Fixes LGTM.

Nov 8 2021, 12:05 PM · Restricted Project, Restricted Project

Oct 1 2021

danalbert added a comment to rG91bfccf83733: add tsan shared library.

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?

Oct 1 2021, 11:05 AM
danalbert added a comment to rG91bfccf83733: add tsan shared library.

Are you sure that's caused by this change? We didn't see that error.

Oct 1 2021, 10:37 AM

Sep 29 2021

danalbert accepted D108394: add tsan shared library.

LGTM. @vitalybuka, do you want to have one last look, or are we clear to merge?

Sep 29 2021, 2:08 PM · Restricted Project, Restricted Project

Sep 28 2021

danalbert requested changes to D108394: add tsan shared library.
Sep 28 2021, 1:49 PM · Restricted Project, Restricted Project

Sep 27 2021

danalbert accepted D110379: [Driver] Remove confusing *-linux-android detection with non-android --target=.
Sep 27 2021, 1:19 PM · Restricted Project

Sep 17 2021

danalbert accepted D108394: add tsan shared library.

LGTM but we should wait for @vitalybuka.

Sep 17 2021, 2:10 PM · Restricted Project, Restricted Project

Sep 16 2021

danalbert added a comment to D108394: add tsan shared library.

I guess testing is getting more complicated.
Maybe we should separate tests and non-test patches here? I suspect we well have several of them. Even after lit will run them as expected, they will likely just crash on start.
WDYT?

Sep 16 2021, 1:16 AM · Restricted Project, Restricted Project

Sep 10 2021

danalbert accepted D109459: [libc++][ABI BREAK] Do not use the C++03 emulation for std::nullptr_t by default.

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 10 2021, 2:42 PM · Restricted Project

Sep 2 2021

danalbert added inline comments to D108394: add tsan shared library.
Sep 2 2021, 12:57 PM · Restricted Project, Restricted Project

Sep 1 2021

danalbert accepted D108394: add tsan shared library.

(I'd still like to hear what vitalybuka/eugenis have to say, but LGTM)

Sep 1 2021, 4:47 PM · Restricted Project, Restricted Project
danalbert added inline comments to D108394: add tsan shared library.
Sep 1 2021, 12:37 PM · Restricted Project, Restricted Project

Aug 19 2021

danalbert requested changes to D108394: add tsan shared library.

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 19 2021, 1:23 PM · Restricted Project, Restricted Project

Aug 17 2021

danalbert committed rGd42be2d63ccf: Remove unused imports. (authored by danalbert).
Remove unused imports.
Aug 17 2021, 1:05 PM

Jul 30 2021

danalbert added a comment to D102570: [ELF] Add -Bsymbolic-non-weak-functions.

Do you plan to land this? I thought of it as potentially useful for https://github.com/android/ndk/issues/1551. The issue has the full details, but the summary is that we only want a single copy of __emutls_get_address that's used by all shared objects in a process, otherwise different DSOs will see different emulated TLS state for the same variable. One way to ensure that would be to put the emulated TLS bits in their own DSO and have everyone else reference that, but a full DSO is pretty heavy-weight. Another would be to make it default visibility and weak and let the dynamic linker coalesce multiple copies at runtime; -Bsymbolic and -Bsymbolic-functions would break that, but -Bsymbolic-non-weak-functions wouldn't.

I was waiting for users:) Glad that this can find a user.

Jul 30 2021, 3:27 PM · Restricted Project
danalbert accepted D107127: [builtins] Try to ensure single copy of emulated TLS state.

LGTM, but @rprichard knows this area better than I do so he should probably weigh in before submitting :)

Jul 30 2021, 1:42 PM · Restricted Project

May 19 2021

danalbert updated the diff for D77657: [WIP][libc++] Add an executor that bundles tests for deferred execution.

Fix remaining tests for the deferred test configuration.

May 19 2021, 10:47 AM · Restricted Project, Restricted Project

May 13 2021

danalbert updated the diff for D77657: [WIP][libc++] Add an executor that bundles tests for deferred execution.

Adds XFAIL-BUILD to deal with the deferred-run cases that fail to build.

May 13 2021, 3:50 PM · Restricted Project, Restricted Project

May 12 2021

danalbert updated the diff for D77657: [WIP][libc++] Add an executor that bundles tests for deferred execution.

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 12 2021, 5:28 PM · Restricted Project, Restricted Project
danalbert commandeered D77657: [WIP][libc++] Add an executor that bundles tests for deferred execution.
May 12 2021, 5:25 PM · Restricted Project, Restricted Project

May 4 2021

danalbert added a comment to D77657: [WIP][libc++] Add an executor that bundles tests for deferred execution.

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.

May 4 2021, 3:42 PM · Restricted Project, Restricted Project

Mar 5 2021

danalbert added inline comments to D97993: [Driver] Suppress GCC detection under -B.
Mar 5 2021, 2:55 PM · Restricted Project, Restricted Project

Feb 19 2021

danalbert added a comment to D91841: [builtins] Define fmax and scalbn inline.

Implementing the relevant functions called by divxc3 is one choice, another choice is to emulate libgcc - don't use scalbn.

The libgcc version goes about the computation in a slightly different way -- e.g. it computes an intermediate ratio of c / d or d / c (ensuring that the fabs(ratio) is at most 1.0) to avoid overflow, whereas the current compiler-rt version uses scalbn (and closely matches the reference implementation in Annex G of the C specification). The scalbn approach seems to be more accurate? e.g.:

  • From G.5.1 Multiplicative operators (paragraph 9):

Scaling the denominator alleviates the main overflow and underflow problem, which is more serious than for multiplication.
In the spirit of the multiplication example above, this code does not defend against overflow and underflow in the calculation
of the numerator. Scaling with the scalbn function, instead of with division, provides better roundoff characteristics.

/* ??? We can get better behavior from logarithmic scaling instead of
   the division.  But that would mean starting to link libgcc against
   libm.  We could implement something akin to ldexp/frexp as gcc builtins
   fairly easily...  */

I'm guessing we wouldn't want to reduce accuracy to break the libm dependency?

Feb 19 2021, 4:11 PM · Restricted Project
danalbert added a comment to D93003: [libunwind] unw_* alias fixes for ELF and Mach-O.

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 19 2021, 4:02 PM · Restricted Project, Restricted Project, Restricted Project

Feb 10 2021

danalbert accepted D96403: [Android] Use -l:libunwind.a with --rtlib=compiler-rt.
Feb 10 2021, 12:23 PM · Restricted Project
danalbert accepted D96404: [Android] Default to --rtlib=compiler-rt.
Feb 10 2021, 12:20 PM · Restricted Project

Feb 3 2021

danalbert added a comment to D95166: Disable rosegment for old Android versions..

We've since added -fuse-ld=lld to the three CMAKE_*_FLAGS so I think we're likely set on our end.

https://reviews.llvm.org/D76452 not being accepted means that Android toolchains must have LLD installed as ld.

If you pass -fuse-ld=lld everwhere, that's not needed, right (?)

Feb 3 2021, 6:12 PM · Restricted Project
danalbert added a comment to D95166: Disable rosegment for old Android versions..

We've since added -fuse-ld=lld to the three CMAKE_*_FLAGS so I think we're likely set on our end.

Feb 3 2021, 2:22 PM · Restricted Project

Jan 29 2021

danalbert added a comment to D95166: Disable rosegment for old Android versions..

Landed revert in 1608ba09462d877111230e9461b895f696f8fcb1. Someone should file a PR to make sure that gets merged to the 12.0 branch.

We can then reland on trunk once there are clear instructions for folks building with trunk clang but old NDK but without the toolchain file (either "use toolchain file" or "set LLVM_USE_LLD" or something like that, I'm guessing?)

Jan 29 2021, 12:24 PM · Restricted Project

Jan 28 2021

danalbert added a comment to D95166: Disable rosegment for old Android versions..

It does feel kind of awkward to me that _this_ is the patch that ends up breaking the builds, versus something at the cmake level that says "you are explicitly unsupported".

Jan 28 2021, 6:05 PM · Restricted Project
danalbert added a comment to D95166: Disable rosegment for old Android versions..

Our Android build bot is broken after this patch http://lab.llvm.org:8011/#/builders/77/builds/3234

Jan 28 2021, 2:47 PM · Restricted Project

Jan 27 2021

danalbert added a comment to D95166: Disable rosegment for old Android versions..

Firefox has a build break from this change. In certain Android configurations we use bfd or gold. The statement in the commit message "Android only supports LLD" is news to me, could you point me to any references for this?

Jan 27 2021, 2:38 PM · Restricted Project

Jan 26 2021

danalbert requested review of D95492: Pack relocations for Android when possible..
Jan 26 2021, 4:48 PM · Restricted Project
danalbert committed rGfae16fc0eed7: Disable rosegment for old Android versions. (authored by danalbert).
Disable rosegment for old Android versions.
Jan 26 2021, 4:16 PM
danalbert closed D95166: Disable rosegment for old Android versions..
Jan 26 2021, 4:16 PM · Restricted Project

Jan 21 2021

danalbert committed rG866d480fe054: [libc++abi] Add an option to avoid demangling in terminate. (authored by danalbert).
[libc++abi] Add an option to avoid demangling in terminate.
Jan 21 2021, 1:33 PM
danalbert closed D88189: [libc++abi] Add an option to avoid demangling in terminate..
Jan 21 2021, 1:33 PM · Restricted Project
danalbert added a comment to D88189: [libc++abi] Add an option to avoid demangling in terminate..

Thanks! Has been up for a while and I haven't heard any objections. Will submit and we can revert if issues are found.

Jan 21 2021, 1:31 PM · Restricted Project
danalbert requested review of D95166: Disable rosegment for old Android versions..
Jan 21 2021, 1:11 PM · Restricted Project

Dec 14 2020

danalbert closed D91664: Add a less ambiguous macro for Android version..

https://github.com/llvm/llvm-project/commit/0849047860a343d8bcf1f828a82d585e89079943

Dec 14 2020, 10:52 AM · Restricted Project
danalbert added a comment to D88189: [libc++abi] Add an option to avoid demangling in terminate..

Ping.

Dec 14 2020, 10:49 AM · Restricted Project

Dec 2 2020

danalbert committed rG0849047860a3: Add a less ambiguous macro for Android version. (authored by danalbert).
Add a less ambiguous macro for Android version.
Dec 2 2020, 1:27 PM

Nov 17 2020

danalbert requested review of D91664: Add a less ambiguous macro for Android version..
Nov 17 2020, 3:24 PM · Restricted Project

Nov 6 2020

danalbert added a comment to D88189: [libc++abi] Add an option to avoid demangling in terminate..

Ping

Nov 6 2020, 1:49 PM · Restricted Project

Oct 28 2020

danalbert added inline comments to D89924: [llvm-zorg] Use ndk21 for buildbot.
Oct 28 2020, 1:16 PM

Oct 27 2020

danalbert added inline comments to D89924: [llvm-zorg] Use ndk21 for buildbot.
Oct 27 2020, 12:28 PM

Oct 26 2020

danalbert requested changes to D87950: Make libc++ headers "just work" on all Android versions..

Actually, talk about timing! The Clang patch was just posted here: https://reviews.llvm.org/D90188

Oct 26 2020, 3:14 PM · Restricted Project
danalbert accepted D87950: Make libc++ headers "just work" on all Android versions..
Oct 26 2020, 2:26 PM · Restricted Project

Oct 12 2020

danalbert accepted D68269: [libc++] Define new/delete in libc++abi only by default.

LGTM for Android.

Oct 12 2020, 2:19 AM · Restricted Project, Restricted Project

Sep 23 2020

danalbert requested review of D88189: [libc++abi] Add an option to avoid demangling in terminate..
Sep 23 2020, 4:08 PM · Restricted Project

Aug 20 2020

danalbert accepted D86321: Fix test for D77924..
Aug 20 2020, 4:04 PM · Restricted Project

Jun 17 2020

danalbert accepted D81622: [Clang] Search computed sysroot for libc++ header paths.

While this doesn't look wrong to me -- and the correctness of this depends entirely on where vendors decide to put their headers so it's hard for me to verify -- I'm wondering why not all toolchains use this mechanism. We seem to be adding an abstraction that's used only by some toolchains, but not all. I think it would be great to have a single canonical way of representing the system root.

Jun 17 2020, 3:40 PM · Restricted Project

Jun 1 2020

danalbert accepted D80828: [Clang][A32/T32][Linux] -O1 implies -fomit-frame-pointer.
Jun 1 2020, 3:11 PM · Restricted Project, Restricted Project

May 21 2020

danalbert added a comment to D56456: [Driver] Default to -fno-addrsig on Android..

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.

May 21 2020, 12:59 PM

Apr 29 2020

danalbert committed rG2e090e4884d0: Reset another globalMemCounter. (authored by danalbert).
Reset another globalMemCounter.
Apr 29 2020, 5:51 PM
danalbert committed rG0a78e42b1f9d: Revert "[lld] Remove special cases from default ld driver mode." (authored by danalbert).
Revert "[lld] Remove special cases from default ld driver mode."
Apr 29 2020, 2:34 PM
danalbert added a reverting change for rGda093c388fc9: [lld] Remove special cases from default ld driver mode.: rG0a78e42b1f9d: Revert "[lld] Remove special cases from default ld driver mode.".
Apr 29 2020, 2:34 PM
danalbert committed rGda093c388fc9: [lld] Remove special cases from default ld driver mode. (authored by danalbert).
[lld] Remove special cases from default ld driver mode.
Apr 29 2020, 12:56 PM
danalbert closed D78837: [lld] Remove special cases from default ld driver mode..
Apr 29 2020, 12:55 PM · Restricted Project, lld

Apr 24 2020

danalbert added a comment to D76452: Use LLD by default for Android..

Yes, I was referring to that question too :) I'm working on the new lld-macho implementation, under the DarwinNew flavor. I'm not sure if anything depends on the old Darwin flavor, which is why we haven't removed it yet, though we plan to do that once we get the new implementation to a more mature stage.

Ah, gotcha :) Thanks! Will wait for @ruiu to chime in.

I vote for deleting the #ifdef __APPLE__ chunk so we don't have to add more code to either clang or lld....
The code owner of the existing lld darwin has explicitly expressed that we can drop the existing Darwin flavor at any time.

SGTM. Abandoning this. I'll send a patch to remove the LLD side shortly.

Apr 24 2020, 4:48 PM · Restricted Project
danalbert created D78837: [lld] Remove special cases from default ld driver mode..
Apr 24 2020, 4:48 PM · Restricted Project, lld
danalbert abandoned D76452: Use LLD by default for Android..

Yes, I was referring to that question too :) I'm working on the new lld-macho implementation, under the DarwinNew flavor. I'm not sure if anything depends on the old Darwin flavor, which is why we haven't removed it yet, though we plan to do that once we get the new implementation to a more mature stage.

Ah, gotcha :) Thanks! Will wait for @ruiu to chime in.

I vote for deleting the #ifdef __APPLE__ chunk so we don't have to add more code to either clang or lld....
The code owner of the existing lld darwin has explicitly expressed that we can drop the existing Darwin flavor at any time.

Apr 24 2020, 4:18 PM · Restricted Project
danalbert added a comment to D76452: Use LLD by default for Android..

Yes, I was referring to that question too :) I'm working on the new lld-macho implementation, under the DarwinNew flavor. I'm not sure if anything depends on the old Darwin flavor, which is why we haven't removed it yet, though we plan to do that once we get the new implementation to a more mature stage.

Apr 24 2020, 3:44 PM · Restricted Project
danalbert added a comment to D76452: Use LLD by default for Android..

I don't think I have enough context here to answer the question, but I'm pretty sure that change wouldn't affect what I'm working on

Apr 24 2020, 2:39 PM · Restricted Project