]> git.ipfire.org Git - thirdparty/gcc.git/commit
[RISC-V][V2] Fix type on vector move patterns
authorJeff Law <jlaw@ventanamicro.com>
Thu, 14 Nov 2024 23:57:50 +0000 (16:57 -0700)
committerJeff Law <jlaw@ventanamicro.com>
Thu, 14 Nov 2024 23:58:53 +0000 (16:58 -0700)
commitaaada43a2a91194a3c50e364d9dde277fb4b764b
tree1234f740dd5910fcc6605d061ca3c8b1c696b9ea
parentf70c1d517e09c4dde421774a8cec591ca3c479a0
[RISC-V][V2] Fix type on vector move patterns

Updated version of my prior patch to fix type attributes on the
pre-allocation vector move pattern.  This version just adds a suitable
set of attributes to a second pattern that was obviously wrong.

Passed on my tester for rv64 and rv32 crosses.  Bootstrapped and
regression tested on riscv64-linux-gnu as well.

--

So I was looking into a horrific schedule for SAD a week or so ago and
came across this gem.

Basically we were treating a vector load as a vector move from a
scheduling standpoint during sched1.  Naturally we didn't expose much
ILP during sched1.  That in turn caused the register allocator to pack
the pseudos onto the physical vector registers tightly.  regrename
didn't do anything useful and the resulting code had too many false
dependencies for sched2 to do anything useful.

As a result we were taking many load->use stalls in x264's SAD routine.

I'm confident the types are fine, but I'm a lot less sure about the
other attributes (mode, avl_type_index, mode_idx).  If someone could
take a look at that, it'd be greatly appreciated.

There's other cases that may need similar treatment.  But I didn't want
to muck with them until I understood those other attributes and how they
need adjustments.

In particular mov<VLS_AVL_REG:mode><P:mode>_lra appears to have the same
problem.

--

gcc/
* config/riscv/vector.md (mov<mode> pattern/splitter): Fix type and
other attributes.
(mov<VLS_AVL_REG:mode><P:mode>_lra): Likewise.
gcc/config/riscv/vector.md