This is an archive of the discontinued LLVM Phabricator instance.

Fixed loads of typos
AbandonedPublic

Authored by GabrielRavier on Jul 30 2022, 7:21 AM.

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 spent a few days looking at it to find probable typos and
fixed a few hundred of them (note, this is not anywhere near all of
them, but it seems like a good start).

Note: A very large majority of these are in comments and other places
that, as far as I can see, should not be capable of breaking anything,
but there are also a few variable name changes and things like that.

Diff Detail

Event Timeline

GabrielRavier created this revision.Jul 30 2022, 7:21 AM
Herald added a reviewer: bollu. · View Herald Transcript
Herald added a reviewer: rriddle. · View Herald Transcript
Herald added a reviewer: aartbik. · View Herald Transcript
Herald added a reviewer: MaskRay. · View Herald Transcript
Herald added a reviewer: ftynse. · View Herald Transcript
Herald added a reviewer: aartbik. · View Herald Transcript
Herald added a reviewer: sjarus. · View Herald Transcript
Herald added a reviewer: rafauler. · View Herald Transcript
Herald added a reviewer: Amir. · View Herald Transcript
Herald added a reviewer: maksfb. · View Herald Transcript
Herald added a reviewer: dang. · View Herald Transcript
Herald added a reviewer: NoQ. · View Herald Transcript
Herald added a reviewer: ributzka. · View Herald Transcript
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
GabrielRavier requested review of this revision.Jul 30 2022, 7:21 AM
Herald added a reviewer: sstefan1. · View Herald Transcript
Herald added a reviewer: herhut. · View Herald Transcript
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
yota9 removed a subscriber: yota9.Jul 30 2022, 9:10 AM
clementval requested changes to this revision.Jul 30 2022, 10:17 AM
clementval added inline comments.
flang/docs/Extensions.md
157

This is just wrong

This revision now requires changes to proceed.Jul 30 2022, 10:17 AM

Could you split this up per project? Large patches like this are really hard to review, since there is no single person/small group that can approve everything.

Well, I assumed if I split it up per project I would have been told to merge all the patches into a big one to avoid having a bunch of small ones essentially all doing the same thing, but then again this also makes sense, I'll split it up.

flang/docs/Extensions.md
157

I'll correct it in my per-project patch

GabrielRavier abandoned this revision.Jul 30 2022, 10:32 AM