r? @compiler-errors

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

indeed, what are we actually trying to achieve here?

This mirrors the scalar bswap intrinsic:

It would be a bit of a pain to special-case i8/u8 vectors in std::simd.

or more precisely, what is the current behavior we're trying to avoid?

or more precisely, what is the current behavior we're trying to avoid?

I thought LLVM would handle a no-op, but instead the compiler crashes:

warning: build failed, waiting for other jobs to finish...
bswap must be an even number of bytes
  %4 = call <32 x i8> @llvm.bswap.v32i8(<32 x i8> %3), !dbg !858
in function _ZN113_$LT$core_simd..core_simd..vector..Simd$LT$i8$C$_$GT$$u20$as$u20$core_simd..core_simd..elements..int..SimdInt$GT$10swap_bytes17h00fedc46e657fc9cE
LLVM ERROR: Broken function found, compilation aborted!
error: could not compile `core_simd` (test "i8_ops")

lmao llvm

Both scalar integers and std::simd integer vectors are macro-generated No users should be seeing these intrinsics.

To be fair to LLVM, it says very clearly that the intrinsic is only defined when BitWidth % 16 == 0.

I guess r=me if @workingjubilee thinks that this is worthwhile

yeaaaaah, I guess.

but I could believe it makes sense with like... macro-generated code or something

that describes all of std's integer impls, baby!

@bors r=compiler-errors

Commit 77ed437 has been approved by compiler-errors

It is now in the queue for this repository.

@bors r- rollup=iffy
cc #114270

I have no idea but I'm almost certain it's this PR.

The failed test is tests/ui/lint/must_not_suspend/handled.rs, I don't see any way this PR could affect that. Maybe spurious?

...hm yeah you're right, I misread the logs at-a-glance.

Probably is spurious then.

@bors r=compiler-errors

Commit 77ed437 has been approved by compiler-errors

It is now in the queue for this repository.

Testing commit 77ed437 with merge 3be07c1...

Test successful - checks-actions
Approved by: compiler-errors
Pushing 3be07c1 to master...

Finished benchmarking commit (3be07c1): comparison URL.

Overall result: regressions - ACTION NEEDED

Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please open an issue or create a new PR that fixes the regressions, add a comment linking to the newly created issue or PR, and then add the perf-regression-triaged label to this PR.

@rustbot label: +perf-regression
cc @rust-lang/wg-compiler-performance

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions (secondary)	5.8%	[5.8%, 5.8%]	1
Improvements ✅ (primary)	-	-	0
Improvements (secondary)	-	-	0
All ❌✅ (primary)	-	-	0

Max RSS (memory usage)

Results

Cycles

Results

Binary size

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

Bootstrap: 652.418s -> 653.862s (0.22%)

One possible, marginal regression, nothing to worry about.

@rustbot label: +perf-regression-triaged