[llvm-commits] [PATCH] [ADT] Add the range library based on the C++1y proposal.

Michael Spencer bigcheesegs at gmail.com
Sat Jan 12 16:38:54 PST 2013



================
Comment at: unittests/ADT/RangeTest.cpp:76
@@ +75,3 @@
+  std::vector<int> v(5);
+  std::iota(v.begin(), v.end(), 0);
+  llvm::range<int *> r = v;
----------------
In clang with -O3 and libc++, v has the value {0, 0, 0, 0, 0} directly after this call (verified by adding code to print out the value). This really looks like a clang bug to me.

================
Comment at: unittests/ADT/RangeTest.cpp:100
@@ +99,3 @@
+  std::list<int> l(5);
+  std::iota(l.begin(), l.end(), 0);
+  takes_range(make_range(l));
----------------
Same here.


http://llvm-reviews.chandlerc.com/D293



More information about the llvm-commits mailing list