diff options
author | Mark Mentovai <mark@chromium.org> | 2017-09-16 00:35:46 -0400 |
---|---|---|
committer | Mark Mentovai <mark@chromium.org> | 2017-09-18 14:00:44 +0000 |
commit | afa9c52715db1e4bfaa4b01c9aec40cc249b689b (patch) | |
tree | bcd10e1138d5e5cce4a5099c515122b6b1b11f47 /src/third_party/libdisasm | |
parent | Add new test data to Makefile.am (diff) | |
download | breakpad-afa9c52715db1e4bfaa4b01c9aec40cc249b689b.tar.xz |
android: Don’t compete with NDK API >= 21 over NDK structures
Chrome uses API 16 for 32-bit builds and API 21 for 64-bit builds. The
NDK’s <link.h> provides r_debug and link_map structure definitions only
at API 21 and above. Breakpad used a custom <link.h> to define these
structures only during 64-bit builds, which worked for Chrome’s
purposes. However, other consumers may wish to build Breakpad at
arbitrary API levels without regard to bitness. This alters Breakpad’s
custom <link.h> to correctly check the NDK API level rather than target
CPU bitness.
Likewise for <sys/user.h> on 32-bit x86, which provided a typedef for
user_fpxregs_struct to user_fxsr_struct. API 21 and above, as well as
the unified headers at any API level, always name the structure
user_fpxregs_struct.
Definitions for 64-bit ARM’s user_regs_struct and user_fpsimd_struct
have been removed from Breakpad’s copy of <sys/user.h>. The header
claims that these fallback definitions are only necessary with NDK r10,
which should no longer be in use even by Chromium, which now uses NDK
r12b. This removes the Chromium-specific ANDROID_NDK_MAJOR_VERSION macro
from use entirely.
Fixes https://stackoverflow.com/questions/44141159/ and b/65630828.
Bug: google-breakpad:733
Change-Id: I5841906297cd15b15ce48b73fd8332fd40afc9a0
Reviewed-on: https://chromium-review.googlesource.com/665740
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Reviewed-by: Joshua Peraza <jperaza@chromium.org>
Diffstat (limited to 'src/third_party/libdisasm')
0 files changed, 0 insertions, 0 deletions