[PATCH] D74333: [ARM,MVE] Add intrinsics for FP rounding operations.

2020-02-18 Thread Simon Tatham via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc8b3196e5430: [ARM,MVE] Add intrinsics for FP rounding operations. (authored by simon_tatham). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74333/new/ http

[PATCH] D74333: [ARM,MVE] Add intrinsics for FP rounding operations.

2020-02-11 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham added a comment. In D74333#1869788 , @miyuki wrote: > vrintx is not mentioned anywhere in the patch. Should it say "llvm.rint" > instead? No, I meant what I said: `vrintx` is the MVE machine instruction that `llvm.rint` already instruction

[PATCH] D74333: [ARM,MVE] Add intrinsics for FP rounding operations.

2020-02-11 Thread Mikhail Maltsev via Phabricator via cfe-commits
miyuki added a comment. > vrintn behaves the same as vrintx vrintx is not mentioned anywhere in the patch. Should it say "llvm.rint" instead? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74333/new/ https://reviews.llvm.org/D74333

[PATCH] D74333: [ARM,MVE] Add intrinsics for FP rounding operations.

2020-02-11 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham updated this revision to Diff 243790. simon_tatham added a comment. Rebased to current master. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74333/new/ https://reviews.llvm.org/D74333 Files: clang/include/clang/Basic/arm_mve.td cl

[PATCH] D74333: [ARM,MVE] Add intrinsics for FP rounding operations.

2020-02-10 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham created this revision. simon_tatham added reviewers: dmgreen, miyuki, MarkMurrayARM, ostannard. Herald added subscribers: llvm-commits, cfe-commits, hiraditya, kristof.beyls. Herald added projects: clang, LLVM. This adds the unpredicated forms of six different MVE intrinsics which all