[PATCH] D35418: Handle clang-tools-extra project in docker scripts.
Mehdi AMINI via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 14 09:29:03 PDT 2017
mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks.
================
Comment at: utils/docker/scripts/build_install_llvm.sh:71
+ fi
+ CMAKE_LLVM_ENABLE_PROJECTS="$CMAKE_LLVM_ENABLED_PROJECTS$PROJ"
+}
----------------
Why don't you just do everything in the if in one assignment?
https://reviews.llvm.org/D35418
More information about the llvm-commits
mailing list