[Lldb-commits] [lldb] [llvm] [lldb][bazel] Support parse_headers (PR #232936)

[Pavel Labath via lldb-commits]

https://github.com/labath commented:

The blaze changes make sense. Given the current state of dependencies, trying to make each directory a separate target is not very helpful.

What I don't understand is how to the source code changes fit into this. In particular, the part about moving the ClangFunctionCallerHelper structors out of line. I am guessing it has something to do with incomplete (forward-declared) objects, but I don't know what, as I would have expected some changes in #included files as well. Can you explain how that works?
Also, is it possible that the files you are modifying are not IWYU clean -- and that making them clean would break the property you're trying to enforce?

https://github.com/llvm/llvm-project/pull/232936