[PATCH] D39537: Rename identifiers named `__output`

2017-11-14 Thread Alexander Richardson via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318144: Rename identifiers named `__output` (authored by arichardson). Repository: rL LLVM https://reviews.llvm.org/D39537 Files: libcxx/trunk/include/algorithm libcxx/trunk/include/experimental/a

[PATCH] D39537: Rename identifiers named `__output`

2017-11-07 Thread Alexander Richardson via Phabricator via cfe-commits
arichardson added inline comments. Comment at: test/support/nasty_macros.hpp:55 +#define __output NASTY_MACRO +#define __input NASTY_MACRO EricWF wrote: > Shouldn't these not be defined when running the tests with a CHER compiler? > Otherwise the macro will co

[PATCH] D39537: Rename identifiers named `__output`

2017-11-06 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. Comment at: test/support/nasty_macros.hpp:55 +#define __output NASTY_MACRO +#define __input NASTY_MACRO Shouldn't these not be

[PATCH] D39537: Rename identifiers named `__output`

2017-11-02 Thread Alexander Richardson via Phabricator via cfe-commits
arichardson created this revision. In the CHERI clang compiler __output and __input are keywords and therefore we can't compile libc++ with our compiler. Repository: rL LLVM https://reviews.llvm.org/D39537 Files: include/algorithm include/experimental/algorithm include/experimental/ite