Skip to content

[libc++][ranges][test-suite][NFC] Formatted and moved transform_view's sentinel.pass.cpp - #211252

Merged
ldionne merged 1 commit into
llvm:mainfrom
H-G-Hristov:hgh/libcxx/move-test-transform_view
Jul 22, 2026
Merged

[libc++][ranges][test-suite][NFC] Formatted and moved transform_view's sentinel.pass.cpp#211252
ldionne merged 1 commit into
llvm:mainfrom
H-G-Hristov:hgh/libcxx/move-test-transform_view

Conversation

@H-G-Hristov

Copy link
Copy Markdown
Contributor

As a pre-requisite to: #193891

@H-G-Hristov
H-G-Hristov requested a review from a team as a code owner July 22, 2026 13:03
@Zingam Zingam added the ranges Issues related to `<ranges>` label Jul 22, 2026
@llvmorg-github-actions llvmorg-github-actions Bot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Jul 22, 2026
@llvmorg-github-actions

Copy link
Copy Markdown

@llvm/pr-subscribers-libcxx

Author: Hristo Hristov (H-G-Hristov)

Changes

As a pre-requisite to: #193891


Full diff: https://github.com/llvm/llvm-project/pull/211252.diff

1 Files Affected:

  • (renamed) libcxx/test/std/ranges/range.adaptors/range.transform/sentinel/sentinel.pass.cpp (+6-6)
diff --git a/libcxx/test/std/ranges/range.adaptors/range.transform/iterator/sentinel.pass.cpp b/libcxx/test/std/ranges/range.adaptors/range.transform/sentinel/sentinel.pass.cpp
similarity index 89%
rename from libcxx/test/std/ranges/range.adaptors/range.transform/iterator/sentinel.pass.cpp
rename to libcxx/test/std/ranges/range.adaptors/range.transform/sentinel/sentinel.pass.cpp
index ce48e09d6eb8f..b0529916775b3 100644
--- a/libcxx/test/std/ranges/range.adaptors/range.transform/iterator/sentinel.pass.cpp
+++ b/libcxx/test/std/ranges/range.adaptors/range.transform/sentinel/sentinel.pass.cpp
@@ -16,7 +16,7 @@
 #include "test_macros.h"
 #include "../types.h"
 
-template<class T>
+template <class T>
 concept EndIsIter = requires(T t) { ++t.end(); };
 
 constexpr bool test() {
@@ -33,7 +33,7 @@ constexpr bool test() {
 
   std::ranges::transform_view transformView2(SizedSentinelView{4}, PlusOne());
   auto sent4 = transformView2.end();
-  auto iter = transformView1.begin();
+  auto iter  = transformView1.begin();
   {
     assert(iter != sent1);
     assert(iter != sent2);
@@ -47,10 +47,10 @@ constexpr bool test() {
   }
 
   {
-    assert(sent1 - iter  == 8);
-    assert(sent4 - iter  == 4);
-    assert(iter  - sent1 == -8);
-    assert(iter  - sent4 == -4);
+    assert(sent1 - iter == 8);
+    assert(sent4 - iter == 4);
+    assert(iter - sent1 == -8);
+    assert(iter - sent4 == -4);
   }
 
   return true;

@ldionne

ldionne commented Jul 22, 2026

Copy link
Copy Markdown
Member

The libunwind failure is unrelated and we're pursuing a fix independently.

@ldionne
ldionne merged commit 045c898 into llvm:main Jul 22, 2026
195 of 198 checks passed
@H-G-Hristov
H-G-Hristov deleted the hgh/libcxx/move-test-transform_view branch July 22, 2026 18:11
midhuncodes7 pushed a commit to midhuncodes7/llvm-project that referenced this pull request Jul 28, 2026
asudarsa-qti pushed a commit to asudarsa-qti/llvm-project that referenced this pull request Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. ranges Issues related to `<ranges>` test-suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants