This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Fix add_subdirectory llvm builds
ClosedPublic

Authored by sebastian-ne on Jul 26 2022, 1:42 AM.

Details

Summary

Fixes a regression from D117973, that used CMAKE_BINARY_DIR instead of
LLVM_BINARY_DIR in some places.

Diff Detail

Event Timeline

sebastian-ne created this revision.Jul 26 2022, 1:42 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 26 2022, 1:42 AM
sebastian-ne requested review of this revision.Jul 26 2022, 1:42 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 26 2022, 1:42 AM
dstuttard accepted this revision.Jul 26 2022, 2:23 AM
dstuttard added a subscriber: dstuttard.

LGTM

This revision is now accepted and ready to land.Jul 26 2022, 2:23 AM
This revision was landed with ongoing or failed builds.Jul 26 2022, 2:25 AM
This revision was automatically updated to reflect the committed changes.

Thanks! good catch.