]> git.lizzy.rs Git - rust.git/commit - src/tools/rust-analyzer
Rollup merge of #83391 - hyd-dev:uwtable, r=alexcrichton
authorDylan DPC <dylan.dpc@gmail.com>
Wed, 24 Mar 2021 00:52:30 +0000 (01:52 +0100)
committerGitHub <noreply@github.com>
Wed, 24 Mar 2021 00:52:30 +0000 (01:52 +0100)
commit30db261023d6b399ce3b16ce6345b5245a8800d2
tree9385e9436d6439463e8a76f8e3b4d97bb3017f86
parenta42e62fa0a59d0ba620889f97513929a113a6fbd
parentf900ee331dfe95493390e1beecb82a277158b60b
Rollup merge of #83391 - hyd-dev:uwtable, r=alexcrichton

Allow not emitting `uwtable` on Android

`uwtable` is marked as required on Android, so it can't be disabled via `-C force-unwind-tables=no`. However, I found that the reason it's marked as required was to resolve a [backtrace issue in Gecko](https://github.com/rust-lang/rust/issues/49867), and I haven't find any other reasons that make it required ([yet](https://rust-lang.zulipchat.com/#narrow/stream/122651-general/topic/Unwind.20tables.20are.20strictly.20required.20on.20Windows.20and.20Android)). Therefore, I assume it's safe to turn it off if a (nice) backtrace is not needed, and submit this PR to allow `-C force-unwind-tables=no` when targeting Android.

Note that I haven't tested this change on Android as I don't have an Android environment for testing.