Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(16)

Issue 1604353002: PNaCl toolchain: Add 'nostdinc++' to build of host tools against libc++ (Closed)

Created:
4 years, 11 months ago by Derek Schuff
Modified:
4 years, 11 months ago
CC:
native-client-reviews_googlegroups.com
Target Ref:
refs/heads/master
Project:
nacl
Visibility:
Public.

Description

PNaCl toolchain: Add 'nostdinc++' to build of host tools against libc++ Previously, the '-stdlib=libc++' flag was sufficient to remove the system C++ headers from the search path because there were no libc++ headers available. With the recent Chromium clang update, the compiler includes its own libc++ headers, which conflict with our libc++ headers that we manually include. So also add -nostdinc++ so that we only get one set of libc++ headers. R=mcgrathr@chromium.org, stichnot@chromium.org Committed: https://chromium.googlesource.com/native_client/src/native_client/+/7f3183734c7927a12700fcf0a29d6baf340a318b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M toolchain_build/toolchain_build_pnacl.py View 1 chunk +1 line, -0 lines 0 comments Download

Dependent Patchsets:

Messages

Total messages: 8 (3 generated)
Derek Schuff
This gets us back to the previous status quo. We should still switch to static ...
4 years, 11 months ago (2016-01-20 17:58:25 UTC) #1
Derek Schuff
On 2016/01/20 17:58:25, Derek Schuff wrote: > This gets us back to the previous status ...
4 years, 11 months ago (2016-01-20 18:10:17 UTC) #4
Jim Stichnoth
lgtm
4 years, 11 months ago (2016-01-20 18:17:26 UTC) #5
Roland McGrath
lgtm
4 years, 11 months ago (2016-01-20 18:41:39 UTC) #6
Derek Schuff
4 years, 11 months ago (2016-01-20 18:52:57 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
7f3183734c7927a12700fcf0a29d6baf340a318b (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698