Skip to content

Commit

Permalink
For builtins, only build the builtins target
Browse files Browse the repository at this point in the history
It seemed like a nice idea to specify the scope via cmake and then
just use the generic `install` target but problems like f546294 (Disable
ctx_profile to fix clangrt-builtins, 2024-11-27) happen.  Recently, there's
some similar but yet-unidentified problem causing similar symptoms (attempts
to build sanitizer code in compiler-rt that's not yet supported, e.g.).

Instead, let's just build the stuff we need by specifying the right target.

Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
  • Loading branch information
androm3da committed Jan 23, 2025
1 parent 9fca427 commit b6750a3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build-toolchain.sh
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ build_clang_rt_builtins() {
-B ./obj_clang_rt \
-S ./llvm-project/compiler-rt

cmake --build ./obj_clang_rt -- -v install
cmake --build ./obj_clang_rt -- -v install-builtins
}


Expand Down

0 comments on commit b6750a3

Please sign in to comment.