Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 1 | package cc |
| 2 | |
| 3 | import ( |
| 4 | "sort" |
| 5 | "strings" |
| 6 | ) |
| 7 | |
| 8 | // Cflags that should be filtered out when compiling with clang |
Dan Willemsen | e654045 | 2015-10-20 15:21:33 -0700 | [diff] [blame^] | 9 | var clangUnknownCflags = sorted([]string{ |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 10 | "-finline-functions", |
| 11 | "-finline-limit=64", |
| 12 | "-fno-canonical-system-headers", |
Dan Willemsen | 3bf6b47 | 2015-09-11 17:41:10 -0700 | [diff] [blame] | 13 | "-Wno-clobbered", |
| 14 | "-fno-devirtualize", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 15 | "-fno-tree-sra", |
Colin Cross | a360e8b | 2015-03-16 16:22:28 -0700 | [diff] [blame] | 16 | "-fprefetch-loop-arrays", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 17 | "-funswitch-loops", |
| 18 | "-Wmaybe-uninitialized", |
Dan Willemsen | 3bf6b47 | 2015-09-11 17:41:10 -0700 | [diff] [blame] | 19 | "-Wno-error=clobbered", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 20 | "-Wno-error=maybe-uninitialized", |
Colin Cross | 74d1ec0 | 2015-04-28 13:30:13 -0700 | [diff] [blame] | 21 | "-Wno-error=unused-but-set-parameter", |
| 22 | "-Wno-error=unused-but-set-variable", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 23 | "-Wno-free-nonheap-object", |
| 24 | "-Wno-literal-suffix", |
| 25 | "-Wno-maybe-uninitialized", |
| 26 | "-Wno-old-style-declaration", |
| 27 | "-Wno-psabi", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 28 | "-Wno-unused-but-set-parameter", |
Colin Cross | 74d1ec0 | 2015-04-28 13:30:13 -0700 | [diff] [blame] | 29 | "-Wno-unused-but-set-variable", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 30 | "-Wno-unused-local-typedefs", |
Colin Cross | 62ec5f4 | 2015-03-18 17:20:28 -0700 | [diff] [blame] | 31 | "-Wunused-but-set-parameter", |
Colin Cross | 74d1ec0 | 2015-04-28 13:30:13 -0700 | [diff] [blame] | 32 | "-Wunused-but-set-variable", |
Dan Willemsen | e654045 | 2015-10-20 15:21:33 -0700 | [diff] [blame^] | 33 | "-fdiagnostics-color", |
| 34 | "-fdebug-prefix-map=/proc/self/cwd=", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 35 | |
| 36 | // arm + arm64 + mips + mips64 |
| 37 | "-fgcse-after-reload", |
| 38 | "-frerun-cse-after-loop", |
| 39 | "-frename-registers", |
| 40 | "-fno-strict-volatile-bitfields", |
| 41 | |
| 42 | // arm + arm64 |
| 43 | "-fno-align-jumps", |
| 44 | "-Wa,--noexecstack", |
| 45 | |
| 46 | // arm |
| 47 | "-mthumb-interwork", |
| 48 | "-fno-builtin-sin", |
| 49 | "-fno-caller-saves", |
| 50 | "-fno-early-inlining", |
| 51 | "-fno-move-loop-invariants", |
| 52 | "-fno-partial-inlining", |
| 53 | "-fno-tree-copy-prop", |
| 54 | "-fno-tree-loop-optimize", |
| 55 | |
| 56 | // mips + mips64 |
| 57 | "-msynci", |
Dan Willemsen | 3bf6b47 | 2015-09-11 17:41:10 -0700 | [diff] [blame] | 58 | "-mno-synci", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 59 | "-mno-fused-madd", |
| 60 | |
| 61 | // x86 + x86_64 |
| 62 | "-finline-limit=300", |
| 63 | "-fno-inline-functions-called-once", |
| 64 | "-mfpmath=sse", |
| 65 | "-mbionic", |
Dan Willemsen | e654045 | 2015-10-20 15:21:33 -0700 | [diff] [blame^] | 66 | }) |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 67 | |
| 68 | func init() { |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 69 | pctx.StaticVariable("clangExtraCflags", strings.Join([]string{ |
| 70 | "-D__compiler_offsetof=__builtin_offsetof", |
| 71 | |
| 72 | // Help catch common 32/64-bit errors. |
| 73 | "-Werror=int-conversion", |
| 74 | |
Colin Cross | 74d1ec0 | 2015-04-28 13:30:13 -0700 | [diff] [blame] | 75 | // Disable overly aggressive warning for macros defined with a leading underscore |
| 76 | // This happens in AndroidConfig.h, which is included nearly everywhere. |
Dan Willemsen | 3bf6b47 | 2015-09-11 17:41:10 -0700 | [diff] [blame] | 77 | // TODO: can we remove this now? |
Colin Cross | 74d1ec0 | 2015-04-28 13:30:13 -0700 | [diff] [blame] | 78 | "-Wno-reserved-id-macro", |
| 79 | |
| 80 | // Disable overly aggressive warning for format strings. |
| 81 | // Bug: 20148343 |
| 82 | "-Wno-format-pedantic", |
| 83 | |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 84 | // Workaround for ccache with clang. |
| 85 | // See http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html. |
| 86 | "-Wno-unused-command-line-argument", |
| 87 | |
| 88 | // Disable -Winconsistent-missing-override until we can clean up the existing |
| 89 | // codebase for it. |
| 90 | "-Wno-inconsistent-missing-override", |
Dan Willemsen | e654045 | 2015-10-20 15:21:33 -0700 | [diff] [blame^] | 91 | |
| 92 | // Force clang to always output color diagnostics. Ninja will strip the ANSI |
| 93 | // color codes if it is not running in a terminal. |
| 94 | "-fcolor-diagnostics", |
Colin Cross | 3f40fa4 | 2015-01-30 17:27:36 -0800 | [diff] [blame] | 95 | }, " ")) |
| 96 | |
| 97 | pctx.StaticVariable("clangExtraConlyflags", strings.Join([]string{ |
| 98 | "-std=gnu99", |
| 99 | }, " ")) |
| 100 | |
| 101 | pctx.StaticVariable("clangExtraTargetCflags", strings.Join([]string{ |
| 102 | "-nostdlibinc", |
| 103 | }, " ")) |
| 104 | } |
| 105 | |
| 106 | func clangFilterUnknownCflags(cflags []string) []string { |
| 107 | ret := make([]string, 0, len(cflags)) |
| 108 | for _, f := range cflags { |
| 109 | if !inListSorted(f, clangUnknownCflags) { |
| 110 | ret = append(ret, f) |
| 111 | } |
| 112 | } |
| 113 | |
| 114 | return ret |
| 115 | } |
| 116 | |
| 117 | func inListSorted(s string, list []string) bool { |
| 118 | for _, l := range list { |
| 119 | if s == l { |
| 120 | return true |
| 121 | } else if s < l { |
| 122 | return false |
| 123 | } |
| 124 | } |
| 125 | return false |
| 126 | } |
Dan Willemsen | e654045 | 2015-10-20 15:21:33 -0700 | [diff] [blame^] | 127 | |
| 128 | func sorted(list []string) []string { |
| 129 | sort.Strings(list) |
| 130 | return list |
| 131 | } |