[PATCH] D51268: [libc++] Implement P0487R1 - Fixing operator>>(basic_istream&, CharT*)

2018-11-19 Thread Zhihao Yuan via Phabricator via cfe-commits
lichray updated this revision to Diff 174722. lichray added a comment. Update cxx2a status. Repository: rCXX libc++ https://reviews.llvm.org/D51268 Files: include/istream test/std/input.output/iostream.format/input.streams/istream.formatted/istream_extractors/signed_char_pointer.pass.cp

[PATCH] D54724: [Driver] Automatically include C++ library dependencies

2018-11-19 Thread Dimitry Andric via Phabricator via cfe-commits
dim added a comment. I think this is the wrong direction, placing "common" code in `addCXXStdlibLinkDeps`, which then has all kinds of ugly ifs and switches for different OSes? Let different OSes handle this in their own way, maybe. Until now I have not encountered an issue where I needed to a

[PATCH] D54724: [Driver] Automatically include C++ library dependencies

2018-11-19 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In https://reviews.llvm.org/D54724#1303809, @dim wrote: > I think this is the wrong direction, placing "common" code in > `addCXXStdlibLinkDeps`, which then has all kinds of ugly ifs and switches for > different OSes? Let different OSes handle this in their own way, may

[PATCH] D54741: [Driver] Link sanitizer runtime deps on Fuchsia when needed

2018-11-19 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added a reviewer: jakehehrlich. Herald added a subscriber: cfe-commits. Even though these deps weren't needed, this makes Fuchsia driver better match other drivers, and it may be necessary when trying to use different C libraries on Fuchsia. Repository: rC

<    1   2