Page MenuHomePhabricator

Add gtest unit tests to the CMake build.
Needs ReviewPublic

Authored by zturner on Mar 12 2015, 1:53 PM.

Details

Summary

After this patch, ninja will automatically build the LLDB unit tests. More work needs to be done to get the unit tests to run automatically as part of "ninja check-lldb", but the plan is to do that in a followup patch.

Diff Detail

Event Timeline

zturner updated this revision to Diff 21875.Mar 12 2015, 1:53 PM
zturner retitled this revision from to Add gtest unit tests to the CMake build..
zturner updated this object.
zturner edited the test plan for this revision. (Show Details)
zturner added reviewers: ovyalov, sivachandra.
zturner added a subscriber: Unknown Object (MLST).
zturner updated this revision to Diff 21876.Mar 12 2015, 1:55 PM

Added missing LLVM header in gtest_common.h

tberghammer added inline comments.
gtest/CMakeLists.txt
13–19

AFAIK the unit tests don't have to be linked together with liblldb. They contain some mock implementation for all dependency from liblldb so I wouldn't be surprised on some linker error because of duplicate definition.