[PATCH] D17053: [libcxx]: vector: Use < instead of != to improve failure mode

2017-03-29 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF resigned from this revision. EricWF added a comment. This review seems stuck and dead. Resigning as a reviewer. Please re-add me if you want this revision to proceed. https://reviews.llvm.org/D17053 ___ cfe-commits mailing list cfe-commits@l

[libcxx] r299052 - Do not pass an explicit reexported symbol list when building libc++ dylib if also defining new/delete

2017-03-29 Thread Mehdi Amini via cfe-commits
Author: mehdi_amini Date: Wed Mar 29 23:40:56 2017 New Revision: 299052 URL: http://llvm.org/viewvc/llvm-project?rev=299052&view=rev Log: Do not pass an explicit reexported symbol list when building libc++ dylib if also defining new/delete Modified: libcxx/trunk/lib/CMakeLists.txt Modified:

[PATCH] D31486: libc++ testing: allow to provide a path for `use_system_cxx_lib`

2017-03-29 Thread Mehdi AMINI via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL299053: libc++ testing: allow to provide a path for `use_system_cxx_lib` (authored by mehdi_amini). Changed prior to commit: https://reviews.llvm.org/D31486?vs=93432&id=93444#toc Repository: rL LLVM

[PATCH] D31272: Do not pass an explicit reexported symbol list when building libc++ dylib if also defining new/delete

2017-03-29 Thread Mehdi AMINI via Phabricator via cfe-commits
mehdi_amini closed this revision. mehdi_amini added a comment. r299052 ; let me know if you want to improve the comment in any way. https://reviews.llvm.org/D31272 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

[libcxx] r299053 - libc++ testing: allow to provide a path for `use_system_cxx_lib`

2017-03-29 Thread Mehdi Amini via cfe-commits
Author: mehdi_amini Date: Wed Mar 29 23:45:33 2017 New Revision: 299053 URL: http://llvm.org/viewvc/llvm-project?rev=299053&view=rev Log: libc++ testing: allow to provide a path for `use_system_cxx_lib` As we're trying to setup testing / bots for all shipping version of libc++ on macOS/iOS, we'll

[PATCH] D30765: Reexport operator new / delete from libc++abi

2017-03-29 Thread Mehdi AMINI via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL299054: Reexport operator new / delete from libc++abi (authored by mehdi_amini). Changed prior to commit: https://reviews.llvm.org/D30765?vs=91091&id=93445#toc Repository: rL LLVM https://reviews.ll

[libcxx] r299054 - Reexport operator new / delete from libc++abi

2017-03-29 Thread Mehdi Amini via cfe-commits
Author: mehdi_amini Date: Wed Mar 29 23:47:19 2017 New Revision: 299054 URL: http://llvm.org/viewvc/llvm-project?rev=299054&view=rev Log: Reexport operator new / delete from libc++abi Both libc++ and libc++abi export a weak definition of operator new/delete. On Darwin, this can often cause dirty

[PATCH] D17469: [libcxx] Add deployment knobs to tests (for Apple platforms)

2017-03-29 Thread Mehdi AMINI via Phabricator via cfe-commits
mehdi_amini closed this revision. mehdi_amini added a comment. Was committed a while back in r297798 https://reviews.llvm.org/D17469 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxx] r299055 - libc++ config testing: `use_system_cxx_lib` can be a bool

2017-03-29 Thread Mehdi Amini via cfe-commits
Author: mehdi_amini Date: Wed Mar 29 23:51:19 2017 New Revision: 299055 URL: http://llvm.org/viewvc/llvm-project?rev=299055&view=rev Log: libc++ config testing: `use_system_cxx_lib` can be a bool Fix the libc++ Green Dragon bot. Modified: libcxx/trunk/utils/libcxx/test/config.py Modified: l

[libcxx] r299056 - libc++ testing: fix assertion around `use_system_cxx_lib`

2017-03-29 Thread Mehdi Amini via cfe-commits
Author: mehdi_amini Date: Thu Mar 30 00:07:18 2017 New Revision: 299056 URL: http://llvm.org/viewvc/llvm-project?rev=299056&view=rev Log: libc++ testing: fix assertion around `use_system_cxx_lib` Actually fix (hopefully) the assertions about `use_system_cxx_lib`, the previous attempt failed becau

[libcxx] r299057 - libc++ testing: fix invalid python syntax

2017-03-29 Thread Mehdi Amini via cfe-commits
Author: mehdi_amini Date: Thu Mar 30 00:21:33 2017 New Revision: 299057 URL: http://llvm.org/viewvc/llvm-project?rev=299057&view=rev Log: libc++ testing: fix invalid python syntax Modified: libcxx/trunk/utils/libcxx/test/config.py Modified: libcxx/trunk/utils/libcxx/test/config.py URL: http

r299058 - [APInt] Remove references to integerPartWidth and integerPart outside of APFloat implentation.

2017-03-29 Thread Craig Topper via cfe-commits
Author: ctopper Date: Thu Mar 30 00:48:58 2017 New Revision: 299058 URL: http://llvm.org/viewvc/llvm-project?rev=299058&view=rev Log: [APInt] Remove references to integerPartWidth and integerPart outside of APFloat implentation. Turns out integerPartWidth only explicitly defines the width of the

<    1   2