Create Atomic<T> type alias

[CAD97]

and use it in core/alloc/std where possible, ignoring test files for now.

This is step one, creating the alias from Atomic<T> to AtomicT. The next step of flipping this and aliasing AtomicT to Atomic<T> will have a bigger impact, since AtomicT imports can be dropped once Atomic::new is a usable name.

First commit is the true change. Second commit is mostly mechanical replacement of AtomicT type names with Atomic<T>.

See how this was done for NonZero for the rough blueprint I'm following.

• ACP: ACP: Generic Atomic<T> libs-team#443 (comment)
• Tracking issue: Tracking Issue for generic Atomic #130539

[rustbot]

r? @Noratrieb

rustbot has assigned @Noratrieb.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[CAD97]

Alright that's the merge conflicts fixed. If I end up needing to continue playing catch-up on some of the touched files, I'll probably just drop files' touch-up to use Atomic<T> from this PR instead of continuously rebasing, as they create conflicts.

[Noratrieb]

r=me after fixing the import issues

[CAD97]

Also found a similar conflict with the windows futex impl plumbing via local check, so that's also resolved by the amend (hopefully; my local checkout suddenly is complaining about duplicate rmeta for crate object for some reason).

IIRC I don't have perms to do this but might as well try:

@bors r=Noratrieb rollup=iffy

[bors]

@CAD97: 🔑 Insufficient privileges: Not in reviewers

[bors]

@CAD97: 🔑 Insufficient privileges: not in try users

[workingjubilee]

@bors r=Noratrieb rollup=iffy

[bors]

📌 Commit fbf092e has been approved by Noratrieb

It is now in the queue for this repository.

[CAD97]

That should be fixed now

@rustbot review

[Noratrieb]

@bors r+

[bors]

📌 Commit 8ae16dd has been approved by Noratrieb

It is now in the queue for this repository.

[bors]

⌛ Testing commit 8ae16dd with merge 7c80d00...

[bors]

☔ The latest upstream changes (presumably #136572) made this pull request unmergeable. Please resolve the merge conflicts.

[rust-log-analyzer]

The job x86_64-fuchsia failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[RUSTC-TIMING] object test:false 7.804
error[E0107]: missing generics for type alias `core::sync::atomic::Atomic`
   --> library/std/src/sys/pal/unix/futex.rs:259:48
    |
259 |     pub type zx_futex_t = crate::sync::atomic::Atomic;
    |                                                ^^^^^^ expected 1 generic argument
note: type alias defined here, with 1 generic parameter: `T`
   --> /checkout/library/core/src/sync/atomic.rs:341:10
    |
    |
341 | pub type Atomic<T> = <T as AtomicPrimitive>::AtomicInner;
help: add missing generic argument
    |
    |
259 |     pub type zx_futex_t = crate::sync::atomic::Atomic<T>;

For more information about this error, try `rustc --explain E0107`.
[RUSTC-TIMING] std test:false 4.941
error: could not compile `std` (lib) due to 1 previous error

[bors]

💔 Test failed - checks-actions

[CAD97]

🤦

Will fix that when I get a chance.

@rustbot author

[GrigorenkoPV]

This got merged in #136316, so it can now be closed, I think