[all-commits] [llvm/llvm-project] 5a1958: Syndicate, test and fix base64 implementation

[serge-sans-paille via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5a1958f2673f8c771e406a7e309e160b432c9a79
      https://github.com/llvm/llvm-project/commit/5a1958f2673f8c771e406a7e309e160b432c9a79
  Author: serge-sans-paille <sguelton at redhat.com>
  Date:   2020-03-02 (Mon, 02 Mar 2020)

  Changed paths:
    M clang-tools-extra/clangd/SemanticHighlighting.cpp
    M compiler-rt/lib/fuzzer/FuzzerUtil.cpp
    A llvm/include/llvm/Support/Base64.h
    A llvm/unittests/Support/Base64Test.cpp
    M llvm/unittests/Support/CMakeLists.txt

  Log Message:
  -----------
  Syndicate, test and fix base64 implementation

Move Base64 implementation from clangd/SemanticHighlighting to
llvm/Support/Base64, fix its implementation and provide a decent test suite.

Previous implementation code was using + operator instead of | to combine some
results, which is a problem when shifting signed values. (0xFF << 16) is
implicitly converted to a (signed) int, and thus results in 0xffff0000, which is
negative. Combining negative numbers with a + in that context is not what we
want to do.

This fixes https://github.com/llvm/llvm-project/issues/149.

Differential Revision: https://reviews.llvm.org/D75057