This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Add LLVM include dirs prior to gtest target in standalone build
ClosedPublic

Authored by mgorny on Oct 23 2022, 5:54 AM.

Details

Summary

Move include_directories() declaration before gtest targets are created
in standalone build. This fixes build failure due to gtest targets
being unable to find LLVM headers, e.g.:

/var/tmp/portage/dev-util/lldb-16.0.0_pre20221023/work/llvm/utils/unittest/googletest/include/gtest/internal/custom/raw-ostream.h:43:10: fatal error: llvm/ADT/Optional.h: No such file or directory

Diff Detail

Event Timeline

mgorny created this revision.Oct 23 2022, 5:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 23 2022, 5:54 AM
Herald added a subscriber: StephenFan. · View Herald Transcript
mgorny requested review of this revision.Oct 23 2022, 5:54 AM
mgorny updated this revision to Diff 469984.Oct 23 2022, 7:42 AM

Rebase to make it independent of D136551.

labath accepted this revision.Oct 24 2022, 1:44 AM
This revision is now accepted and ready to land.Oct 24 2022, 1:44 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 24 2022, 3:18 AM