[PATCH] D134586: [VP][RISCV] Add vp.ceil and RISC-V support
Fraser Cormack via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 27 08:20:15 PDT 2022
frasercrmck added a comment.
In D134586#3818275 <https://reviews.llvm.org/D134586#3818275>, @craig.topper wrote:
> In D134586#3817430 <https://reviews.llvm.org/D134586#3817430>, @frasercrmck wrote:
>
>> Forgive me if I've missed something but you're adding support for widening and splitting in the legalizer without adding tests for that behaviour.
>
> There's a <vscale x 7 x double> <vscale x 16 x double>, <15 x double> and <32 x double> test. Does that cover it?
Yes, thanks. To my shame I didn't read through the whole tests before making that comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134586/new/
https://reviews.llvm.org/D134586
More information about the llvm-commits
mailing list