[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-28 Thread Shoaib Meenai via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL296576: [libc++abi] Clean up visibility (authored by smeenai). Changed prior to commit: https://reviews.llvm.org/D26949?vs=90119&id=90120#toc Repository: rL LLVM https://reviews.llvm.org/D26949 Fil

[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-28 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai updated this revision to Diff 90119. smeenai added a comment. Addressing inline comment https://reviews.llvm.org/D26949 Files: include/__cxxabi_config.h src/abort_message.cpp src/abort_message.h src/cxa_exception.cpp src/cxa_exception.hpp src/cxa_handlers.cpp src/cxa_handl

[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-28 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added inline comments. Comment at: src/abort_message.h:19 -__attribute__((visibility("hidden"), noreturn)) - void abort_message(const char* format, ...) -__attribute__((format(printf, 1, 2))); - +_LIBCXXABI_HIDDEN __attribute__((noreturn)) void +abort

[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-28 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. LGTM minus inline comments. @smeenai confirmed that this patch doesn't change the ABI list for `libc++abi.dylib` using `sym_check.py`. Comment at: src/abort_message.h:19

[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-28 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai updated this revision to Diff 90114. smeenai added a comment. Rebase https://reviews.llvm.org/D26949 Files: include/__cxxabi_config.h src/abort_message.cpp src/abort_message.h src/cxa_exception.cpp src/cxa_exception.hpp src/cxa_handlers.cpp src/cxa_handlers.hpp src/cxa_n

[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-06 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added inline comments. Comment at: src/abort_message.cpp:25 -#pragma GCC visibility push(hidden) - EricWF wrote: > Is this really redundant? There is an `#include ` > after it. Is this not going to affect those symbols? That's a fair point. I think it'

[PATCH] D26949: [libc++abi] Clean up visibility

2017-02-06 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai updated this revision to Diff 87362. smeenai edited the summary of this revision. smeenai added a comment. Addressing comments https://reviews.llvm.org/D26949 Files: include/__cxxabi_config.h src/abort_message.cpp src/abort_message.h src/cxa_exception.cpp src/cxa_exception.hpp

[PATCH] D26949: [libc++abi] Clean up visibility

2017-01-06 Thread Saleem Abdulrasool via Phabricator via cfe-commits
compnerd added inline comments. Comment at: src/cxa_new_delete.cpp:34 */ -__attribute__((__weak__, __visibility__("default"))) +__attribute__((__weak__)) void * EricWF wrote: > Can we abstract this away to a `_LIBCXXABI_NEW_DELETE_VIS` macro? Are `operator new`

[PATCH] D26949: [libc++abi] Clean up visibility

2016-12-29 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF added inline comments. Comment at: src/abort_message.cpp:25 -#pragma GCC visibility push(hidden) - Is this really redundant? There is an `#include ` after it. Is this not going to affect those symbols? Comment at: src/cxa_handlers.hpp

[PATCH] D26949: [libc++abi] Clean up visibility

2016-12-28 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. Ping. https://reviews.llvm.org/D26949 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26949: [libc++abi] Clean up visibility

2016-12-19 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. Ping. https://reviews.llvm.org/D26949 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26949: [libc++abi] Clean up visibility

2016-12-12 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. Ping. https://reviews.llvm.org/D26949 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26949: [libc++abi] Clean up visibility

2016-12-05 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai updated this revision to Diff 80361. smeenai added a comment. Addressing @compnerd's comments https://reviews.llvm.org/D26949 Files: src/abort_message.cpp src/abort_message.h src/cxa_exception.cpp src/cxa_exception.hpp src/cxa_handlers.cpp src/cxa_handlers.hpp src/cxa_new_

[PATCH] D26949: [libc++abi] Clean up visibility

2016-12-05 Thread Saleem Abdulrasool via Phabricator via cfe-commits
compnerd added a comment. I really like the clean up this does. It removes the unnecessary usage of the GCC pragmas and cleans up the definitions by using the headers more properly. Comment at: src/abort_message.h:19 -__attribute__((visibility("hidden"), noreturn)) +_LIBCXX

[PATCH] D26949: [libc++abi] Clean up visibility

2016-11-28 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. Ping. Lmk if there's any other tests I should run on this to ensure there's no functional difference. https://reviews.llvm.org/D26949 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

[PATCH] D26949: [libc++abi] Clean up visibility

2016-11-21 Thread Shoaib Meenai via cfe-commits
smeenai created this revision. smeenai added reviewers: compnerd, EricWF, howard.hinnant, mclow.lists. smeenai added a subscriber: cfe-commits. Use the libc++abi visibility macros instead of pragmas or using visibility attributes directly. Clean up redundant attributes on definitions (where the de