Skip to content

Commit

Permalink
Revert "Remove strip.keep_symbols from libart defaults"
Browse files Browse the repository at this point in the history
This reverts commit c7f5eb7.

Fixes: crdroidandroid/issue_tracker#364
Signed-off-by: Pranav Vashi <neobuddy89@gmail.com>
  • Loading branch information
muhomorr authored and neobuddy89 committed Apr 11, 2024
1 parent 1bbb472 commit f6d02cf
Showing 1 changed file with 31 additions and 0 deletions.
31 changes: 31 additions & 0 deletions runtime/Android.bp
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,37 @@ cc_defaults {
"-Wno-unused-command-line-argument",
],
},
android_arm: {
// Arm 32 bit does not produce complete exidx unwind information
// so keep the .debug_frame which is relatively small and does
// include needed unwind information.
// See b/132992102 and b/145790995 for details.
strip: {
keep_symbols_and_debug_frame: true,
},
},
// For all other architectures, leave the symbols in the shared library
// so that stack unwinders can produce meaningful name resolution.
android_arm64: {
strip: {
keep_symbols: true,
},
},
android_riscv64: {
strip: {
keep_symbols: true,
},
},
android_x86: {
strip: {
keep_symbols: true,
},
},
android_x86_64: {
strip: {
keep_symbols: true,
},
},
},
}

Expand Down

0 comments on commit f6d02cf

Please sign in to comment.