[EXPERIMENTAL] What if we forced frame pointers?

Author: jieyouxu

src/bootstrap/src/core/build_steps/test.rs

@@ -3185,25 +3185,15 @@ impl Step for TestHelpers {
             }
             cfg.compiler(builder.cc(target));
         }
-
-        let build = cfg
-            .cargo_metadata(false)
+        cfg.cargo_metadata(false)
             .out_dir(&dst)
             .target(&target.triple)
             .host(&builder.config.build.triple)
             .opt_level(0)
             .warnings(false)
             .debug(false)
-            .file(builder.src.join("tests/auxiliary/rust_test_helpers.c"));
-
-        // Needed to have line number tables
-        if target.is_msvc() {
-            build.flag("/Zi");
-        } else {
-            build.flag("-g1");
-        }
-
-        build.compile("rust_test_helpers");
+            .file(builder.src.join("tests/auxiliary/rust_test_helpers.c"))
+            .compile("rust_test_helpers");
     }
 }
 

tests/ui/debuginfo/auxiliary/dylib-dep-helper-aux.rs

@@ -1,4 +1,4 @@
-//@ compile-flags: -g -Cstrip=none
+//@ compile-flags: -g -Cstrip=none -Cforce-frame-pointers=yes
 
 #[inline(never)]
 pub fn callback<F>(f: F)

tests/ui/debuginfo/auxiliary/dylib-dep-helper.rs

@@ -1,4 +1,4 @@
-//@ compile-flags: -g -Cstrip=none
+//@ compile-flags: -g -Cstrip=none -Cforce-frame-pointers=yes
 
 #![crate_type = "cdylib"]
 #![crate_type = "rlib"]

tests/ui/debuginfo/backtrace-dylib-dep.rs

@@ -8,7 +8,7 @@
 //@ ignore-fuchsia Backtraces not symbolized
 //@ ignore-musl musl doesn't support dynamic libraries (at least when the original test was written).
 //@ needs-unwind
-//@ compile-flags: -g -Copt-level=0 -Cstrip=none
+//@ compile-flags: -g -Copt-level=0 -Cstrip=none -Cforce-frame-pointers=yes
 //@ aux-crate: dylib_dep_helper=dylib-dep-helper.rs
 //@ aux-crate: auxiliary=dylib-dep-helper-aux.rs
 //@ run-pass