This revision was automatically updated to reflect the committed changes.
Closed by commit rC342034: [CodeGen][ARM] Coerce FP16 vectors to integer
vectors when needed (authored by miyuki, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D50507
Files:
lib/CodeGen/TargetInfo.cpp
efriedma accepted this revision.
efriedma added a comment.
LGTM
https://reviews.llvm.org/D50507
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
miyuki added a comment.
ping
https://reviews.llvm.org/D50507
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
miyuki updated this revision to Diff 160317.
miyuki added a comment.
Handle return of homogeneous aggregates
https://reviews.llvm.org/D50507
Files:
lib/CodeGen/TargetInfo.cpp
test/CodeGen/arm-vfp16-arguments.c
test/CodeGen/arm_neon_intrinsics.c
Index: test/CodeGen/arm_neon_intrinsics.c
efriedma added inline comments.
Comment at: lib/CodeGen/TargetInfo.cpp:5788
+ llvm::Type *Ty = llvm::ArrayType::get(NewVecTy, Members);
+ return ABIArgInfo::getDirect(Ty, 0, nullptr, false);
+}
Do we need equivalent code in
miyuki updated this revision to Diff 160121.
miyuki edited the summary of this revision.
miyuki added a comment.
Fix handling of homogeneous aggregates of FP16 vectors
https://reviews.llvm.org/D50507
Files:
lib/CodeGen/TargetInfo.cpp
test/CodeGen/arm-vfp16-arguments.c
efriedma added a comment.
Do we need to check for homogeneous aggregates of half vectors somewhere?
https://reviews.llvm.org/D50507
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
miyuki created this revision.
miyuki added reviewers: eli.friedman, olista01, SjoerdMeijer.
Herald added a reviewer: javed.absar.
Herald added subscribers: chrib, kristof.beyls.
On targets that do not support FP16 natively LLVM currently legalizes
vectors of FP16 values by scalarizing them and
