[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