This is an archive of the discontinued LLVM Phabricator instance.

[AST] Fix typo in MicrosoftMangle
ClosedPublic

Authored by smeenai on Nov 14 2018, 10:33 AM.

Details

Summary

I noticed that we were spelling it as Artifical instead of Artificial
when working on the mangling for Obj-C RTTI. I'm putting this up for
review instead of committing it directly because I'm not 100% sure what
the policy here would be; it does clutter up the blame, but at the same
time it's fixing a gotcha for people writing new code.

Diff Detail

Event Timeline

smeenai created this revision.Nov 14 2018, 10:33 AM
rnk accepted this revision.Nov 14 2018, 11:15 AM

I'd say feel free to commit typo fixes like this in the future. I'd avoid fixing typos where there is spelling disagreement, like "prolog vs prologue", so just use your best judgement as always.
lgtm

This revision is now accepted and ready to land.Nov 14 2018, 11:15 AM
This revision was automatically updated to reflect the committed changes.