This is an archive of the discontinued LLVM Phabricator instance.

Differential D76513

[ReleaseNotes,ARM] MVE intrinsics are all implemented!
ClosedPublic

Authored by simon_tatham on Mar 20 2020, 10:02 AM.

Details

Reviewers
kristof.beyls
Commits
rGf282b6ab23a0: [ReleaseNotes,ARM] MVE intrinsics are all implemented!
Summary

The next release of LLVM will support the full ACLE spec for MVE intrinsics,
so it's worth saying so in the release notes.

Diff Detail

Event Timeline

simon_tatham created this revision.Mar 20 2020, 10:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 20 2020, 10:02 AM
Herald added subscribers: llvm-commits, dmgreen. · View Herald Transcript
kristof.beyls added a subscriber: hans.Mar 20 2020, 10:20 AM
kristof.beyls added inline comments.
llvm/docs/ReleaseNotes.rst
75–77

The note seems exactly right for the release notes.
However, I'm not fully sure if this should be in the LLVM or the Clang release notes. Or maybe it's best to go in both.
@hans , I wonder if you have an opinion on where release notes related to new C-level intrinsics for a specific backend should go?

Harbormaster completed remote builds in B49912: Diff 251683.Mar 20 2020, 10:50 AM
hans added a comment.Mar 24 2020, 1:47 AM

Sorry for the slow reply. For some reason I didn't see this until now.

llvm/docs/ReleaseNotes.rst
75–77

No strong opinion, but putting it in both the Clang and LLVM notes makes sense to me.

simon_tatham updated this revision to Diff 252252.Mar 24 2020, 2:51 AM

Added an entry to the Clang release notes as well.

Herald added a project: Restricted Project. · View Herald TranscriptMar 24 2020, 2:51 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Harbormaster completed remote builds in B50228: Diff 252252.Mar 24 2020, 3:44 AM
kristof.beyls accepted this revision.Mar 24 2020, 4:02 AM

LGTM. Thanks!

This revision is now accepted and ready to land.Mar 24 2020, 4:02 AM
Closed by commit rGf282b6ab23a0: [ReleaseNotes,ARM] MVE intrinsics are all implemented! (authored by simon_tatham). · Explain WhyMar 24 2020, 4:48 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
llvm/
docs/
3 lines

Diff 251683

llvm/docs/ReleaseNotes.rst

Show First 20 Lines • Show All 66 Lines • ▼ Show 20 Lines
Changes to building LLVM Changes to building LLVM
------------------------ ------------------------
Changes to the ARM Backend Changes to the ARM Backend
-------------------------- --------------------------
During this release ... During this release ...
* Implemented C-language intrinsics for the full Arm v8.1-M MVE instruction
set. ``<arm_mve.h>`` now supports the complete API defined in the Arm C
Language Extensions.
kristof.beylsUnsubmitted
Not Done
ReplyInline Actions

The note seems exactly right for the release notes.
However, I'm not fully sure if this should be in the LLVM or the Clang release notes. Or maybe it's best to go in both.
@hans , I wonder if you have an opinion on where release notes related to new C-level intrinsics for a specific backend should go?

kristof.beyls: The note seems exactly right for the release notes. However, I'm not fully sure if this should…
hansUnsubmitted
Not Done
ReplyInline Actions

No strong opinion, but putting it in both the Clang and LLVM notes makes sense to me.

hans: No strong opinion, but putting it in both the Clang and LLVM notes makes sense to me.
Changes to the MIPS Target Changes to the MIPS Target
-------------------------- --------------------------
During this release ... During this release ...
Changes to the PowerPC Target Changes to the PowerPC Target
▲ Show 20 Lines • Show All 67 Lines • Show Last 20 Lines