[PATCH] D38209: [Sema] Correct nothrow inherited by noexcept

STL_MSFT added a comment. > do you think `__declspec(nothrow)` calling the terminate handler in Clang is > a bug? It's certainly a behavior difference with potential performance impact, although I don't think it can be viewed as a bug, strictly speaking. MSVC treats `__declspec(nothrow)` as an

[PATCH] D38646: [MS] Raise the default value of _MSC_VER to 1910, which is in VS 2017

STL_MSFT added a comment. FYI: 1910 was the value for VS 2017 RTM. 1911 is the value for VS 2017 15.3, the first toolset update. 1912 will be the value for VS 2017 15.5, the second toolset update. Repository: rL LLVM https://reviews.llvm.org/D38646 ___

[PATCH] D39064: implement __has_unique_object_representations

STL_MSFT added a comment. Looks good to me modulo comments. Comment at: lib/Parse/ParseExpr.cpp:719 /// '__is_union' +/// '__has_unique_object_representations' [MS] /// Should this be marked as MS if it's cross-vendor?

[PATCH] D43273: [libcxx] [test] Fix MSVC warnings and errors.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. [libcxx] [test] Fix MSVC warnings and errors. test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan.pass.cpp test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan_init_op.pass.cpp test/std/numerics/numeric.

[PATCH] D43273: [libcxx] [test] Fix MSVC warnings and errors.

STL_MSFT added a comment. Would pragmas guarded by `_MSC_VER` be acceptable for the truncation warnings? https://reviews.llvm.org/D43273 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

[PATCH] D43273: [libcxx] [test] Fix MSVC warnings and errors.

STL_MSFT updated this revision to Diff 135352. STL_MSFT added a comment. Update based on code review feedback. https://reviews.llvm.org/D43273 Files: test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan.pass.cpp test/std/numerics/numeric.ops/exclusive.scan/exclusive_scan_init_op.pass

[PATCH] D33741: [libc++] Undef min/max in test_macros.h

STL_MSFT added a comment. I haven't seen min/max test failures, probably because our CRT/STL headers never drag in Windows.h. I have no objection to undeffing min/max although I wouldn't do this myself (it creates order dependencies which I think are totally evil). https://reviews.llvm.org/D3

[PATCH] D33953: [libcxx] [test] Add more tests to tuple_size_structured_bindings.pass.cpp and make it friendlier to C1XX.

STL_MSFT created this revision. [libcxx] [test] Add more tests to tuple_size_structured_bindings.pass.cpp and make it friendlier to C1XX. Style/paranoia: 42.1 doesn't have an exact binary representation. Although this doesn't cause failures, it makes me uncomfortable, so I'm changing it to 42.5

[PATCH] D33955: [libcxx] [test] Remove a Clang/C2 workaround.

STL_MSFT created this revision. [libcxx] [test] Remove a Clang/C2 workaround. Clang/LLVM doesn't need this workaround. https://reviews.llvm.org/D33955 Files: test/std/utilities/optional/optional.object/optional.object.mod/reset.pass.cpp Index: test/std/utilities/optional/optional.object/o

[PATCH] D33953: [libcxx] [test] Add more tests to tuple_size_structured_bindings.pass.cpp and make it friendlier to C1XX.

STL_MSFT added a comment. Ping? https://reviews.llvm.org/D33953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D34534: [libcxx] [test] Fix MSVC warning C4242 "conversion from 'int' to 'const char', possible loss of data".

STL_MSFT created this revision. [libcxx] [test] Fix MSVC warning C4242 "conversion from 'int' to 'const char', possible loss of data". https://reviews.llvm.org/D34534 Files: test/std/input.output/iostream.format/ext.manip/put_money.pass.cpp test/std/input.output/iostream.format/input.stre

[PATCH] D34535: [libcxx] [test] Fix MSVC warning C4067 "unexpected tokens following preprocessor directive - expected a newline".

STL_MSFT created this revision. [libcxx] [test] Fix MSVC warning C4067 "unexpected tokens following preprocessor directive - expected a newline". Also fixes Clang/LLVM 4.0 (for Windows) error "function-like macro 'TEST_GLIBC_PREREQ' is not defined". https://reviews.llvm.org/D34535 Files:

[PATCH] D34536: [libcxx] [test] Fix Clang -Wunused-local-typedef warnings.

STL_MSFT created this revision. [libcxx] [test] Fix Clang -Wunused-local-typedef warnings. https://reviews.llvm.org/D34536 Files: test/std/numerics/complex.number/complex.transcendentals/acos.pass.cpp test/std/numerics/complex.number/complex.transcendentals/acosh.pass.cpp test/std/numeric

[PATCH] D41213: [libcxx] [test] Improve MSVC portability.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. Herald added a subscriber: jfb. [libcxx] [test] Improve MSVC portability. test/support/msvc_stdlib_force_include.hpp When testing MSVC's STL with C1XX, simulate a couple more compiler feature-test macros. When testin

[PATCH] D41213: [libcxx] [test] Improve MSVC portability.

STL_MSFT added a comment. Ping? (And happy new year!) https://reviews.llvm.org/D41213 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D41213: [libcxx] [test] Improve MSVC portability.

STL_MSFT added a comment. Ping? (And happy Patch Tuesday!) https://reviews.llvm.org/D41213 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D41213: [libcxx] [test] Improve MSVC portability.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, I've checked this in as-is. I would support an LWG issue to change the Standardese here. https://reviews.llvm.org/D41213 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http:/

[PATCH] D53912: [Headers] [MS] Add intrin0.h

STL_MSFT added a comment. Yes, the "real builtin" approach seems to be best. For a recent example, https://reviews.llvm.org/D49606 added `__shiftright128` as an inline function in intrin.h, but that didn't work with MSVC's STL when I moved our declaration of `__shiftright128` from intrin.h to i

[PATCH] D51868: [libcxx] Build and test fixes for Windows

STL_MSFT added inline comments. Comment at: test/support/test_macros.h:147 -# elif defined(_WIN32) -#if defined(_MSC_VER) && !defined(__MINGW32__) -# define TEST_HAS_C11_FEATURES // Using Microsoft's C Runtime library compnerd wrote: > I think that the

[PATCH] D52843: Update Clang Windows getting started docs

STL_MSFT added inline comments. Comment at: clang/www/get_started.html:160 http://www.cmake.org/cmake/resources/software.html";> http://www.cmake.org/cmake/resources/software.html +Visual Studio 2015 or later This URL redirects to `https://c

[PATCH] D17444: [MSVC] Recognize "static_assert" keyword in C mode

STL_MSFT added a comment. @rnk I've forwarded this to the compiler front-end and Universal CRT teams. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D17444/new/ https://reviews.llvm.org/D17444 ___ cfe-co

[PATCH] D38209: [Sema] Correct nothrow inherited by noexcept

STL_MSFT added a comment. This sounds right to me and the test looks good. (I'll let an actual compiler dev sign off) https://reviews.llvm.org/D38209 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/list

[PATCH] D45594: [libcxx] [test] Silence MSVC warning C4146.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. [libcxx] [test] Silence MSVC warning C4146. This test code triggers the MSVC warning: "unary minus operator applied to unsigned type, result still unsigned" Although it would be possible to change the test code to av

[PATCH] D45595: [libcxx] [test] Fix nodiscard warnings.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. [libcxx] [test] Fix nodiscard warnings. MSVC's STL has marked to_bytes/from_bytes as nodiscard. https://reviews.llvm.org/D45595 Files: test/std/localization/locales/locale.convenience/conversions/conversions.stri

[PATCH] D45596: [libcxx] [test] Avoid unary_function.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. [libcxx] [test] Avoid unary_function. Replace unary_function inheritance (which was never required, even in C++98) with argument_type and result_type typedefs. This increases portability, as unary_function was removed

[PATCH] D45594: [libcxx] [test] Silence MSVC warning C4146.

STL_MSFT closed this revision. STL_MSFT added a comment. Committed, with a follow-up to use the C1XX macro. https://reviews.llvm.org/D45594 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co

[PATCH] D45016: [libcxx] [test] Avoid MSVC truncation warnings.

STL_MSFT accepted this revision. STL_MSFT added a comment. This revision is now accepted and ready to land. Approved by Eric. https://reviews.llvm.org/D45016 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailm

[PATCH] D45595: [libcxx] [test] Fix nodiscard warnings.

STL_MSFT closed this revision. STL_MSFT added a comment. Committed with a follow-up to use the requested macro. https://reviews.llvm.org/D45595 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[PATCH] D48616: Implement LWG 2946, 3075 and 3076

STL_MSFT added inline comments. Comment at: test/std/strings/basic.string/string.cons/string_view_deduction.fail.cpp:26 +// The deduction guide shall not participate in overload resolution if Allocator is +// is a type that does not qualify as an allocator. +

[PATCH] D49338: Implement - P0122R7

STL_MSFT added inline comments. Comment at: test/std/containers/views/span.comparison/op.eq.pass.cpp:23 +#include +#include + The comparison tests appear to be unnecessarily including ``. Comment at: test/std/containers/views/span.cons/assign

[PATCH] D69363: [www] Change URLs to HTTPS.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, ldionne, mclow.lists, zoecarver. Herald added a reviewer: bollu. Herald added subscribers: llvm-commits, arphaman, dexonsmith, christof. Herald added a reviewer: jdoerfert. Herald added projects: clang, LLVM. This changes most URLs

[PATCH] D69363: [www] Change URLs to HTTPS.

STL_MSFT marked an inline comment as done. STL_MSFT added inline comments. Comment at: clang/www/cxx_status.html:78 Rvalue references - http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2006/n2118.html";>N2118 + https://wg21.link/n2118";>N2118 Clang 2.9 -

[PATCH] D69363: [www] Change URLs to HTTPS.

STL_MSFT closed this revision. STL_MSFT added a comment. Committed: https://github.com/llvm/llvm-project/commit/2e4f1e112dfee1d16c138d42f2bc7ee639f9ae05 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69363/new/ https://reviews.llvm.org/D69363 _

[PATCH] D40991: [libcxx] [test] Fix line endings, avoid unnecessary non-ASCII.

STL_MSFT created this revision. [libcxx] [test] Fix line endings, avoid unnecessary non-ASCII. I recently wrote a tool to audit MSVC's codebase for inconsistent line endings and unnecessary non-ASCII characters, and I ran it over libcxx's codebase too. I don't need any of these changes to be co

[PATCH] D41030: [libcxx] [test] Fix MSVC warnings, null pointer deref.

STL_MSFT created this revision. [libcxx] [test] Fix MSVC warnings, null pointer deref. test/std/algorithms/alg.modifying.operations/alg.generate/generate_n.pass.cpp Silence MSVC warning C4244. This is expected when passing floating-point values for size. test/std/utilities/template.bitset/bitset

[PATCH] D40991: [libcxx] [test] Fix line endings, avoid unnecessary non-ASCII.

STL_MSFT added a comment. Would you like me to drop the changes to TODO.TXT? https://reviews.llvm.org/D40991 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D40991: [libcxx] [test] Fix line endings, avoid unnecessary non-ASCII.

STL_MSFT accepted this revision. STL_MSFT added a comment. This revision is now accepted and ready to land. Thanks, I've checked this in without the changes to TODO.TXT. Please let me know if you still want them, otherwise I consider this to be complete. https://reviews.llvm.org/D40991 _

[PATCH] D105951: [clang] P2266 implicit moves STL workaround

STL_MSFT added a comment. FYI, we merged https://github.com/microsoft/STL/pull/2025 fixing the 2 affected `return _Istr;` occurrences in `` and `` for VS 2022 17.0 Preview 4. If you find any other affected return statements, please let us know ASAP (as the VS release process locks down the rele

[PATCH] D66770: Move EH spec mismatches under -fms-compatibility

STL_MSFT added a comment. This will definitely help us clean up our code, since we run the MSVC STL test suite with `-fno-ms-compatibility -fno-delayed-template-parsing`. Other possible occurrences of ms-extensions that should be considered ms-compatibility: - ParseDeclCXX.cpp says "MSVC permi

[PATCH] D36503: [libcxx] [test] Update for C++17 feature removals.

STL_MSFT created this revision. [libcxx] [test] Update for C++17 feature removals. test/std/containers/Emplaceable.h test/std/containers/NotConstructible.h test/support/counting_predicates.hpp Replace unary_function/binary_function inheritance with typedefs. test/std/depr/depr.function.objects/d

[PATCH] D36860: [Driver] Recognize DevDiv internal builds of MSVC, with a different directory structure

STL_MSFT created this revision. This is a minimally intrusive change, which I've verified works for both x86 and x64. The idea is to add another bool data member, `IsDevDivInternal`. (This could be unified with `IsVS2017OrNewer`, since it's a 3-way setting. Either a build is DevDiv-internal, r

[PATCH] D36503: [libcxx] [test] Update for C++17 feature removals.

STL_MSFT added a comment. Ping? https://reviews.llvm.org/D36503 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36860: [Driver] Recognize DevDiv internal builds of MSVC, with a different directory structure

STL_MSFT updated this revision to Diff 111776. STL_MSFT edited the summary of this revision. STL_MSFT added a comment. This addresses Don Hinton's feedback. It also unifies `IsVS2017OrNewer` and `IsDevDivInternal` into a 3-state `enum class ToolsetLayout`. https://reviews.llvm.org/D36860 Files

[PATCH] D36860: [Driver] Recognize DevDiv internal builds of MSVC, with a different directory structure

STL_MSFT marked an inline comment as done. STL_MSFT added a comment. In https://reviews.llvm.org/D36860#845469, @thakis wrote: > This approach looks good to me. Thanks! > Is there any chance we could have a test for this? Unfortunately, testing this involves having a VS toolset with the DevD

[PATCH] D36860: [Driver] Recognize DevDiv internal builds of MSVC, with a different directory structure

STL_MSFT added a comment. In https://reviews.llvm.org/D36860#846232, @thakis wrote: > Many driver tests check in a basic representative directory structure (e.g. > test/Driver/Inputs/basic_freebsd_tree/ and its many siblings). > > But if you're happy with others breaking this code, I suppose hav

[PATCH] D36503: [libcxx] [test] Update for C++17 feature removals.

STL_MSFT added a comment. It's been two weeks - if there are no objections, I'd like to commit this soon. Thanks! https://reviews.llvm.org/D36503 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo

[PATCH] D37024: [libcxx] [test] Cleanup nullopt_t tests

STL_MSFT added a comment. Otherwise, looks cromulent to me. Comment at: test/std/utilities/optional/optional.nullopt/nullopt_t.pass.cpp:38 -static_assert(test(nullopt) == 3, ""); +static_assert(std::is_same_v); +static_assert(test()); You're sayin

[PATCH] D36503: [libcxx] [test] Update for C++17 feature removals.

STL_MSFT accepted this revision. STL_MSFT added a comment. This revision is now accepted and ready to land. Looks good to myself. https://reviews.llvm.org/D36503 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/m

[PATCH] D134468: [Driver] Ignore -fmsc-version= -fms-compatibility-version= values smaller than 19

STL_MSFT added a comment. A couple of notes/reminders that may help you make a decision here (I have no particular opinion as this doesn't impact how VS and microsoft/STL use Clang): 1. VS 2013's support lifecycle, documented at https://learn.microsoft.com/en-us/lifecycle/products/visual-studio

[PATCH] D69981: [www] More HTTPS and outdated link fixes.

STL_MSFT created this revision. STL_MSFT added reviewers: ldionne, zoecarver. Herald added a reviewer: bollu. Herald added subscribers: llvm-commits, arphaman, dexonsmith. Herald added projects: clang, LLVM. [www] More HTTPS and outdated link fixes. Repository: rG LLVM Github Monorepo https:/

[PATCH] D69981: [www] More HTTPS and outdated link fixes.

STL_MSFT accepted this revision. STL_MSFT added a comment. This revision is now accepted and ready to land. Committed: https://github.com/llvm/llvm-project/commit/3a7a22445e806c08f80cf6d83d1760f7ff732ed0 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D19031: [clang-format] Flexible line endings

STL_MSFT added a comment. MSVC's STL currently uses CRLF (DOS) line endings, not LF (Unix). I wrote a validator, https://github.com/microsoft/STL/blob/58bb49d63d92e7a0346a05af29816aeea6b4cf0f/tools/validate/validate.cpp , to detect LF files, mixed line endings (LF and CRLF in the same file), d

[PATCH] D70791: Workaround for MSVC 16.3.* pre-c++17 type trait linkage

STL_MSFT added a comment. > I would like to revive D66394 to get it > disabled by default soon. As I recently informed @thakis via email, our compiler documentation was incorrect (I've informed the doc team and they're going to publish a fixed article soon). M

[PATCH] D45016: [libcxx] [test] Avoid MSVC truncation warnings.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. [libcxx] [test] Avoid MSVC truncation warnings. MSVC emits "warning C4244: 'initializing': conversion from 'int' to 'short', possible loss of data" when it sees pair constructed from (whatever, 4), because int is being

[PATCH] D45016: [libcxx] [test] Avoid MSVC truncation warnings.

STL_MSFT added a comment. Ping? https://reviews.llvm.org/D45016 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27270: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 4/4.

STL_MSFT abandoned this revision. STL_MSFT added a comment. Abandoning; I have changes in our STL to fix the bulk of these warnings, plus a small number of test changes that I'm about to send out. https://reviews.llvm.org/D27270 ___ cfe-commits mai

[PATCH] D27777: [libcxx] [test] Fix MSVC x64 truncation warnings with 32-bit allocator size_type/difference_type.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC x64 truncation warnings with 32-bit allocator size_type/difference_type. test/std/containers/container.adaptors/queue/queue.cons.alloc/ctor_container_

[PATCH] D27785: [libcxx] [test] Fix recently introduced warnings emitted by MSVC.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix recently introduced warnings emitted by MSVC. test/std/utilities/allocator.adaptor/allocator.adaptor.members/construct_pair_const_lvalue_pair.pass.cpp test/

[PATCH] D27786: [libcxx] [test] Strip trailing whitespace.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Strip trailing whitespace. https://reviews.llvm.org/D27786 Files: test/std/experimental/utilities/meta/meta.detect/detected_or.pass.cpp test/std/experimen

[PATCH] D28591: [libcxx] [test] Don't ask whether Incomplete& can be assigned to.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Don't ask whether Incomplete& can be assigned to. This is the subject of an active NB comment. Regardless of what the Working Paper currently says, asking this

[PATCH] D28592: [libcxx] [test] Fix MSVC warning C4127 "conditional expression is constant".

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC warning C4127 "conditional expression is constant". MSVC has a compiler warning (enabled at /https://reviews.llvm.org/W4) that's potentially useful, a

[PATCH] D31513: [Sema] Add __is_aggregate type-trait and implement LWG 2015

STL_MSFT added inline comments. Comment at: docs/LanguageExtensions.rst:996 * ``__is_abstract`` (GNU, Microsoft) +* ``__is_aggregate`` (GNU, Microsoft) * ``__is_base_of`` (GNU, Microsoft) aaron.ballman wrote: > EricWF wrote: > > aaron.ballman wrote: > > > Has M

[PATCH] D31966: [libcxx] [test] Avoid Clang's -Wunused-const-variable in is_constructible.pass.cpp.

STL_MSFT created this revision. [libcxx] [test] Avoid Clang's -Wunused-const-variable in is_constructible.pass.cpp. This happens when using Clang with MSVC's STL, so there are no actual uses of this variable. https://reviews.llvm.org/D31966 Files: test/std/utilities/meta/meta.unary/meta.un

[PATCH] D28592: [libcxx] [test] Fix MSVC warning C4127 "conditional expression is constant".

STL_MSFT abandoned this revision. STL_MSFT added a comment. Actually, this pattern annoys "warning C6326: Potential comparison of a constant with another constant." from /analyze. I'll figure out a better fix, probably with integral_constant. https://reviews.llvm.org/D28592

[PATCH] D28837: [libcxx] [test] Fix MSVC warnings C4127 and C6326 about constants.

STL_MSFT created this revision. [libcxx] [test] Fix MSVC warnings C4127 and C6326 about constants. MSVC has compiler warnings C4127 "conditional expression is constant" (enabled by /https://reviews.llvm.org/W4) and C6326 "Potential comparison of a constant with another constant" (enabled by /ana

[PATCH] D29135: [libcxx] [test] Fix Clang -Wunused-local-typedef, part 1/3.

STL_MSFT created this revision. [libcxx] [test] Fix Clang -Wunused-local-typedef, part 1/3. Mark typedefs as LIBCPP_ONLY when their only usage is within LIBCPP_STATIC_ASSERT. test/std/containers/sequences/vector.bool/move_assign_noexcept.pass.cpp test/std/containers/sequences/vector.bool/move_n

[PATCH] D29136: [libcxx] [test] Fix Clang -Wunused-local-typedef, part 2/3.

STL_MSFT created this revision. [libcxx] [test] Fix Clang -Wunused-local-typedef, part 2/3. These typedefs were completely unused. https://reviews.llvm.org/D29136 Files: test/std/containers/sequences/deque/deque.cons/size.pass.cpp test/std/containers/sequences/list/list.cons/size_type.pass

[PATCH] D29137: [libcxx] [test] Fix Clang -Wunused-local-typedef, part 3/3.

STL_MSFT created this revision. [libcxx] [test] Fix Clang -Wunused-local-typedef, part 3/3. test/std/strings/string.classes/typedefs.pass.cpp Actually test what basic_string's typedefs stand for. test/std/utilities/meta/meta.trans/meta.trans.other/result_of11.pass.cpp NotDerived and ND were comp

[PATCH] D29138: [libcxx] [test] Fix Clang -Wdeprecated-declarations with MSVC's CRT.

STL_MSFT created this revision. [libcxx] [test] Fix Clang -Wdeprecated-declarations with MSVC's CRT. libcxx's tests use various C Standard Library functions that have been marked by MSVC's CRT as deprecated by Microsoft (not by ISO). libcxx's usage is cromulent (just checking with decltype to se

[PATCH] D29139: [libcxx] [test] Fix Clang -Wpessimizing-move "moving a temporary object prevents copy elision".

STL_MSFT created this revision. [libcxx] [test] Fix Clang -Wpessimizing-move "moving a temporary object prevents copy elision". N4618 30.6.6 [futures.unique_future]/12 declares "shared_future share() noexcept;". https://reviews.llvm.org/D29139 Files: test/std/thread/futures/futures.unique_

[PATCH] D29140: [libcxx] [test] Avoid MSVC's non-Standard ABI in underlying_type.pass.cpp.

STL_MSFT created this revision. [libcxx] [test] Avoid MSVC's non-Standard ABI in underlying_type.pass.cpp. When compiled with Clang for Windows, this was emitting "enumerator value evaluates to 4294967295, which cannot be narrowed to type 'int' [-Wc++11-narrowing]". The test should more strenuo

[PATCH] D29135: [libcxx] [test] Fix Clang -Wunused-local-typedef, part 1/3.

STL_MSFT accepted this revision. STL_MSFT added a comment. This revision is now accepted and ready to land. Committed with the requested changes. Thanks! https://reviews.llvm.org/D29135 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://

[PATCH] D34536: [libcxx] [test] Fix Clang -Wunused-local-typedef warnings.

STL_MSFT added a comment. Ping? https://reviews.llvm.org/D34536 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D34536: [libcxx] [test] Fix Clang -Wunused-local-typedef warnings.

STL_MSFT added a comment. Yeah, I just wanted to check that you didn't mean to actually use these typedefs for something. Thanks! https://reviews.llvm.org/D34536 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

[PATCH] D34535: [libcxx] [test] Fix MSVC warning C4067 "unexpected tokens following preprocessor directive - expected a newline".

STL_MSFT added a comment. Ping for this one - there's no functional change here, but since I don't have any glibc coverage, I wanted to get a review. https://reviews.llvm.org/D34535 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

[PATCH] D34534: [libcxx] [test] Fix MSVC warning C4242 "conversion from 'int' to 'const char', possible loss of data".

STL_MSFT added a comment. As I've made this exact change in other tests before, I'll commit this soon-ish if there are no comments or objections. https://reviews.llvm.org/D34534 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D42354: Fix libcxx MSVC C++17 redefinition of 'align_val_t'

STL_MSFT added a comment. Seems reasonable to me. Repository: rCXX libc++ https://reviews.llvm.org/D42354 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27266: [libcxx] [test] Remove spurious semicolons.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Remove spurious semicolons. https://reviews.llvm.org/D27266 Files: test/std/containers/unord/unord.multimap/unord.multimap.cnstr/range_size_hash_equal_allo

[PATCH] D27267: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 1/4.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 1/4. Replace "int n = str_.size();" with "int n

[PATCH] D27268: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 2/4.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 2/4. Use static_cast when storing size_t in int

[PATCH] D27269: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 3/4.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 3/4. test/std/containers/sequences/vector.bool/

[PATCH] D27270: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 4/4.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 4/4. Change a few allocators' size_type/differe

[PATCH] D27023: [libcxx] [test] Fix MSVC warning C4389 "signed/unsigned mismatch", part 10/12.

STL_MSFT updated this revision to Diff 80347. STL_MSFT added a comment. Changed loop indices to int as requested. https://reviews.llvm.org/D27023 Files: test/std/algorithms/alg.modifying.operations/alg.move/move.pass.cpp test/std/algorithms/alg.modifying.operations/alg.move/move_backward.pa

[PATCH] D27268: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 2/4.

STL_MSFT updated this revision to Diff 80348. STL_MSFT added a comment. Changed data members to size_t as requested (also changed parameters for consistency). https://reviews.llvm.org/D27268 Files: test/std/utilities/optional/optional.specalg/make_optional_explicit_initializer_list.pass.cpp

[PATCH] D27021: [libcxx] [test] Fix MSVC warning C4389 "signed/unsigned mismatch", part 8/12.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288745. https://reviews.llvm.org/D27021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27022: [libcxx] [test] Fix MSVC warning C4389 "signed/unsigned mismatch", part 9/12.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288746. https://reviews.llvm.org/D27022 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27023: [libcxx] [test] Fix MSVC warning C4389 "signed/unsigned mismatch", part 10/12.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288747. https://reviews.llvm.org/D27023 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27024: [libcxx] [test] Fix MSVC warning C4389 "signed/unsigned mismatch", part 11/12.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288748. https://reviews.llvm.org/D27024 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27266: [libcxx] [test] Remove spurious semicolons.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288750. https://reviews.llvm.org/D27266 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27267: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 1/4.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288751. https://reviews.llvm.org/D27267 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27025: [libcxx] [test] Fix MSVC warning C4389 "signed/unsigned mismatch", part 12/12.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288749. https://reviews.llvm.org/D27025 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27268: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 2/4.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288752. https://reviews.llvm.org/D27268 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D27269: [libcxx] [test] Fix MSVC x64 warning C4267 "conversion from 'size_t' to 'int' [or 'unsigned int'], possible loss of data", part 3/4.

STL_MSFT closed this revision. STL_MSFT added a comment. Thanks, r288753. https://reviews.llvm.org/D27269 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26623: [libcxx] [test] Swapping non-equal non-POCS allocators is UB.

STL_MSFT updated this revision to Diff 80352. STL_MSFT added a comment. Merged with the recent addition of static_cast to various lines in the unordered container tests. No actual changes to the diff, only to the context. https://reviews.llvm.org/D26623 Files: test/std/containers/associativ

[PATCH] D27436: [libcxx] [test] std::get<0>([std::variant constant expression]) *is* noexcept

STL_MSFT added a comment. I also saw get_type.pass.cpp failing. I would recommend commenting the ifndef with the clang bug URL you found. https://reviews.llvm.org/D27436 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

[PATCH] D27538: [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 1/7.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 1/7. Given `std::basic_streambuf::int_type __c`, `std::basic_string str_`, an

[PATCH] D27539: [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 2/7.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 2/7. These tests for some guy's transparent operator functors were needlessly

[PATCH] D27540: [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 3/7.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 3/7. Add static_cast when constructing pair from (Something, int). https://

[PATCH] D27541: [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 4/7.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 4/7. Change char to long and remove some char casts. This preserves test cover

[PATCH] D27542: [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 5/7.

STL_MSFT created this revision. STL_MSFT added reviewers: EricWF, mclow.lists. STL_MSFT added a subscriber: cfe-commits. [libcxx] [test] Fix MSVC warning C4244 "conversion from 'X' to 'Y', possible loss of data", part 5/7. Instead of storing double in double and then truncating to int, store int

  1   2   >