This is an archive of the discontinued LLVM Phabricator instance.

[Release Notes] Add release note for llvm-mca
ClosedPublic

Authored by andreadb on Mar 19 2018, 10:28 AM.

Details

Summary

As the title says, this adds a small paragraph about llvm-mca in ReleaseNotes.rst.

Please let me know if it is okay to commit.

Thanks,
Andrea

Diff Detail

Event Timeline

andreadb created this revision.Mar 19 2018, 10:28 AM
Eugene.Zelenko added inline comments.
docs/ReleaseNotes.rst
48

Please :program: prefix in both references to tool.

andreadb updated this revision to Diff 138998.Mar 19 2018, 1:47 PM

Addressed review comments.

Added :program: prefix to every occurrence of llvm-mca

hans added a comment.Mar 20 2018, 12:30 AM

Thanks for adding release notes! Can you include a link to the CommandGuide/llvm-mca.html to make it easier for the curious reader to find more info?

andreadb updated this revision to Diff 139093.Mar 20 2018, 3:06 AM

Hi Hans,

Thanks for the feedback.

I modified the text so that the first occurrence of llvm-mca references the llvm-mca.rst in the CommandGuide.
I have verified that the href in the html page works fine.

hans accepted this revision.Mar 20 2018, 3:21 AM

Great, thanks!

This revision is now accepted and ready to land.Mar 20 2018, 3:21 AM
This revision was automatically updated to reflect the committed changes.