[PATCH] D23890: Fix ArrayRef initializer_list Ctor Test

Erich Keane via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 13:11:49 PDT 2016


erichkeane added inline comments.

================
Comment at: unittests/ADT/ArrayRefTest.cpp:150
@@ -148,1 +149,3 @@
 
+static void ArrayRefInitArg(ArrayRef<int> A) {
+  for (int i = 0; i < 5; ++i)
----------------
mehdi_amini wrote:
> Why not turn it into a lambda so it stays local to the particular test that is using it?
Good idea, New patch incoming.


Repository:
  rL LLVM

https://reviews.llvm.org/D23890





More information about the llvm-commits mailing list