From 9f36ec0d4272006eb02f156280626efef7292dde Mon Sep 17 00:00:00 2001 From: Tamir Duberstein Date: Wed, 29 Apr 2015 11:12:25 -0700 Subject: [PATCH] Use PIE on Android This is OK to do given: - PIE is supported on Android starting with API 16. - The bots are running API 18. - API < 16 now has a 12.5% market share[0] as of 2015-04-29. Unfortunately, this breaks backtrace support. See #17520. Closes #17437. [0] https://developer.android.com/about/dashboards/index.html --- src/librustc_back/target/arm_linux_androideabi.rs | 2 -- src/test/run-pass/backtrace.rs | 1 + 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/librustc_back/target/arm_linux_androideabi.rs b/src/librustc_back/target/arm_linux_androideabi.rs index b3baa78f4939..cbaa2b205b05 100644 --- a/src/librustc_back/target/arm_linux_androideabi.rs +++ b/src/librustc_back/target/arm_linux_androideabi.rs @@ -13,8 +13,6 @@ use target::Target; pub fn target() -> Target { let mut base = super::android_base::opts(); base.features = "+v7".to_string(); - // FIXME #17437 (and #17448): Android doesn't support position dependent executables anymore. - base.position_independent_executables = false; Target { data_layout: "e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-\ diff --git a/src/test/run-pass/backtrace.rs b/src/test/run-pass/backtrace.rs index f4b62eb2e7c5..3f4849dbcb2b 100644 --- a/src/test/run-pass/backtrace.rs +++ b/src/test/run-pass/backtrace.rs @@ -10,6 +10,7 @@ // no-pretty-expanded FIXME #15189 // ignore-windows FIXME #13259 +// ignore-android FIXME #17520 use std::env; use std::process::{Command, Stdio};