Page MenuHomePhabricator

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

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

Details

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
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?

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.

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
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
This revision was automatically updated to reflect the committed changes.