This is an archive of the discontinued LLVM Phabricator instance.

[ELF] Add a corrector for case mismatch problems
ClosedPublic

Authored by MaskRay on Nov 20 2019, 12:48 PM.

Event Timeline

MaskRay created this revision.Nov 20 2019, 12:48 PM
MaskRay updated this revision to Diff 230317.EditedNov 20 2019, 12:49 PM
MaskRay added reviewers: grimar, peter.smith, ruiu.

Internally we have a user who sees undefined reference to ...::UpdateToGet when they define UpdateToGET in a .cc file while declaring the function UpdateToGet in a .h file. It was suggested that such case mismatch spell corrector can be helpful.

MaskRay edited the summary of this revision. (Show Details)Nov 20 2019, 12:49 PM
Harbormaster completed remote builds in B41264: Diff 230317.
MaskRay updated this revision to Diff 230318.Nov 20 2019, 12:52 PM

Fix a comment...

peter.smith accepted this revision.Nov 21 2019, 4:58 AM

I think this is a good idea, I'd say most of my typos are of this form, such as SubTarget vs Subtarget. Will be worth waiting to see if anyone in US timezone has any objections.

This revision is now accepted and ready to land.Nov 21 2019, 4:58 AM
grimar accepted this revision.EditedNov 21 2019, 5:46 AM

LGTM too.

MaskRay updated this revision to Diff 230564.Nov 21 2019, 4:26 PM

Rebase after we decide to use symtab->symbols()

thakis added a subscriber: thakis.Nov 22 2019, 1:53 AM

This one looks like it would make sense in the COFF linker too.

This revision was automatically updated to reflect the committed changes.