diff options
author | George Burgess IV <gbiv@google.com> | 2020-06-11 14:27:13 -0700 |
---|---|---|
committer | George Burgess <gbiv@chromium.org> | 2020-06-15 22:30:38 +0000 |
commit | a6ad382ccc7f60ed9feda06d8fcbab62980c62f8 (patch) | |
tree | 1ff04f60665c1c81a49044bd54e35c44c6efc4d9 /compiler_wrapper/cros_llvm_next_flags.go | |
parent | 90f20eb35d5fd4bf1f63ca2ce751e60fca8b3143 (diff) | |
download | toolchain-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.go | 18 |
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{} |