ab added a comment.
Thanks for checking, much appreciated!
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58306/new/
https://reviews.llvm.org/D58306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
This revision was automatically updated to reflect the committed changes.
Closed by commit rL354538: [AArch64] Change size suffix for FP16FML intrinsics.
(authored by ab, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://revi
SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.
LGTM
The ACLE has been updated and a new version with change included will be
released soon.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58306/n
SjoerdMeijer added a comment.
I am discussing this with our GCC team as we would like both Clang/GCC
implementation to be the same. But you're right that _f16 looks like to be the
more consistent choice. I will let you know as soon I know more.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
ab created this revision.
ab added reviewers: SjoerdMeijer, bryanpkc.
ab added a project: clang.
Herald added a subscriber: javed.absar.
These currently use _u32, but they should instead use _f32 or _f16, the types
of the accumulator, and of the multiplication.
I'm starting with _f16 (because th
