[llvm] [LV] Add initial support for vectorizing literal struct return values (PR #109833)

Benjamin Maxwell via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 02:27:23 PST 2024


MacDue wrote:

> This seems reasonable in general, although would be good if possible to avoid too many changes, possibly by keeping `VPRecipeIRFlags` as single def for now, moving the IR flags to a separate class with `VPRecipeIRFlags` inheriting from both `VPSingleDefRecipe` & the class to manage the flags?

That is pretty much what I did in #112402, but I called that class `VPSingleDefRecipeWithIRFlags`. If this is the approach we want to use, I think I can split that out into its own patch (which is pretty much an NFC)

> Would it be possible to get an initial version working without support for replicate recipes?

Maybe, I think the minimum support I need is this working for both `VPWidenCallRecipe` and `VPWidenIntrinsicRecipe`.
Though `VPWidenIntrinsicRecipe` may be able to wait for a follow-up patch (as it'll need additional cost model changes).





https://github.com/llvm/llvm-project/pull/109833


More information about the llvm-commits mailing list