[compiler-rt] [cmake] [interception] Remove duplicate gtest from test COMPILE_DEPS

[mgorny]

Summary

Fix the gtest dependency to be included in DEPS only, rather than
in COMPILE_DEPS + DEPS. The former variable is apparently used to
provide unconditional dependencies, while the latter are only used
for non-standalone builds. Since they are concatenated, specifying gtest
in both is redundant. Furthermore, including it in COMPILE_DEPS causes
build failure for standalone builds where 'gtest' target is not present.

[george.karpenkov]

Yep, thanks! This was probably the root cause of the failure.

lib/interception/tests/CMakeLists.txt

 # Adds interception tests for architecture.
 macro(add_interception_tests_for_arch arch)
   set(INTERCEPTION_TEST_OBJECTS)
   get_interception_lib_for_arch(${arch} INTERCEPTION_COMMON_LIB)
   generate_compiler_rt_tests(INTERCEPTION_TEST_OBJECTS
     InterceptionUnitTests "Interception-${arch}-Test" ${arch}
     RUNTIME ${INTERCEPTION_COMMON_LIB}
     SOURCES ${INTERCEPTION_UNITTESTS} ${COMPILER_RT_GTEST_SOURCE}
-    COMPILE_DEPS gtest ${INTERCEPTION_TEST_HEADERS}
+    COMPILE_DEPS ${INTERCEPTION_TEST_HEADERS}
     DEPS gtest
     CFLAGS ${INTERCEPTION_TEST_CFLAGS_COMMON}
     LINK_FLAGS ${INTERCEPTION_TEST_LINK_FLAGS_COMMON})
 endmacro()
 
 if(COMPILER_RT_CAN_EXECUTE_TESTS AND NOT ANDROID AND NOT APPLE)
   # We use just-built clang to build interception unittests, so we must
   # be sure that produced binaries would work.