[PATCH] D118058: [SelectionDAG][VP] Provide expansion for VP_MERGE
Fraser Cormack via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 24 10:27:57 PST 2022
frasercrmck added a comment.
I haven't done much work in expansions so don't know exactly how/when best to check to unroll. For example, I didn't want to check whether `VSELECT` was marked as `Expand` because we rely on that expansion. But if the VSELECT were to unroll then arguably VP_MERGE should unroll.. So should the expansion of VP_MERGE duplicate logic that should be internal to ExpandVSELECT?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D118058/new/
https://reviews.llvm.org/D118058
More information about the llvm-commits
mailing list