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

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 07:29:57 PDT 2019


c-rhodes updated this revision to Diff 216610.
c-rhodes edited the summary of this revision.
c-rhodes added a comment.

- Used existing `masked.load` intrinsic in .ll test that checks the verifier catches bad type when passing a fixed-width vector where a scalable is expected.
- Removed definition of `abs` and `neg` intrinsics that were previously used in the above test, these are implemented in the follow-up patch.
- Updated call to CreateIntrinsic in IRBuilderTest to make types argument empty for call to fp-convert intrinsic, only required for overloaded types.


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

https://reviews.llvm.org/D65930

Files:
  include/llvm/IR/Intrinsics.h
  include/llvm/IR/IntrinsicsAArch64.td
  include/llvm/Support/ScalableSize.h
  lib/IR/Function.cpp
  test/Verifier/intrinsic-bad-arg-type.ll
  unittests/IR/IRBuilderTest.cpp
  utils/TableGen/IntrinsicEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65930.216610.patch
Type: text/x-patch
Size: 9098 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190822/360529f3/attachment.bin>


More information about the llvm-commits mailing list