]> git.ipfire.org Git - thirdparty/gcc.git/commit
Revert part of r16-869-ge3d3d6d7d2c8ab
authorRichard Biener <rguenther@suse.de>
Fri, 6 Feb 2026 13:25:06 +0000 (14:25 +0100)
committerRichard Biener <rguenther@suse.de>
Sat, 7 Feb 2026 10:06:36 +0000 (11:06 +0100)
commit5f5ca32dd175ccd2946cedef23769482c309945e
tree4b71718b92a469fc8b25928ab4030694349dceb6
parent16cbe70b95227a119fbd146f2947f5e162bbdbd4
Revert part of r16-869-ge3d3d6d7d2c8ab

The following reverts part of the changes done by r16-869-ge3d3d6d7d2c8ab
which AFAICS was never posted on the mailing list.  The hunk reverted
adds extra costs for vector construction of AVX or AVX512 vectors
because Honza thought we're only adding 1 conversion for all constructs,
but in fact we're costing each individual unique scalar source used.

Noticed when investigating PR120234, but this does not affect it
since there it's all SSE sized operations.

Still this is a regression, though reading r16-869-ge3d3d6d7d2c8ab
commit message indicating "correct numbers regress benchmarks" might
make this somewhat dangerous.  Still r16-531-g37e61c793c1b22 already
made the constructor costs much more expensive for integers, so
this might compensate.

* config/i386/i386.cc (ix86_vector_costs::add_stmt_cost):
Remove double and triple accounting of GPR -> XMM moves
in construction of AVX and AVX512 vectors.
gcc/config/i386/i386.cc