This is an archive of the discontinued LLVM Phabricator instance.

[clang] [python] [tests] Update priority values in code completion test
ClosedPublic

Authored by mgorny on Nov 9 2017, 5:29 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

mgorny created this revision.Nov 9 2017, 5:29 AM
ilya-biryukov accepted this revision.Nov 9 2017, 6:05 AM

LGTM.

Can we run those tests as part of check-all cmake target or setup a buildbot that runs those? Seems surprising it went unnoticed for so long.

This revision is now accepted and ready to land.Nov 9 2017, 6:05 AM

LGTM.

Can we run those tests as part of check-all cmake target or setup a buildbot that runs those? Seems surprising it went unnoticed for so long.

I was thinking of that but I think it'd be nice to find someone who knows them better. I know they're loading libclang.so dynamically somehow, and I'm not really sure how to make sure that they're going to test the right library. On Gentoo we run the tests after installing clang to the system.

This revision was automatically updated to reflect the committed changes.