about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAdam Perry <lol@anp.lol>2020-07-26 09:53:24 -0700
committerAdam Perry <lol@anp.lol>2020-07-26 10:12:12 -0700
commit4c710e72c42d988b09f7f3d877b1a280fdc46a49 (patch)
treec792600e3badce183df31e34f0acb4ccf534f223
parentd7f94516345a36ddfcd68cbdf1df835d356795c3 (diff)
downloadrust-4c710e72c42d988b09f7f3d877b1a280fdc46a49.tar.gz
rust-4c710e72c42d988b09f7f3d877b1a280fdc46a49.zip
Fix #[track_caller] shims for trait objects.
We were missing an Instance::resolve_for_fn_ptr in resolve_for_vtable.

Closes #74764.
-rw-r--r--src/librustc_middle/ty/instance.rs2
-rw-r--r--src/test/ui/rfc-2091-track-caller/tracked-trait-obj.rs23
2 files changed, 24 insertions, 1 deletions
diff --git a/src/librustc_middle/ty/instance.rs b/src/librustc_middle/ty/instance.rs
index f627d05d3e9..5c9c9ef6487 100644
--- a/src/librustc_middle/ty/instance.rs
+++ b/src/librustc_middle/ty/instance.rs
@@ -411,7 +411,7 @@ impl<'tcx> Instance<'tcx> {
             debug!(" => associated item with unsizeable self: Self");
             Some(Instance { def: InstanceDef::VtableShim(def_id), substs })
         } else {
-            Instance::resolve(tcx, param_env, def_id, substs).ok().flatten()
+            Instance::resolve_for_fn_ptr(tcx, param_env, def_id, substs)
         }
     }
 
diff --git a/src/test/ui/rfc-2091-track-caller/tracked-trait-obj.rs b/src/test/ui/rfc-2091-track-caller/tracked-trait-obj.rs
new file mode 100644
index 00000000000..3b2a2238fa8
--- /dev/null
+++ b/src/test/ui/rfc-2091-track-caller/tracked-trait-obj.rs
@@ -0,0 +1,23 @@
+// run-pass
+
+trait Tracked {
+    #[track_caller]
+    fn handle(&self) {
+        let location = std::panic::Location::caller();
+        assert_eq!(location.file(), file!());
+        // we only call this via trait object, so the def site should *always* be returned
+        assert_eq!(location.line(), line!() - 4);
+        assert_eq!(location.column(), 5);
+    }
+}
+
+impl Tracked for () {}
+impl Tracked for u8 {}
+
+fn main() {
+    let tracked: &dyn Tracked = &5u8;
+    tracked.handle();
+
+    const TRACKED: &dyn Tracked = &();
+    TRACKED.handle();
+}