On Tue, Apr 14, 2015 at 3:08 PM, Greg Clayton <[email protected]> wrote: > Author: gclayton > Date: Tue Apr 14 17:08:17 2015 > New Revision: 234949 > > URL: http://llvm.org/viewvc/llvm-project?rev=234949&view=rev > Log: > MacOSX needs a space between $(ARCHFLAG) and $(ARCH) otherwise we try to call > clang with "-archx86_64" which doesn't work. > > > Modified: > lldb/trunk/test/make/Makefile.rules > > Modified: lldb/trunk/test/make/Makefile.rules > URL: > http://llvm.org/viewvc/llvm-project/lldb/trunk/test/make/Makefile.rules?rev=234949&r1=234948&r2=234949&view=diff > ============================================================================== > --- lldb/trunk/test/make/Makefile.rules (original) > +++ lldb/trunk/test/make/Makefile.rules Tue Apr 14 17:08:17 2015 > @@ -117,11 +117,11 @@ else > endif > > CFLAGS ?= -g -O0 > -CFLAGS += $(ARCHFLAG)$(ARCH) $(FRAMEWORK_INCLUDES) $(CFLAGS_EXTRAS) > -I$(LLDB_BASE_DIR)include > +CFLAGS += $(ARCHFLAG) $(ARCH) $(FRAMEWORK_INCLUDES) $(CFLAGS_EXTRAS) > -I$(LLDB_BASE_DIR)include
This breaks linux as the arch flags now become "-m 32" or "-m 64" which clang or GCC will not accept. How about making ARCHFLAG ?= "-arch " on line 68? > CFLAGS += -include $(THIS_FILE_DIR)test_common.h > > # Use this one if you want to build one part of the result without debug > information: > -CFLAGS_NO_DEBUG = -O0 $(ARCHFLAG)$(ARCH) $(FRAMEWORK_INCLUDES) > $(CFLAGS_EXTRAS) > +CFLAGS_NO_DEBUG = -O0 $(ARCHFLAG) $(ARCH) $(FRAMEWORK_INCLUDES) > $(CFLAGS_EXTRAS) > > CXXFLAGS += -std=c++11 > CXXFLAGS += $(CFLAGS) > > > _______________________________________________ > lldb-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits _______________________________________________ lldb-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
