Page MenuHomePhabricator

Stop passing site cfg files via --param to llvm-lit.
ClosedPublic

Authored by thakis on Apr 6 2020, 1:30 PM.

Details

Summary

This has been unnecessary since https://reviews.llvm.org/D37756.

https://reviews.llvm.org/D37838 removed it for llvm.

This removes it for clang, lld, clang-tools-extra (and for the GN build).

No intended behavior change.

Diff Detail

Event Timeline

thakis created this revision.Apr 6 2020, 1:30 PM
hans accepted this revision.Apr 7 2020, 1:10 AM
hans added a subscriber: hans.

lgtm

clang/test/CMakeLists.txt
113

This and ANALYZER_TEST_PARAMS_Z3 are just dropped because they're unused, right?

This revision is now accepted and ready to land.Apr 7 2020, 1:10 AM
thakis marked an inline comment as done.Apr 7 2020, 5:21 AM

Thanks!

clang/test/CMakeLists.txt
113

Yes. It looks like the uses of them were dropped accidentally in D62445. If we restore the uses, we can restore this too.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2020, 5:23 AM