This is an archive of the discontinued LLVM Phabricator instance.

[clang-tools-extra] Fixed a number of typos
ClosedPublic

Authored by GabrielRavier on Jul 30 2022, 3:01 PM.

Details

Summary

I went over the output of the following mess of a command:

(ulimit -m 2000000; ulimit -v 2000000; git ls-files -z | parallel --xargs -0 cat | aspell list --mode=none --ignore-case | grep -E '^[A-Za-z][a-z]*$' | sort | uniq -c | sort -n | grep -vE '.{25}' | aspell pipe -W3 | grep : | cut -d' ' -f2 | less)

and proceeded to spend a few days looking at it to find probable typos
and fixed a few hundred of them in all of the llvm project (note, the
ones I found are not anywhere near all of them, but it seems like a
good start).

Diff Detail

Event Timeline

GabrielRavier created this revision.Jul 30 2022, 3:01 PM
GabrielRavier requested review of this revision.Jul 30 2022, 3:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 30 2022, 3:01 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
kadircet accepted this revision.Aug 1 2022, 4:11 AM

thanks, lgtm!

This revision is now accepted and ready to land.Aug 1 2022, 4:11 AM

also LMK if I should land this for you

Well I'm pretty sure I don't have commit rights to LLVM and it seems landing means to push the revision onto main, so that would be nice, yes

This revision was automatically updated to reflect the committed changes.