[PATCH] D103880: [SDAG] Fix pow2 assumption when splitting vectors

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 05:27:34 PDT 2021


RKSimon added a comment.

In D103880#2809550 <https://reviews.llvm.org/D103880#2809550>, @critson wrote:

> In D103880#2807899 <https://reviews.llvm.org/D103880#2807899>, @RKSimon wrote:
>
>> test case?
>
> I am not sure if I can trigger this without v6 types, as the current non-pow2 types v3 and v5 are not divisible by 2.
>
> Without this change simply adding v6i32 types will cause an assertion failure while compiling: llvm/test/CodeGen/X86/pr44976.ll

OK - if this per-emptively fixes an existing test regression then that's fine.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103880/new/

https://reviews.llvm.org/D103880



More information about the llvm-commits mailing list