[PATCH] D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors

Justin Bogner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 26 13:18:37 PDT 2019


bogner requested changes to this revision.
bogner added a comment.
This revision now requires changes to proceed.

I hit a couple of minor issues trying to apply the patch:

1. We hit an assertion in test/CodeGen/AMDGPU/kernel-args.ll. Can you take a look?
2. I don't have test/CodeGen/X86/load-local-v3i1.ll in my tree, so that part doesn't apply. Is it reasonable to drop this part of the change?
3. A couple of the X86 tests have changed slightly - these are easy enough to upgrade with update_llc_test_checks though.


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

https://reviews.llvm.org/D56201





More information about the llvm-commits mailing list