[PATCH] D23890: Fix ArrayRef initializer_list Ctor Test

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 12:40:27 PDT 2016


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

LGTM.


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


Repository:
  rL LLVM

https://reviews.llvm.org/D23890





More information about the llvm-commits mailing list