feature/spaceship: Merge VS 2019 16.8 Preview 1 toolset update #1178
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…sion 16.8. (microsoft#1016) Co-authored-by: Casey Carter <cartec69@gmail.com> Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
Co-authored-by: Stephan T. Lavavej <stl@nuwen.net> Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com> Co-authored-by: Casey Carter <cartec69@gmail.com>
Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com> Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
Mathematically, `_Par0._Pcdf.back()` should be one. However, when it is actually slightly smaller than one due to rounding error, there is a small probability that `_Px > _Par0._Pcdf.back()` and the original code returns the invalid value of `_Par0._Pcdf.size()`.
Co-authored-by: Stephan T. Lavavej <stl@nuwen.net> Co-authored-by: Casey Carter <Casey@Carter.net>
Co-authored-by: statementreply <statementreply@gmail.com>
Co-authored-by: Casey Carter <cartec69@gmail.com> Co-authored-by: Stephan T. Lavavej <stl@nuwen.net>
* Remove unnecessary comments in ranges tests. We taught clang-format how to sort like this automatically. * Fix x86 compiler error in stljobs.h. This never compiled for x86 where size_t and SIZE_T are different types. * Update .clang-format for Clang 10. AllowShortBlocksOnASingleLine changed from false/true to Never/Empty/Always. We still want the default of Never, as Empty appears to misbehave. DeriveLineEnding was added, defaulting to true. We want to disable line ending auto-detection. SortPriority was added. Its documentation is difficult to understand, but it can be used to group includes together while modifying their otherwise-lexicographic ordering. Here, I'm using it to group WinSDK includes together, while still sorting WinIoCtl.h last. IncludeIsMainSourceRegex was added. This doesn't appear to be relevant to us. IndentGotoLabels was added. We've avoided goto. SpaceInEmptyBlock was added. We prefer the default of no space. SpacesInConditionalStatement was added. We definitely prefer the default of no space. SpaceBeforeSquareBrackets was added. Again, we definitely prefer the default of no space. The Standard option was overhauled. Previously, Cpp11 meant "use the latest supported standard". That was confusing, so it has been deprecated in favor of Latest. UseCRLF was added. We currently use CRLF for all files. (Previously, validate.cpp would detect LF files, but clang-format wouldn't fix them.) * In .clang-format, delete empty lines and move comment. The empty lines were intended to make it easier to see customized options, but they were actually making it harder to diff the default config against our config. Move the StatementMacros comment next to our customized settings, and rewrap it. * Reformat filesystem.cpp.
Includes the `mergeable` concept and algorithms `ranges::includes`, `ranges::set_union`, `ranges::set_intersection`, `ranges::set_difference`, and `ranges::set_symmetric_difference`.
* Implement ranges::move_backward Co-authored-by: Stephan T. Lavavej <stl@nuwen.net> Co-authored-by: mnatsuhara <46756417+mnatsuhara@users.noreply.github.com>
`_Pass_fn` returns a copy of arguments no larger than `void*`, and a call-forwarding reference-wrapper-alike `_Ref_fn` for larger function objects. This became a problem when Ranges started throwing pointers-to-member at `_Pass_fn` with the expectation that they would eventually be `invoke`d: pointers-to-member can be larger than `void*`, but `_Ref_fn` doesn't speak the `invoke` protocol. There are two relatively obvious possible fixes: 1. Teach `_Pass_fn` to always return copies of pointers-to-member regardless of size 2. Teach `_Ref_fn` to obey the `invoke` protocol (1) results in "large" pointers-to-member that can't be enregistered being copied and passed by hidden reference resulting in larger codesize than (2), but with no corresponding performance benefits. We therefore implement (2) by teaching `_Ref_fn` to call-forward through `invoke` when it wraps a pointer-to-member. Fixes microsoftGH-1089.
…y, and indirect_array (microsoft#988) Co-authored-by: S. B. Tam <cpplearner@outlook.com> Co-authored-by: Casey Carter <cartec69@gmail.com> Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
* Removes workaround for missing `bit_cast` and mark `lerp` constexpr. * Changes how `lerp` handles infinite inputs according to microsoft#65 (comment) and microsoft#65 (comment). * Adds constexpr tests. Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
...by transposing arguments to `_Iter_exchange_move`. Fixes microsoft#1067
Co-authored-by: Casey Carter <Casey@Carter.net>
... for enregistration, to improve performance. Per a suggestion from Statementreply in the discussion at microsoft#1049 (comment).
Fixes microsoft#198. Co-authored-by: Casey Carter <cacarter@microsoft.com> Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com> Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
Co-authored-by: Casey Carter <Casey@Carter.net> Co-authored-by: Stephan T. Lavavej <stl@nuwen.net>
Co-authored-by: Curtis J Bezault <curtbezault@gmail.com> Co-authored-by: Stephan T. Lavavej <stl@nuwen.net> Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com> Co-authored-by: Casey Carter <cartec69@gmail.com>
Co-authored-by: Stephan T. Lavavej <stl@microsoft.com>
Disable some tests due to impact of P0960R3: allow initializing aggregates from a parenthesized list of values Co-authored-by: Stephan T. Lavavej <stl@nuwen.net> Co-authored-by: Casey Carter <cartec69@gmail.com>
…1136) Co-authored-by: Adam Bucior <35536269+AdamBucior@users.noreply.github.com>
Fixes microsoft#1134. Co-authored-by: Casey Carter <cartec69@gmail.com> Co-authored-by: Billy O'Neal <bion@microsoft.com>
CaseyCarter
approved these changes
Aug 10, 2020
This was referenced Aug 10, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge #1153 into
feature/spaceshipso we can delete the old VMSS pool.There was a simple merge conflict in
tests/std/tests/Dev11_0135139_vector_bool_comparisons/test.cpp, wherefeature/spaceshiprenamed the test directory and extended the test so much that git no longer detected it as a rename, whilemasterupdated a bug citation.