[PATCH] D148096: [AMDGPU][GlobalISel] Widen the vector operand in G_BUILD/INSERT/EXTRACT_VECTOR

Mateja Marjanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 06:04:55 PDT 2023


matejam added a comment.

In D148096#4299323 <https://reviews.llvm.org/D148096#4299323>, @arsenm wrote:

> Needs new tests that now legalize? The test updates only show regressions?

Well, the point of this patch is to fix the errors that occur in insertelements.ll and extractelements.ll
when the conditions for get*ClassForBitWidth are changed from <= to ==.

Test cases that were changed from GFX10PLUS to GFX10 and GFX11 differentiate from
one another by just the register name (one uses s0 and the other one s4) .

Other test cases that were change include only the ones in which the vector type didn't have
a corresponding Register Class and had to be widened.


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

https://reviews.llvm.org/D148096



More information about the llvm-commits mailing list