[libc-commits] [PATCH] D110222: [libc] Add an implementation of bsearch.

Michael Jones via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Sep 22 09:01:42 PDT 2021


michaelrj added a comment.

LGTM once comments are addressed



================
Comment at: libc/test/src/stdlib/bsearch_test.cpp:54
+  // Non existent keys
+  for (size_t s = 1; s <= array_size; ++s) {
+    int key = 5;
----------------
why is this a loop?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110222



More information about the libc-commits mailing list