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

Unified Diff: tools/clang/scripts/update.sh

Issue 1385403006: CFI: download LLVM Gold plugin, if requested by env var or GYP_DEFINES. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/clang/scripts/update.sh
diff --git a/tools/clang/scripts/update.sh b/tools/clang/scripts/update.sh
index 8195f990cdbee03dd9b7fa9500d1ae8fdf44f9f5..851ca847eed8d1ecf59bc4827ab37eb9d37ff0c1 100755
--- a/tools/clang/scripts/update.sh
+++ b/tools/clang/scripts/update.sh
@@ -53,6 +53,12 @@ if [[ -z "$LLVM_DOWNLOAD_GOLD_PLUGIN" ]]; then
LLVM_DOWNLOAD_GOLD_PLUGIN=
fi
+if [[ "${OS}" == "Linux" ]] && \
+ [[ "$GYP_DEFINES" =~ .*buildtype=Official.* ]] && \
+ [[ "$GYP_DEFINES" =~ .*branding=Chrome.* ]] ; then
+ # LLVM Gold plugin is required to build with this configuration.
+ LLVM_DOWNLOAD_GOLD_PLUGIN=1
+fi
# Die if any command dies, error on undefined variable expansions.
set -eu
@@ -235,7 +241,10 @@ if [[ -f "${STAMP_FILE}" ]]; then
[[ "${PREVIOUSLY_BUILT_REVISON}" = \
"${PACKAGE_VERSION}" ]]; then
echo "Clang already at ${PACKAGE_VERSION}"
- exit 0
+ if [[ -z "${LLVM_DOWNLOAD_GOLD_PLUGIN}" ]] || \
+ [[ -f "${LLVM_BUILD_DIR}/lib/LLVMgold.so" ]]; then
+ exit 0
+ fi
fi
fi
# To always force a new build if someone interrupts their build half way.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698