aboutsummaryrefslogtreecommitdiff
path: root/compiler_wrapper/cros_llvm_next_flags.go
diff options
context:
space:
mode:
authorGeorge Burgess IV <gbiv@google.com>2020-06-11 14:27:13 -0700
committerGeorge Burgess <gbiv@chromium.org>2020-06-15 22:30:38 +0000
commita6ad382ccc7f60ed9feda06d8fcbab62980c62f8 (patch)
tree1ff04f60665c1c81a49044bd54e35c44c6efc4d9 /compiler_wrapper/cros_llvm_next_flags.go
parent90f20eb35d5fd4bf1f63ca2ce751e60fca8b3143 (diff)
downloadtoolchain-utils-a6ad382ccc7f60ed9feda06d8fcbab62980c62f8.tar.gz
wrapper: allow Android to specify their own llvm-flags
Given the discussion on https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1330695, it sounds like our Android friends want to be able to tweak llvm-next flags without going through chromium-review + having to pull their changes back into Android. This CL allows them to make changes to their llvm-next flags entirely locally, and isolates CrOS' llvm-next flags from Android's. BUG=None TEST=Built the wrapper for CrOS and Android. Change-Id: If1bcfe762a4f368db9816557752651edc877f497 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2247003 Reviewed-by: Pirama Arumuga Nainar <pirama@google.com> Reviewed-by: Tiancong Wang <tcwang@google.com> Tested-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'compiler_wrapper/cros_llvm_next_flags.go')
-rw-r--r--compiler_wrapper/cros_llvm_next_flags.go18
1 files changed, 18 insertions, 0 deletions
diff --git a/compiler_wrapper/cros_llvm_next_flags.go b/compiler_wrapper/cros_llvm_next_flags.go
new file mode 100644
index 00000000..6cd7cd2a
--- /dev/null
+++ b/compiler_wrapper/cros_llvm_next_flags.go
@@ -0,0 +1,18 @@
+// Copyright 2020 The Chromium OS Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// +build !android_llvm_next_flags
+
+package main
+
+// This file defines extra flags for llvm-next testing for Chrome OS. Importantly, these flags don't
+// apply to Android's llvm-next wrapper. Android's toolchain-utils copy has a
+// `android_llvm_next_flags.go` file downstream that defines its llvm-next arguments. As you can
+// probably infer, `android_llvm_next_flags.go` is only compiled if the `android_llvm_next_flags`
+// tag is set.
+
+// TODO: Enable test in config_test.go, once we have new llvm-next flags.
+var llvmNextFlags = []string{}
+
+var llvmNextPostFlags = []string{}