New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 799628 link

Starred by 2 users
Status: Archived
Owner:
█ euge...@chromium.org
Last visit > 30 days ago
Closed: Jan 17
Cc:
█ euge...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Android ASan bot red since Jan 3 with a dl_iterate_phdr link error

Project Member Reported by r...@chromium.org, Jan 5 2018 
First failing build:
https://ci.chromium.org/buildbot/chromium.clang/ToTAndroidASan/360

Link error:

[5024/58792] LINK ./generate_test_dump
FAILED: generate_test_dump exe.unstripped/generate_test_dump 
...
/b/c/b/ToTAndroidASan/src/third_party/llvm-build/Release+Asserts/lib/clang/7.0.0/lib/linux/libclang_rt.asan-arm-android.so: error: undefined reference to 'dl_iterate_phdr'

I don't see any immediately obvious problematic ASan changes, so I'm not sure what's up. We started getting this error immediately after the mmsghdr build errors were resolved.

Comment 1 by euge...@chromium.org, Jan 8 2018

Owner: euge...@chromium.org
Probably caused by the switch to NDK r16, which expects __ANDROID_API__ to be defined by the compiler, and provides a wrapper script in the standalone toolchain to add the definition if its missing. This breaks when we use self-built clang with the standalone toolchain.

Also see b/68212952.

The fix is to use explicit api level in the target triple, as in --target=arm-linux-android19. I'll prepare a patch.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 17 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/230fa45ee2d6573b32ff4b0790620faa1e7bce60

commit 230fa45ee2d6573b32ff4b0790620faa1e7bce60
Author: Evgenii Stepanov <eugenis@google.com>
Date: Wed Jan 17 04:50:37 2018

Use explicit android api level in target triple in compiler-rt build.

Without this clang fails to define __ANDROID_API__ macro which NDK headers depend on.

R=pcc@chromium.org, rnk@chromium.org

Bug:  799628 
Change-Id: I88f4922115652a484afc1457a85c2d3dd9f62ba8
Reviewed-on: https://chromium-review.googlesource.com/854686
Commit-Queue: Evgeniy Stepanov <eugenis@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529614}
[modify] https://crrev.com/230fa45ee2d6573b32ff4b0790620faa1e7bce60/tools/clang/scripts/update.py
Project Member

Comment 3 by sheriffbot@chromium.org, Jan 17 (5 days ago)

Status: Archived (was: Unconfirmed)
Issue has not been modified or commented on in the last 365 days, please re-open or file a new bug if this is still an issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
►

Sign in to add a comment