This is an archive of the discontinued LLVM Phabricator instance.

Use InitLLVM in clang-tidy
ClosedPublic

Authored by MrTrillian on Nov 25 2019, 1:35 PM.

Details

Summary

Update clang-tidy to use InitLLVM, like several other llvm tools that were previously updated. On Windows, this allows clang-tidy to operate on arguments containing characters which cannot be represented in the system's ANSI code page such as filenames with Unicode characters.

Fixes bugzilla bug 43751

Patch by Tristan Labelle.

Diff Detail

Event Timeline

MrTrillian created this revision.Nov 25 2019, 1:35 PM

This is my first LLVM submission! Please let me know if there's any part of the process I missed. Thanks :)

Bug link: https://bugs.llvm.org/show_bug.cgi?id=43751 (not sure how to format it in the description)
See previous change introducing InitLLVM: https://reviews.llvm.org/D45602

ruiu accepted this revision.Nov 25 2019, 4:50 PM

LGTM

This revision is now accepted and ready to land.Nov 25 2019, 4:50 PM

Thanks @ruiu ! Any further actions on my part required?

ruiu added a comment.Nov 26 2019, 6:09 PM

Nope, if you have a commit access, can you commit this?

I'm just a random guy popping in here for the first time, I doubt I have commit access :)

ruiu added a comment.Nov 27 2019, 6:15 PM

Sure, let me submit this patch for you.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptNov 27 2019, 9:02 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript