Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(100)

Issue 1085733002: Subzero: Auto-detect cmake versus autoconf LLVM build. (Closed)

Created:
5 years, 8 months ago by Jim Stichnoth
Modified:
5 years, 8 months ago
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Subzero: Auto-detect cmake versus autoconf LLVM build. The CMAKE=1 option is no longer needed. Pretty much all the tools we need are now in pnacl_newlib_raw/bin, so use PNACL_BIN_PATH set to that instead of using LLVM_BIN_PATH and BINUTILS_BIN_PATH. However, for the autoconf build, libgtest and libtest_main and clang-format are only under the llvm_x86_64_linux_work directory, so they need special casing. This also means that you have to actually do an LLVM build and not blow away the work directory in order to "make check-unit" or "make format". BUG= none R=jvoung@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=0a9e12617f007a24efe5cb70dbebf53f45abbd3e

Patch Set 1 #

Patch Set 2 : Better explanation of the hack #

Patch Set 3 : Make a best effort at locating clang-format #

Patch Set 4 : Combine {LLVM,BINUTILS}_BIN_PATH into PNACL_BIN_PATH, and similar #

Total comments: 4

Patch Set 5 : Fix the sb build libs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -74 lines) Patch
M Makefile.standalone View 1 2 3 4 9 chunks +37 lines, -28 lines 0 comments Download
M crosstest/lit.cfg View 1 2 3 1 chunk +4 lines, -4 lines 0 comments Download
M pydir/run-pnacl-sz.py View 1 2 3 5 chunks +9 lines, -13 lines 0 comments Download
M tests_lit/lit.cfg View 1 2 3 4 chunks +15 lines, -29 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
Jim Stichnoth
5 years, 8 months ago (2015-04-17 22:06:11 UTC) #2
jvoung (off chromium)
otherwise LGTM https://codereview.chromium.org/1085733002/diff/60001/Makefile.standalone File Makefile.standalone (right): https://codereview.chromium.org/1085733002/diff/60001/Makefile.standalone#newcode128 Makefile.standalone:128: GTEST_LIB_PATH ?= Add a comment that gtest ...
5 years, 8 months ago (2015-04-17 22:22:57 UTC) #3
Jim Stichnoth
https://codereview.chromium.org/1085733002/diff/60001/Makefile.standalone File Makefile.standalone (right): https://codereview.chromium.org/1085733002/diff/60001/Makefile.standalone#newcode128 Makefile.standalone:128: GTEST_LIB_PATH ?= On 2015/04/17 22:22:57, jvoung wrote: > Add ...
5 years, 8 months ago (2015-04-17 23:06:22 UTC) #4
Jim Stichnoth
5 years, 8 months ago (2015-04-21 16:59:26 UTC) #5
Message was sent while issue was closed.
Committed patchset #5 (id:80001) manually as
0a9e12617f007a24efe5cb70dbebf53f45abbd3e (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698