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

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Mar 4 12:12:52 PST 2019

labath added inline comments.

Comment at: lldb/packages/Python/lldbsuite/test/make/Makefile.rules:257
-CFLAGS += -I$(SRCDIR) -include $(THIS_FILE_DIR)test_common.h -I$(THIS_FILE_DIR)
+ifndef NO_INC_DIRS
+  CFLAGS += -I$(SRCDIR) -include $(THIS_FILE_DIR)test_common.h -I$(THIS_FILE_DIR)
teemperor wrote:
> labath wrote:
> > I'm wondering why have you needed to do this (and whether we can figure out another solution to that).
> Yeah I should have added this as a comment to the test. Our sysroot system has only a few dummy headers, so we can resolve the `#include <sys/prctl.h>` in `test_common.h`. The removed `-I` flags are actually unintentional, I'll revert removing them.
Ah, cool. I suspected something like that. In that case I suggest also renaming the flag to NO_TEST_COMMON_H or something. (We should probably replace the force-include by an actual include some day.)



More information about the lldb-commits mailing list