[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