[Lldb-commits] [PATCH] D25922: Test infra: expose CFLAGS_NO_ARCH for use by test custom build rules

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 25 04:24:53 PDT 2016

labath added a comment.

I am not that excited by this, but I don't see a much better way to achieve the result. :)
 Possibly I'd consider making the variable name positive (instead of CFLAGS_NO_DEBUG, have a INCLUDES var, and then the test can use `$(INCLUDES) $(TRIPLE_CFLAGS)`).

Comment at: Python/lldbsuite/test/make/Makefile.rules:197
-ifeq "$(OS)" "Darwin"
Are you sure this won't actually be missed on darwin for other tests? `-archi386` does not seem to be a valid argument to clang (you need the space here).


More information about the lldb-commits mailing list