[libcxx-commits] [PATCH] D116346: [libc++] Fix bug #52902

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Jan 1 10:51:33 PST 2022


Quuxplusone added inline comments.


================
Comment at: libcxx/test/std/strings/basic.string/string.iterators/iterators.pass.cpp:58
+        i2 = i1;
+        assert ( i1 == i2 );
+    }
----------------
Before line 57, let's `assert ( i1 != i2 );`
Notice that this test won't work until std::string becomes constexpr, which is why it's `#if`'ed out on line 82 below.
...Waitaminute. Since we're still waiting for CI to come back, anyway, could you please investigate why this godbolt (using `span` instead of `string` to get its `__wrap_iter`s) does //not// reproduce the bug? I think it //should// fail, but I must be missing something.
https://godbolt.org/z/4aGTK79b5



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116346/new/

https://reviews.llvm.org/D116346



More information about the libcxx-commits mailing list