[PATCH] D93031: Enable fexec-charset option

Abhina Sree via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 2 09:01:47 PST 2021


abhina.sreeskantharajan added a comment.

Hi Tom, @tahonermann I renamed the LiteralTranslator class to LiteralConverter.cpp and have renamed a lot of the functions. Let me know what you think. I agree that the setConverters function is awkward, the problem stems from initializing the member early in Preprocessor but only being able to create the Converters once we know the target host later in the compilation process.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93031



More information about the llvm-commits mailing list