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

Issue 2928503002: Remove some no-longer-needed workarounds in libc++abi (Closed)

Created:
3 years, 6 months ago by Tom Anderson
Modified:
3 years, 6 months ago
Reviewers:
hans
Target Ref:
refs/heads/master
Project:
buildtools
Visibility:
Public.

Description

Remove some no-longer-needed workarounds in libc++abi This CL: * Removes libc++abi_warnings to workaround [1], since that bug was fixed * Makes libc++abi always a source_set, and only excludes cxa_thread_atexit.cpp on mac (see [2] for context). * Limits the visibility of libc++abi to libc++. This is to prevent abusing including it like in [3]. [1] https://bugs.llvm.org/show_bug.cgi?id=25978 [2] https://bugs.llvm.org/show_bug.cgi?id=33272 [3] https://chromium.googlesource.com/chromium/deps/icu.git/+/2341038bf72869a5683a893a2b319a48ffec7f62 R=hans@chromium.org Committed: b7a35318665f61f31e811e8cafba44c9c2c52132

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -23 lines) Patch
M third_party/libc++abi/BUILD.gn View 4 chunks +5 lines, -23 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
Tom Anderson
hans ptal
3 years, 6 months ago (2017-06-05 23:10:28 UTC) #2
hans
lgtm Ultra nit: consistent URLs for the LLVM bug tracker in the commit message? :-)
3 years, 6 months ago (2017-06-05 23:18:48 UTC) #3
Tom Anderson
On 2017/06/05 23:18:48, hans wrote: > lgtm > > Ultra nit: consistent URLs for the ...
3 years, 6 months ago (2017-06-05 23:20:35 UTC) #5
Tom Anderson
3 years, 6 months ago (2017-06-05 23:22:08 UTC) #7
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
b7a35318665f61f31e811e8cafba44c9c2c52132 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698