[PATCH] D69508: [ARM] Add vrev32 NEON fp16 patterns

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 06:53:11 PDT 2019


SjoerdMeijer added a comment.

> These Neon patterns for vrev32.16 appear to be missing, only the i16 patterns are present.

Hmm, that's kind of what I was expecting: support for i16s, but nog f16s because that not only requires `HasNeon` but also `HasFullFP16`.  Without HasFullFP16, are vectors of f16s legal?


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

https://reviews.llvm.org/D69508





More information about the llvm-commits mailing list