This is an archive of the discontinued LLVM Phabricator instance.

[flang] Fix typo in README.md
AbandonedPublic

Authored by k-arrows on Feb 23 2023, 8:50 PM.

Details

Reviewers
PeteSteinfeld
Summary

(1) replacing "includig" with "including"
(2) replacing "requres" with "requires"

Diff Detail

Event Timeline

k-arrows created this revision.Feb 23 2023, 8:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 23 2023, 8:50 PM
k-arrows requested review of this revision.Feb 23 2023, 8:50 PM

Thanks, @k-arrows! Note that you might also replace "includig" with "including".

k-arrows updated this revision to Diff 500394.Feb 25 2023, 1:59 AM
k-arrows edited the summary of this revision. (Show Details)
k-arrows added a reviewer: PeteSteinfeld.

fix another typo

This revision is now accepted and ready to land.Feb 27 2023, 6:45 AM
This comment was removed by k-arrows.
This revision now requires review to proceed.Feb 28 2023, 6:28 AM
k-arrows added a reviewer: PeteSteinfeld.EditedFeb 28 2023, 6:38 AM

@PeteSteinfeld

Thank you for your review, but I mistakenly removed you, so I re-added.
Could you please re-review and commit this patch because I don't have the required access rights?
Sorry to bother you with such a trivial fix, but I'm a complete novice at LLVM and its community.

@PeteSteinfeld

Thank you for your review, but I mistakenly removed you, so I re-added.
Could you please re-review and commit this patch because I don't have the required access rights?
Sorry to bother you with such a trivial fix, but I'm a complete novice at LLVM and its community.

I won't commit this for you. But I will give you whatever help you need to do it yourself.

If you don't have commit access, the first step is to request it -- https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access

Feel free to contact me on email or Slack to ask questions -- psteinfeld@nvidia.com

k-arrows abandoned this revision.Mar 1 2023, 5:28 AM