Hi Paul-Antoine,

overall the patch looks reasonable to me now, provided the fr2vr followup.

BTW it's the late-combine pass that performs the optimization, not the combine pass. You might still want to fix this in the commit message.

Please CC patchworks...@rivosinc.com for the next version so we have CI
coverage.

Hmm, that somehow didn't work. ISTM that it doesn't pick up "new" patches in a thread. Could you re-send in a new thread please? It should pick it up even without CC then. If the CI is green this is good to go.

Checkstyle has a few minor complaints still, like tabs vs spaces in the md changes, but we haven't been very strict there so I won't insist ;)

Starting with vfmn{add,sub}, my plan is to first check whether actual gains can be achieved and if so I will be coming up with a patch similar to this one. How does that sound?

The other patterns are definitely necessary, regardless of whether they show up in SPEC or not.

--
Regards
Robin

Reply via email to