mshawcroft commented on a change in pull request #5456:
URL: https://github.com/apache/incubator-tvm/pull/5456#discussion_r418525124
##########
File path: Makefile
##########
@@ -36,18 +36,30 @@ INCLUDE_FLAGS = -Iinclude -I$(DLPACK_PATH)/include
-I$(DMLC_CORE_PATH)/include
PKG_CFLAGS = -std=c++11 -Wall -O2 $(INCLUDE_FLAGS) -fPIC
PKG_LDFLAGS =
+# a way to pass in extra args to the various cmake calls
+# for an example see `make build-llvm`
+CMAKE_ARGS =
+
+# a way to pass in extra args to the make calls, ex: running with 8 threads
via "MAKE_ARGS='-j8'"
+MAKE_ARGS =
all:
- @mkdir -p build && cd build && cmake .. && $(MAKE)
+ mkdir -p build && cd build && cmake $(CMAKE_ARGS) .. && $(MAKE)
$(MAKE_ARGS)
Review comment:
Looks like @ was inadvertently removed here.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]