This is an archive of the discontinued LLVM Phabricator instance.

CMake: Stop using LLVM's custom parse_arguments. NFC
ClosedPublic

Authored by filcab on Jun 17 2015, 10:01 PM.

Details

Summary

Use CMake's cmake_parse_arguments() instead.
It's called in a slightly different way, but supports all our use cases.
It's in CMake 2.8.8, which is our minimum supported version.

CMake 3.0 doc (roughly the same. No direct link to 2.8.8 doc):
http://www.cmake.org/cmake/help/v3.0/module/CMakeParseArguments.html?highlight=cmake_parse_arguments

Since I was already changing these calls, I changed ARCH and LIB into
ARCHS and LIBS to make it more clear that they're lists of arguments.

Diff Detail

Event Timeline

filcab updated this revision to Diff 27912.Jun 17 2015, 10:01 PM
filcab retitled this revision from to CMake: Stop using LLVM's custom parse_arguments. NFC.
filcab updated this object.
filcab edited the test plan for this revision. (Show Details)
filcab added reviewers: eugenis, samsonov, beanz.
filcab added a subscriber: Unknown Object (MLST).
filcab updated this revision to Diff 27916.Jun 17 2015, 10:30 PM

Missed two s/ARCH/ARCHS/

I don't oppose as far as it works.

beanz accepted this revision.Jun 18 2015, 10:48 AM
beanz edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jun 18 2015, 10:48 AM
samsonov accepted this revision.Jun 18 2015, 10:56 AM
samsonov edited edge metadata.

LGTM as long as it works.

This revision was automatically updated to reflect the committed changes.