[llvm] [unittest] Add missing LLVM include directory to target (PR #148362)
Michał Górny via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 12 04:44:57 PDT 2025
https://github.com/mgorny created https://github.com/llvm/llvm-project/pull/148362
Add the top-level LLVM include directory to the `llvm_gtest` target, as it requires `llvm/Support/raw_os_ostream.h` header. This fixes a flang-rt unittest build failure introduced in #143682.
>From d9a37280973eace756fc3eba3450812b9ac0244a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20G=C3=B3rny?= <mgorny at gentoo.org>
Date: Sat, 12 Jul 2025 13:42:14 +0200
Subject: [PATCH] [unittest] Add missing LLVM include directory to target
Add the top-level LLVM include directory to the `llvm_gtest` target,
as it requires `llvm/Support/raw_os_ostream.h` header. This fixes
a flang-rt unittest build failure introduced in #143682.
---
third-party/unittest/CMakeLists.txt | 2 ++
1 file changed, 2 insertions(+)
diff --git a/third-party/unittest/CMakeLists.txt b/third-party/unittest/CMakeLists.txt
index ea3a310cbbfae..b9f3593320d77 100644
--- a/third-party/unittest/CMakeLists.txt
+++ b/third-party/unittest/CMakeLists.txt
@@ -74,9 +74,11 @@ if (NOT LLVM_ENABLE_THREADS)
target_compile_definitions(llvm_gtest PUBLIC GTEST_HAS_PTHREAD=0)
endif ()
+# Top-level include directory required for "llvm/Support/raw_os_ostream.h"
target_include_directories(llvm_gtest
PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/googletest/include>
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/googlemock/include>
+ $<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}/>
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}/llvm-gtest/>
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}/llvm-gmock/>
PRIVATE googletest googlemock
More information about the llvm-commits
mailing list