[Lldb-commits] [PATCH] D132868: [lldb][test] Enable clang modules in API/tools/lldb-server (NFC)

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

I don't think this will work in general, as the modules feature requires all your dependencies to be modularized, and linux systems generally don't have that. You might be able to do a feature like "build no_debug_info tests with modules if the platform supports it" and then make sure darwin supports it.

Or you could split this test up into multiple files. Personally I think it would be super cool if we were able to compile this binary just once (like, putting the `build` action in the `@static setUpClass(cls)` method or something), as all of the tests in this file (and in this folder, actually) are going to be using the same one.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132868/new/

https://reviews.llvm.org/D132868