[PATCH] D10725: Improve testing for the C API

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 14:41:38 PST 2016


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

This is looking pretty good. Definitely for a first start. Thanks for doing this and sticking with it.

-eric


================
Comment at: tools/llvm-c-test/echo.cpp:271
@@ +270,3 @@
+        fprintf(stderr, "Final instruction does not match Last\n");
+        exit(-1);
+      }
----------------
Sure.

================
Comment at: tools/llvm-c-test/echo.cpp:284
@@ +283,3 @@
+    Cur = Next;
+  }
+
----------------
Nothing is coming to mind at the moment, we can just defer it.

================
Comment at: tools/llvm-c-test/echo.cpp:335-338
@@ +334,6 @@
+
+  ValueMap VMap;
+  if (Count == 0)
+    return VMap;
+
+  LLVMValueRef SrcFirst = LLVMGetFirstParam(Src);
----------------
I think I was just wanting a testcase with more than zero parameters when I asked.


http://reviews.llvm.org/D10725





More information about the llvm-commits mailing list