This revision was automatically updated to reflect the committed changes.
Closed by commit rL320019: [ARM] ACLE parallel arithmetic and DSP style
multiplications (authored by SjoerdMeijer).
Changed prior to commit:
https://reviews.llvm.org/D40888?vs=125705&id=125909#toc
Repository:
rL LLVM
This revision was automatically updated to reflect the committed changes.
Closed by commit rC320019: [ARM] ACLE parallel arithmetic and DSP style
multiplications (authored by SjoerdMeijer).
Repository:
rC Clang
https://reviews.llvm.org/D40888
Files:
include/clang/Basic/BuiltinsARM.def
lib
samparker accepted this revision.
samparker added a comment.
This revision is now accepted and ready to land.
Great, LGTM, many thanks for doing this!
https://reviews.llvm.org/D40888
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lis
SjoerdMeijer added inline comments.
Comment at: include/clang/Basic/BuiltinsARM.def:39
BUILTIN(__builtin_arm_qsub, "iii", "nc")
+BUILTIN(__builtin_arm_qdbl, "ii", "nc")
BUILTIN(__builtin_arm_ssat, "iiUi", "nc")
samparker wrote:
> Do we now need a codegen tests
samparker added a comment.
Thanks for looking into this!
Comment at: include/clang/Basic/BuiltinsARM.def:39
BUILTIN(__builtin_arm_qsub, "iii", "nc")
+BUILTIN(__builtin_arm_qdbl, "ii", "nc")
BUILTIN(__builtin_arm_ssat, "iiUi", "nc")
Do we now need a codegen te
SjoerdMeijer created this revision.
Herald added subscribers: kristof.beyls, javed.absar, aemerson.
This is a follow up of r302131, in which we forgot to add SemaChecking
tests. Adding these tests revealed two problems which have been fixed:
- added missing intrinsic __qdbl,
- properly ran
