This is an archive of the discontinued LLVM Phabricator instance.

[llvm] NFC: Fix trivial typos in comments
ClosedPublic

Authored by kiszk on Jan 3 2020, 5:31 AM.

Diff Detail

Event Timeline

kiszk created this revision.Jan 3 2020, 5:31 AM
jhenderson accepted this revision.Jan 3 2020, 5:38 AM

LGTM, with one nit.

llvm/lib/Target/AMDGPU/R600AsmPrinter.cpp
1

Whilst you're here, can you remove the extra space after "printer" please.

This revision is now accepted and ready to land.Jan 3 2020, 5:38 AM
kiszk marked an inline comment as done.Jan 3 2020, 6:20 AM
kiszk added inline comments.
llvm/lib/Target/AMDGPU/R600AsmPrinter.cpp
1

Sure, I will do it soon.

kiszk updated this revision to Diff 236049.Jan 3 2020, 6:26 AM
  • address review comment
kiszk added a comment.Jan 3 2020, 6:33 AM

I do not have commit rights to the repository. Could someone commit this change if it is OK?
Kazuaki Ishizaki <ishizaki@jp.ibm.com>

I do not have commit rights to the repository. Could someone commit this change if it is OK?
Kazuaki Ishizaki <ishizaki@jp.ibm.com>

Sure, assuming nobody else does before then, I'll commit on Monday.

This revision was automatically updated to reflect the committed changes.