[Lldb-commits] [PATCH] D58125: Add ability to import std module into expression parser to improve C++ debugging

Adrian Prantl via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 18 09:26:12 PST 2019

aprantl added inline comments.

Comment at: lldb/packages/Python/lldbsuite/test/expression_command/import-std-module/conflicts/Makefile:3
+CXXFLAGS += -std=c++11 -fmodules -glldb -fimplicit-module-maps
+CXX_SOURCES := main.cpp
teemperor wrote:
> aprantl wrote:
> > Makefile.rules defines MANDATORY_MODULE_BUILD_FLAGS for this. Can you instead add -fimplicit-module-maps to that variable and use it here?
> Well, we would need to add `-glldb` (needed on Linux at least, where this is not default) and `-fimplicit-module-maps` to that flag (and maybe even `-fcxx-modules` on macOS, but I'm not sure about that). Also I'm not sure if we want to have `-gmodules` in the same flag set as the importing of `std` should also work without. What about a separate `MANDATORY_CXXMODULE_BUILD_FLAGS` that we use in all the tests?
That works for me. The bit I really care about is the `-fmodules-cache-path`. If we don't set it an incremental bot may blow up when upstream clang changes the serialization format.



More information about the lldb-commits mailing list