Bump compiler-builtins to 0.1.69

[Amanieu]

This includes rust-lang/compiler-builtins#452 which should fix some issues with duplicate symbol defintions of some intrinsics.

[rust-highfive]

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

[Amanieu]

@bors rollup=iffy

[Mark-Simulacrum]

Can you update this comment - https://github.com/rust-lang/rust/blob/master/Cargo.toml#L68

I'm worried that it seems like the comment suggests separate modules shouldn't be necessary, but maybe I am missing something.

[Amanieu]

This comment still applies, it's just that codegen unit splitting only works at module boundaries so each function needs to be wrapped in its own mod. This was already the case for most intrinsics but there were a few that weren't wrapped which were causing errors.

[Mark-Simulacrum]

That makes sense to me. It may be worth adding a flag to rustc which partitions without the module boundary grouping, FWIW, but seems like this solution works OK for now.

@bors r+ rollup=never

[bors]

📌 Commit 59b2ee564093747c0b4de8036942fa8145750504 has been approved by Mark-Simulacrum

[bors]

⌛ Testing commit 59b2ee564093747c0b4de8036942fa8145750504 with merge 18f60772bab1f08de99ceb7409fc0f20c29a5d50...

[bors]

💔 Test failed - checks-actions

[Amanieu]

There was a typo in the inline assembly which caused the wrong function to be called. This is fixed in compiler-builtins 0.1.69.

[Mark-Simulacrum]

@bors r+ rollup=never

[bors]

📌 Commit 7b8f6ac has been approved by Mark-Simulacrum

[bors]

⌛ Testing commit 7b8f6ac with merge 01c4c41...

[bors]

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing 01c4c41 to master...

[rust-timer]

Finished benchmarking commit (01c4c41): comparison url.

Summary: This benchmark run did not return any relevant results.

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

@rustbot label: -perf-regression