This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Remove `CMAKE_.*_OUTPUT_DIRECTORY` (NFCI)
ClosedPublic

Authored by modocache on Sep 28 2017, 7:38 PM.

Details

Summary

Three CMAKE_.*_OUTPUT_DIRECTORY variables used to be set in CMake and
referenced in various other parts of the project. However, in r198205
chapuni added a note to "don't set them anymore", and any remaining
references to them were subsequently removed in r198316 and r199592.

Now that the variables are no longer used anywhere, remove them, along
with the comments advising against using them any longer.

Test Plan:
I ran check-all and confirmed the tests built and passed.

Event Timeline

modocache created this revision.Sep 28 2017, 7:38 PM
beanz accepted this revision.Sep 29 2017, 11:10 AM

LGTM. Please watch the bots closely with this change because CMake can be used in strange ways.

This revision is now accepted and ready to land.Sep 29 2017, 11:10 AM
modocache closed this revision.Sep 29 2017, 12:36 PM

Thanks @beanz! I'll keep an eye on the bots. Here goes nothing! :)

It turns out polly makes use of these variables. I'll revert this for now and try again after taking a closer look at polly.