Sidneym/add cmake #45
Conversation
Add options to measure test durations and provide more detail about test results. Also: reformat the arguments for the subsets of the test suite so they're easier to read. Signed-off-by: Brian Cain <[email protected]>
Signed-off-by: Brian Cain <[email protected]>
Signed-off-by: Brian Cain <[email protected]>
Signed-off-by: Brian Cain <[email protected]>
Signed-off-by: Brian Cain <[email protected]>
Windows QEMU was left out of the commits below. Let's catch the windows build up with the linux build: * 4deecaf Add slirp support for QEMU * 102f2f8 Add FDT support to qemu Signed-off-by: Brian Cain <[email protected]>
Signed-off-by: Brian Cain <[email protected]>
Use LLVM_ENABLE_PIC=ON to build the toolchains, required for building eld. FIXME: this has the effect of causing some of the cross-built toolchains to fail, we'll need to investigate this. Disabling these for now. Signed-off-by: Brian Cain <[email protected]>
| clang --version | ||
| ninja --version | ||
| cmake --version | ||
| python3.8 --version |
There was a problem hiding this comment.
IIRC this change will cause failures, please test it w/the container build used by the release process - ARTIFACT_TAG=x.y.z ./build-in-container.sh
| add_custom_target(eld | ||
| DEPENDS clone | ||
| COMMAND ${CMAKE_COMMAND} -E chdir llvm-project git clone [email protected]:cgit/eld.git | ||
| ) |
There was a problem hiding this comment.
This repo is not public and cannot be used here.
| @@ -0,0 +1,59 @@ | |||
| cmake_minimum_required(VERSION 3.22) | |||
There was a problem hiding this comment.
In general I don't love having so many different ways to build the toolchain. If we're going to use cmake-to-run-shell-to-run-cmake it feels like we can cut out middle-man somehow.
If the environment variables are a PITA (I get it, they are), let's just fix the scripts to no longer depend on environment variables (instead take args?).
Supporting both the docker and the ad-hoc build mechanisms is challenging and the only reason it's survived so long is that there's very few people who actually do these builds.
If we have some consensus that this cmake is preferred, then please also include an update to the README so that we can abandon any support for running it outside of cmake and docker.
| # Set variables with CACHE option to allow command line override | ||
| set(HOST_CLANG_VER 21 CACHE STRING "Host Clang version") | ||
| set(ARTIFACT_TAG 21.0.0 CACHE STRING "Artifact tag") | ||
| set(INSTALL_ROOT /tmp/$ENV{USER}/hexagon-qemu CACHE STRING "Install root") |
There was a problem hiding this comment.
This seems like it's not appropriately labeled to accept a whole toolchain - do we want it to be called hexagon-qemu? Also: tmpfs is probably not big enough to fit the biggest toolchains here.
There was a problem hiding this comment.
The commit msg suggests using: cmake -B ../build -DINSTALL_ROOT=$some_dir/install
This is just a place holder.
SidManning
force-pushed
the
sidneym/add-cmake
branch
2 times, most recently
from
e662cf4 to
791b1fa
Compare
This keeps all the environment variables sane and allows for
out-of-this-tree builds.
Usage:
cmake -B ../build -DINSTALL_ROOT=$some_dir/install
cd to ../build and issue a, "make tools" command to build everything.
Signed-off-by: Sid Manning <[email protected]>
SidManning
force-pushed
the
sidneym/add-cmake
branch
from
791b1fa to
082478c
Compare
|
I was driving a variation of this with a Makefile but since cmake is the new thing I changed it over and updated to match this baseline. |
androm3da
force-pushed
the
bcain/eld_support
branch
3 times, most recently
from
fe05f1a to
0233429
Compare
androm3da
force-pushed
the
bcain/eld_support
branch
4 times, most recently
from
c238eeb to
5e49de6
Compare
androm3da
force-pushed
the
bcain/eld_support
branch
2 times, most recently
from
5909626 to
7767024
Compare
No description provided.