SjoerdMeijer added a comment.
FYI: a new ACLE version has been published, please find it here:
https://developer.arm.com/architectures/system-architectures/software-standards/acle
The "Neon Intrinsics" section contains these new intrinsics.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
ab added inline comments.
Comment at: cfe/trunk/test/CodeGen/aarch64-neon-fp16fml.c:12
+
+float32x2_t test_vfmlal_low_u32(float32x2_t a, float16x4_t b, float16x4_t c) {
+// CHECK-LABEL: define <2 x float> @test_vfmlal_low_u32(<2 x float> %a, <4 x
half> %b, <4 x half> %c)
SjoerdMeijer added inline comments.
Comment at: cfe/trunk/test/CodeGen/aarch64-neon-fp16fml.c:12
+
+float32x2_t test_vfmlal_low_u32(float32x2_t a, float16x4_t b, float16x4_t c) {
+// CHECK-LABEL: define <2 x float> @test_vfmlal_low_u32(<2 x float> %a, <4 x
half> %b, <4 x half>
SjoerdMeijer added inline comments.
Comment at: cfe/trunk/test/CodeGen/aarch64-neon-fp16fml.c:12
+
+float32x2_t test_vfmlal_low_u32(float32x2_t a, float16x4_t b, float16x4_t c) {
+// CHECK-LABEL: define <2 x float> @test_vfmlal_low_u32(<2 x float> %a, <4 x
half> %b, <4 x half>
ab added inline comments.
Herald added a subscriber: jdoerfert.
Herald added a project: LLVM.
Comment at: cfe/trunk/test/CodeGen/aarch64-neon-fp16fml.c:12
+
+float32x2_t test_vfmlal_low_u32(float32x2_t a, float16x4_t b, float16x4_t c) {
+// CHECK-LABEL: define <2 x float>
This revision was automatically updated to reflect the committed changes.
Closed by commit rL345344: [AArch64] Implement FP16FML intrinsics (authored by
bryanpkc, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
bryanpkc added a comment.
In https://reviews.llvm.org/D53633#1274621, @t.p.northover wrote:
> I think this is reasonable.
Thanks Tim. Could you also review https://reviews.llvm.org/D53632, which is the
LLVM part of this implementation?
Repository:
rC Clang
https://reviews.llvm.org/D53633
t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.
I think this is reasonable.
Repository:
rC Clang
https://reviews.llvm.org/D53633
___
cfe-commits mailing list
bryanpkc created this revision.
bryanpkc added reviewers: SjoerdMeijer, bogden, efriedma, t.p.northover.
Herald added subscribers: cfe-commits, kristof.beyls, javed.absar.
Generate the FP16FML intrinsics into arm_neon.h (AArch64 only for now).
Add two new type modifiers to NeonEmitter to handle
