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/config.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/config.go')
| -rw-r--r-- | compiler_wrapper/config.go | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/compiler_wrapper/config.go b/compiler_wrapper/config.go index fc43a29b..e82e6b3d 100644 --- a/compiler_wrapper/config.go +++ b/compiler_wrapper/config.go @@ -93,11 +93,6 @@ func getConfig(configName string, useCCache bool, useLlvmNext bool, version stri return &cfg, nil } -// TODO: Enable test in config_test.go, once we have new llvm-next flags. -var llvmNextFlags = []string{} - -var llvmNextPostFlags = []string{} - // Full hardening. // Temporarily disable function splitting because of chromium:434751. var crosHardenedConfig = &config{ |