proc_macro/bridge: Add #[inline] to RunningSameThreadGuard methods

mystor · mystor · commit 49138846590f · 2022-09-05T14:36:41.000-04:00
This is a potential fix to the perf regression from #101414. r? @eddyb
diff --git a/library/proc_macro/src/bridge/server.rs b/library/proc_macro/src/bridge/server.rs
@@ -160,6 +160,7 @@ thread_local! {
 struct RunningSameThreadGuard(());
 
 impl RunningSameThreadGuard {
+    #[inline]
     fn new() -> Self {
         let already_running = ALREADY_RUNNING_SAME_THREAD.replace(true);
         assert!(
@@ -171,6 +172,7 @@ impl RunningSameThreadGuard {
 }
 
 impl Drop for RunningSameThreadGuard {
+    #[inline]
     fn drop(&mut self) {
         ALREADY_RUNNING_SAME_THREAD.set(false);
     }

Original file line number	Diff line number	Diff line change
`@@ -160,6 +160,7 @@ thread_local! {`
`160`	`160`	`struct RunningSameThreadGuard(());`
`161`	`161`
`162`	`162`	`impl RunningSameThreadGuard {`
	`163`	`+ #[inline]`
`163`	`164`	`fn new() -> Self {`
`164`	`165`	`let already_running = ALREADY_RUNNING_SAME_THREAD.replace(true);`
`165`	`166`	`assert!(`
`@@ -171,6 +172,7 @@ impl RunningSameThreadGuard {`
`171`	`172`	`}`
`172`	`173`
`173`	`174`	`impl Drop for RunningSameThreadGuard {`
	`175`	`+ #[inline]`
`174`	`176`	`fn drop(&mut self) {`
`175`	`177`	`ALREADY_RUNNING_SAME_THREAD.set(false);`
`176`	`178`	`}`