[PATCH] D46238: [llvm-rc] Add rudimentary support for codepages
Adrian McCarthy via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 30 13:50:51 PDT 2018
amccarth added a comment.
Looks good to me now, but I'm not an official reviewer.
It would be nice if the CP-1252 test exercised one of the unusual characters that you had to build the map for. Since the sources for the tests are stored as binary files, I'm having a hard time peeking into it from Phabricator, so maybe you did, but it's not obvious.
https://reviews.llvm.org/D46238
More information about the llvm-commits
mailing list