[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