The Annotations helper class does not have a gtest or gmock dependency, but because it's bundled with the rest of TestingSupport, it gets one. By splitting it out, a target can use it without being forced to use LLVM's copy of gtest.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
It seems like the same logic would extend to the CMake build. Could we make the same change there?
Comment Actions
The simplest (only?) way to do that is to have it literally in a separate directory, so I did that. It's a bit large now but mechanical.
Comment Actions
LGTM (+/- nits), but maybe good to get review from someone more closely associated with the TestingSupport library
utils/bazel/llvm-project-overlay/llvm/BUILD.bazel | ||
---|---|---|
4529 | I think this comment is now a bit redundant |
Comment Actions
This looks good, i think it also doesn't belong in Testing/Support because it's not a peer to anything in Support/.
I think you can use PARTIAL_SOURCES_INTENDED to partition directories, but it's unusual and a bit of a hassle. Either way seems fine to me.
I think this comment is now a bit redundant