Add missing inline attributes to Duration trait impls

[elichai]

Currently Duration::checked_add is marked #[inline] but it's trait relative Add::add is not.
Leading to a case where:

pub fn foo() -> Duration {
    Duration::from_secs(10) + Duration::from_millis(6)
}

pub fn bar() -> Duration {
    Duration::from_secs(10).checked_add(Duration::from_millis(6)).expect("overflow when adding durations")
}

compiles to:

playground::foo:
	movl	$10, %edi
	xorl	%esi, %esi
	xorl	%edx, %edx
	movl	$6000000, %ecx
	jmpq	*<core::time::Duration as core::ops::arith::Add>::add@GOTPCREL(%rip)

playground::bar:
	movl	$10, %eax
	movl	$6000000, %edx
	retq

(The same happens for all arithmetic operation)

[rustbot]

r? @thomcc

(rustbot has picked a reviewer for you, use r? to override)

[thomcc]

Seems fine. The divs might be a bit much to inline, but it's not like these are inline(always).

@bors r+

[bors]

📌 Commit ebc2923 has been approved by thomcc

It is now in the queue for this repository.

[elichai]

Ops, did this with Github's editor and I misplaced the attribute inside the function instead of ontop of it, force pushed a fix :)

[thomcc]

@bors r+

[bors]

📌 Commit 92c9bcd has been approved by thomcc

It is now in the queue for this repository.

[bors]

⌛ Testing commit 92c9bcd with merge 4910642...

[bors]

☀️ Test successful - checks-actions
Approved by: thomcc
Pushing 4910642 to master...

[rust-timer]

Finished benchmarking commit (4910642): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	6.7%	[4.1%, 9.4%]	2
Regressions ❌ (secondary)	3.2%	[3.2%, 3.2%]	1
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-2.6%	[-2.6%, -2.6%]	1
All ❌✅ (primary)	6.7%	[4.1%, 9.4%]	2

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-3.8%	[-5.5%, -2.4%]	6
All ❌✅ (primary)	-	-	0

Binary size

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	0.1%	[0.0%, 0.1%]	10
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-0.2%	[-0.4%, -0.2%]	5
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-0.0%	[-0.4%, 0.1%]	15

Bootstrap: 622.655s -> 622.307s (-0.06%)
Artifact size: 272.02 MiB -> 272.04 MiB (0.01%)