On 9/10/2023 8:37 AM, Jeff Law via Gcc-patches wrote:


On 8/31/23 11:36, Edwin Lu wrote:
Related Discussion:
https://inbox.sourceware.org/gcc-patches/12fb5088-3f28-0a69-de1e-f387371a5...@gmail.com/

This patch updates the THEAD instructions to ensure that no insn is left
without a type attribute.

Tested for regressions using rv32/64 multilib for linux/newlib.

gcc/Changelog:

    * config/riscv/thead.md: Update types
OK.  THe first could arguably be "multi", but both instructions it generates appear to be move/conversions, so "fmove" is reasonable as well.

Ok for the trunk.  And I think that's should allow us to turn on the assertion, right?

jeff

I just did one final check, there's one insn in autovec-opt.md that was added (line 635) but otherwise I think that should be it. I've typed it "vector" for now and am currently running a test to make sure nothing breaks. Would you like me to send another patch for it?

Edwin

Reply via email to