[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 Apr 26 01:53:59 PDT 2023


matejam added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:237
 
+static LegalityPredicate isNotAnExistingRegClassType(unsigned TypeIdx) {
+  return [=](const LegalityQuery &Query) {
----------------
foad wrote:
> Do we really need this new concept of "RegClassType" in addition to "RegisterType"? I'd really prefer if we can make them the same thing.
I changed it so that the condition is that the type has to be a RegisterType and that it doesn't have a corresponding RegClass.
I can't completely abandon the concept of RegClass, because depending on the existing RegClasses, we widen the vector operand in G_EXTRACT/INSERT_VECTOR_ELT.


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

https://reviews.llvm.org/D148096



More information about the llvm-commits mailing list