This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Support externalizing debug info on non-Darwin platforms
ClosedPublic

Authored by phosek on Jan 11 2017, 12:53 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek updated this revision to Diff 84010.Jan 11 2017, 12:53 PM
phosek retitled this revision from to [CMake] Support externalizing debug info on non-Darwin platforms.
phosek updated this object.
phosek added a reviewer: beanz.
phosek set the repository for this revision to rL LLVM.
phosek added a subscriber: llvm-commits.
beanz accepted this revision.Feb 8 2017, 3:02 PM

A few small comments below. Otherwise LGTM.

cmake/modules/AddLLVM.cmake
1398

This block should be unneeded because you're already setting strip_command above.

1410–1414

This also should be unneeded.

This revision is now accepted and ready to land.Feb 8 2017, 3:02 PM
phosek updated this revision to Diff 87916.Feb 9 2017, 4:48 PM
phosek marked 2 inline comments as done.
phosek updated this revision to Diff 88574.Feb 15 2017, 10:18 AM
beanz added a comment.Feb 15 2017, 3:05 PM

LGTM! Thanks!

This revision was automatically updated to reflect the committed changes.