Commit 33668ca6 authored by thakis@chromium.org's avatar thakis@chromium.org

Hotfix to make clang-by-default work with gcc 4.8

A better fix is to set clang further up, so that gcc_version is set to
0 when clang is on automatically
( https://codereview.chromium.org/384633005 ). This is however blocked
on nacl also using the "clang" symbol, so that needs to be fixed first
( https://codereview.chromium.org/384793002 ). To make the build go for
people in the meantime, land this hack. I'll revert it once the nacl change has
landed in nacl and then rolled into chromium, and the better fix has landed.

BUG=360311
TBR=thestig@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/379693003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282377 0039d316-1c4b-4281-b951-d872f2087c98
parent 05fbf80a
......@@ -4193,7 +4193,7 @@
# gcc -- http://gcc.gnu.org/onlinedocs/gcc-4.8.0/gcc/Optimize-Options.html
# TODO(mithro): Watch for clang support at following thread:
# http://clang-developers.42468.n3.nabble.com/Adding-fuse-ld-support-to-clang-td4032180.html
['gcc_version>=48', {
['gcc_version>=48 and clang==0', {
'target_conditions': [
['_toolset=="target"', {
'ldflags': [
......@@ -4202,7 +4202,7 @@
}],
],
}],
['host_gcc_version>=48', {
['host_gcc_version>=48 and clang==0', {
'target_conditions': [
['_toolset=="host"', {
'ldflags': [
......@@ -4240,7 +4240,7 @@
'-Wl,--disable-new-dtags',
],
}],
['gcc_version>=48', {
['gcc_version>=48 and clang==0', {
'target_conditions': [
['_toolset=="target"', {
'cflags_cc': [
......@@ -4253,7 +4253,7 @@
}],
],
}],
['host_gcc_version>=48', {
['host_gcc_version>=48 and clang==0', {
'target_conditions': [
['_toolset=="host"', {
'cflags_cc': [
......@@ -5487,7 +5487,7 @@
# Don't warn about the "typedef 'foo' locally defined but not used"
# for gcc 4.8.
# TODO: remove this flag once all builds work. See crbug.com/227506
['gcc_version>=48', {
['gcc_version>=48 and clang==0', {
'target_defaults': {
'cflags': [
'-Wno-unused-local-typedefs',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment