[PATCH] D49987: [ARM] Make FP16 vectors legal

Mikhail Maltsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 7 06:23:56 PDT 2018


miyuki updated this revision to Diff 159496.
miyuki added a comment.

I have added a hardfp test. Clang has some logic to handle scalar FP16 values, it performs softening when needed (e.g. tools/clang/test/CodeGen/arm-fp16-arguments.c), whereas NEON vectors are lowered into LLVM vector types. So we don't need additional changes neither in Clang nor in the LLVM scalar FP16 handing.


https://reviews.llvm.org/D49987

Files:
  lib/CodeGen/SelectionDAG/LegalizeFloatTypes.cpp
  lib/CodeGen/SelectionDAG/LegalizeTypes.h
  lib/Target/ARM/ARMISelLowering.cpp
  lib/Target/ARM/ARMISelLowering.h
  test/CodeGen/ARM/fp16-promote.ll
  test/CodeGen/ARM/fp16-v3.ll
  test/CodeGen/ARM/vfp16-calling-conv.ll
  test/Transforms/LoopVectorize/ARM/interleaved_cost.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49987.159496.patch
Type: text/x-patch
Size: 16369 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180807/dd5e8c48/attachment.bin>


More information about the llvm-commits mailing list