[PATCH] D123112: [VP] Legalize the stride operand for EXPERIMENTAL_VP_STRIDED SDNodes
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 9 08:21:50 PDT 2022
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
In D123112#3570064 <https://reviews.llvm.org/D123112#3570064>, @loralb wrote:
> In D123112#3537527 <https://reviews.llvm.org/D123112#3537527>, @craig.topper wrote:
>
>> Is it possible to test this?
>
> I am testing this in D121113 <https://reviews.llvm.org/D121113>, which indeed depends on this patch.
> In order to move the tests here, I would have to duplicate the ones in D121113 <https://reviews.llvm.org/D121113> both for `riscv32` and `riscv64` and add that revision as parent of this one.
> Which approach would you suggest?
Ok, we can leave them in D121113 <https://reviews.llvm.org/D121113>.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123112/new/
https://reviews.llvm.org/D123112
More information about the llvm-commits
mailing list