This is an archive of the discontinued LLVM Phabricator instance.

Rename directory housing clang-include-fixer to be eponymous
ClosedPublic

Authored by thakis on Mar 24 2019, 10:31 AM.

Diff Detail

Event Timeline

thakis created this revision.Mar 24 2019, 10:31 AM
thakis edited reviewers, added: bkramer; removed: serge-sans-paille.Mar 24 2019, 10:31 AM
thakis added a subscriber: cfe-commits.

I accidentally hit some keyboard shortcut and created the revision before I was done adding reviewer and subscriber lists. Fixed now; apologies for the confusing email this will send.

bkramer accepted this revision.Mar 24 2019, 3:35 PM

lg. Is the reference from libclang still around? Might need an update.

This revision is now accepted and ready to land.Mar 24 2019, 3:35 PM

lg. Is the reference from libclang still around? Might need an update.

I only found http://llvm-cs.pcc.me.uk/tools/clang/tools/libclang/CIndex.cpp#8955 which in http://llvm-cs.pcc.me.uk/tools/clang/tools/libclang/CMakeLists.txt#55 depends on the clangIncludeFixerPlugin target and doesn't use a qualified path, as far as I can tell.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 25 2019, 7:08 AM
unittests/include-fixer/IncludeFixerTest.cpp