[all-commits] [llvm/llvm-project] 37458f: [libc][math] Move str method from FPBits class to ...

lntue via All-commits all-commits at lists.llvm.org
Fri Jun 9 23:51:21 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 37458f66935e1809b4c18a0fee55783ed9e808bb
      https://github.com/llvm/llvm-project/commit/37458f66935e1809b4c18a0fee55783ed9e808bb
  Author: Tue Ly <lntue at google.com>
  Date:   2023-06-10 (Sat, 10 Jun 2023)

  Changed paths:
    M libc/src/__support/FPUtil/CMakeLists.txt
    M libc/src/__support/FPUtil/FPBits.h
    A libc/src/__support/FPUtil/fpbits_str.h
    M libc/src/__support/FPUtil/x86_64/LongDoubleBits.h
    M libc/test/UnitTest/CMakeLists.txt
    M libc/test/UnitTest/ErrnoSetterMatcher.h
    M libc/test/UnitTest/FPMatcher.h
    M libc/test/src/__support/FPUtil/CMakeLists.txt
    M libc/test/src/__support/FPUtil/fpbits_test.cpp
    M libc/utils/MPFRWrapper/CMakeLists.txt
    M libc/utils/MPFRWrapper/MPFRUtils.cpp
    M utils/bazel/llvm-project-overlay/libc/BUILD.bazel
    M utils/bazel/llvm-project-overlay/libc/test/UnitTest/BUILD.bazel
    M utils/bazel/llvm-project-overlay/libc/utils/MPFRWrapper/BUILD.bazel

  Log Message:
  -----------
  [libc][math] Move str method from FPBits class to testing utils.

str method of FPBits class is only used for pretty printing its objects
in tests.  It brings cpp::string dependency to FPBits class, which is not ideal
for embedded use case.  We move str method to a free function in test utils and
remove this dependency of FPBits class.

Reviewed By: sivachandra

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




More information about the All-commits mailing list