> Thanks, the expr.cc change looks good to me. I'd like sign-off from a > RISC-V maintainer on the testsuite changes before pushing to trunk though.
The test change is OK but I'd prefer an extra test case. As the PR is not really related and the patch impacts vector code gen a separate test seems warranted. In particular we'd want to check that we do not create a vector store but a scalar store rather than just checking the number of vsetvls which might be affected by other things as well. -- Regards Robin