[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