On 8/23/23 08:56, Robin Dapp wrote:
Does this patch fix these 2 following PR:
108271 – Missed RVV cost model (gnu.org)
<https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108271>
108412 – RISC-V: Negative optimization of GCSE && LOOP INVARIANTS (gnu.org)
<https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108412>
If yes, plz append these 2 cases into testsuite and indicate those 2 PR are
fixed.
So that we can close them.
The second one is fixed on my local branch, the first not yet because there
is more to it still. The second one is more due to pressure-aware scheduling
and I'm going to add it to the commit as well as the PR to the commit once this
is verified.
Basically there's a little conditional in IRA which basically says don't
rematerialize constants at their use points if register pressure
sensitive scheduling has been enabled. So turning on -fsched-pressure
helps, it's a bit of a cop-out, but probably good enough for now.
Jeff