[libc-commits] [PATCH] D144995: [libc] Fix strcspn

Alex Brachet via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Mar 1 10:58:23 PST 2023


abrachet marked an inline comment as done.
abrachet added inline comments.


================
Comment at: libc/src/string/string_utils.h:169
   for (; *segment; ++segment)
-    bitset.set(*segment);
-  for (; *src && !bitset.test(*src); ++src)
+    bitset.set((uint8_t)*segment);
+  for (; *src && !bitset.test((uint8_t)*src); ++src)
----------------
sivachandra wrote:
> I am guessing that you are seeing a signed vs unsigned problem? If yes, then may be `*reinterpret_cast<const unsigned char *>(segment)` ?
Done in commit


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144995



More information about the libc-commits mailing list