[PATCH] D65930: [IntrinsicEmitter] Support scalable vectors in intrinsics

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 8 02:49:56 PDT 2019


sdesmalen added a comment.

Thanks for this patch @c-rhodes!

I see the other patch adds the intrinsics that use (and test) this functionality, but I think it would help to also have some tests in this patch. Specifically to assert that passing a fixed-width vector where a scalable vector is expected (and vice-versa), fails.
You can probably tests these by just running `opt -verify`.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65930





More information about the llvm-commits mailing list