abhina.sreeskantharajan added a comment.

In D153417#4438764 <https://reviews.llvm.org/D153417#4438764>, @efriedma wrote:

> As I mentioned at 
> https://discourse.llvm.org/t/rfc-enabling-fexec-charset-support-to-llvm-and-clang-reposting/71512
>  , I think SimplifyLibCalls needs to be aware of encodings.  To make that 
> work, this probably needs to be somewhere in llvm/ , not clang/ .

There was previous resistance to putting this in LLVM 
https://discourse.llvm.org/t/rfc-adding-a-charset-converter-to-the-llvm-support-library/69795/17,
 I'm not sure if that sentiment has changed. I think it would be best to have 
some sort of implementation plan to tackle the SimplifyLibCalls issue before 
shifting this code


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153417

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to