> Yes, that's the default "len" codegen, two (or more) minimums. So if we have > a "min" select_vl doesn't make the code much better. It's just that we need > vsetvl anyway for vtype setting.
After some thought, I think I'll be dropping this patch as well as the vec_set one. The vec_set optimization has the unintended consequence of causing more spilling under certain conditions and might be more of a de-optimization. The VLS select_vl doesn't seem to be as universally useful as I hoped after checking several examples. We still get most of the benefit from the patch that adds VLS modes to loads and stores, just a more regular loop control. Going to post a smaller v3 series later today after testing. -- Regards Robin
