[all-commits] [llvm/llvm-project] d7917f: [libc] Use cpp::byte instead of char in mem* funct...

Guillaume Chatelet via All-commits all-commits at lists.llvm.org
Mon Oct 24 03:30:51 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d7917fdc0fd25dd032838b781c74e15d7c17400c
      https://github.com/llvm/llvm-project/commit/d7917fdc0fd25dd032838b781c74e15d7c17400c
  Author: Guillaume Chatelet <gchatelet at google.com>
  Date:   2022-10-24 (Mon, 24 Oct 2022)

  Changed paths:
    M libc/src/string/memory_utils/CMakeLists.txt
    M libc/src/string/memory_utils/op_x86.h
    M libc/src/string/memory_utils/utils.h
    M libc/test/src/string/memory_utils/CMakeLists.txt
    M libc/test/src/string/memory_utils/op_tests.cpp

  Log Message:
  -----------
  [libc] Use cpp::byte instead of char in mem* functions

`cpp::byte` is better than `char` which -depending on platform- can be `signed char` or `unsigned char`.  This has introduced subtle arithmetic errors.




More information about the All-commits mailing list