[all-commits] [llvm/llvm-project] a9ee8e: Create a EncodingConverter class with both iconv a...

[Abhina Sree via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a9ee8e4a454ec01fefba8829d2847527aa80623f
      https://github.com/llvm/llvm-project/commit/a9ee8e4a454ec01fefba8829d2847527aa80623f
  Author: Abhina Sree <Abhina.Sreeskantharajan at ibm.com>
  Date:   2025-05-20 (Tue, 20 May 2025)

  Changed paths:
    M llvm/CMakeLists.txt
    M llvm/cmake/config-ix.cmake
    M llvm/include/llvm/Config/config.h.cmake
    A llvm/include/llvm/Support/TextEncoding.h
    M llvm/lib/Support/CMakeLists.txt
    A llvm/lib/Support/TextEncoding.cpp
    M llvm/unittests/Support/CMakeLists.txt
    M llvm/unittests/Support/ConvertEBCDICTest.cpp
    A llvm/unittests/Support/TextEncodingTest.cpp

  Log Message:
  -----------
  Create a EncodingConverter class with both iconv and icu support. (#138893)

This patch adds a wrapper class called EncodingConverter for
ConverterEBCDIC. This class is then extended to support the ICU library
or iconv library. The ICU library currently takes priority over the
iconv library.

Relevant RFCs:

https://discourse.llvm.org/t/rfc-adding-a-charset-converter-to-the-llvm-support-library/69795

https://discourse.llvm.org/t/rfc-enabling-fexec-charset-support-to-llvm-and-clang-reposting/71512

Stacked PR to enable fexec-charset that depends on this:
https://github.com/llvm/llvm-project/pull/138895

See old PR for review and commit history:
https://github.com/llvm/llvm-project/pull/74516



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications